Skip to content

docs: add dia page#3042

Merged
pete-vielhaber merged 14 commits intoOffchainLabs:masterfrom
khawlahssn:master
Mar 19, 2026
Merged

docs: add dia page#3042
pete-vielhaber merged 14 commits intoOffchainLabs:masterfrom
khawlahssn:master

Conversation

@khawlahssn
Copy link
Copy Markdown
Contributor

@khawlahssn khawlahssn commented Jan 29, 2026

Thank you for contributing to our docs!

Please fill out the below to ensure your doc gets quickly approved and merged.

Description

Move DIA oracle documentation to the third-party content section and update integration guide.

Document type

  • Gentle introduction
  • Quickstart
  • How-to
  • Concept
  • FAQ
  • Troubleshooting
  • Reference
  • Third-party content
  • Not applicable

Checklist

  • I have read the CONTRIBUTE.md guidelines
  • My changes follow the style conventions outlined in CONTRIBUTE.md
  • I have used sentence-case for titles and headers
  • I have used descriptive link text (not "here" or "this")
  • I have separated procedural from conceptual content where appropriate
  • I have tested my changes locally with yarn start or yarn build
  • My code follows the existing code style and conventions
  • I have added/updated frontmatter for new documents
  • I have checked for broken links
  • I have verified that my changes don't break the build
  • Third-party docs only: Do you agree to the third-party content policy outlined within CONTRIBUTE.md?
    • Yes
    • Not applicable

Additional Notes

Build could not be verified locally, yarn build fails on master due to a missing sdk-sidebar.js. This is a pre-existing issue unrelated to these changes.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 29, 2026

CLA assistant check
All committers have signed the CLA.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 29, 2026

@khawlahssn is attempting to deploy a commit to the Offchain Labs - AF Billed Team on Vercel.

A member of the Team first needs to authorize it.

@anegg0
Copy link
Copy Markdown
Contributor

anegg0 commented Feb 4, 2026

Hi @khawlahssn , and thank you for submitting a PR.
For your content to be reviewed, please submit it as third-party content. You can find more info on CONTRIBUTE.
You might want to consider making your article more substantial: it needs to be a full how-to or tutorial.

@khawlahssn
Copy link
Copy Markdown
Contributor Author

Hi @anegg0, thanks for the feedback! I've made the following changes:

  • Moved the document to docs/for-devs/third-party-docs/DIA/dia.mdx
  • Updated the frontmatter to follow the third-party content format (added third_party_content_owner, author, sme, sidebar_label)
  • Updated the PR checklist accordingly

Note: I wasn't able to verify the build locally, yarn build fails on master due to a missing sdk-sidebar.js (the sdkDocsHandler.ts plugin can't load). This appears to be a pre-existing issue unrelated to my changes.

Regarding making the article more substantial, I'd appreciate any guidance on what you'd like to see expanded. I noticed other oracle docs follow a similar format and length. Happy to add more detail if needed!

@khawlahssn khawlahssn marked this pull request as ready for review February 10, 2026 11:37
@khawlahssn
Copy link
Copy Markdown
Contributor Author

Hi @anegg0, just wanted to follow up on this PR. please let us know if any changes are needed from our end!

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
arbitrum-docs Ready Ready Preview Mar 19, 2026 2:41pm

Request Review

Copy link
Copy Markdown
Contributor

@pete-vielhaber pete-vielhaber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pete-vielhaber pete-vielhaber merged commit 068d351 into OffchainLabs:master Mar 19, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants