Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRAFT: docs-how-arbitrum-works #1866

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

DRAFT: docs-how-arbitrum-works #1866

wants to merge 37 commits into from

Conversation

pete-vielhaber
Copy link
Contributor

  • DRAFT DO NOT MERGE
  • website/docusaurus.config.js
    • lines 14-15 need changed back to 'throw'
  • challenges.mdx
    • line 156 is commented out due to error being thrown by yarn
  • many links are broken, will fix once we have consensus on restructure

Copy link

vercel bot commented Nov 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
arbitrum-docs ✅ Ready (Inspect) Visit Preview Dec 13, 2024 2:51pm

Copy link
Contributor

@TucksonDev TucksonDev left a comment

Choose a reason for hiding this comment

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

Good job @pete-vielhaber !
There are still a few things to change:

  • I left comments in all sections that seem to break the regular reading flow
  • There are many broken links throught all the document
  • We need to place redirections from the old pages to the new ones. That's done in the vercel.json file
  • Some of the old pages are still active, even though they are not in the sidebar anymore. For example, Inside nitro, which is a page that's linked from many different places. If we want to keep the contents of these pages in the repo for a bit, one solution is to move them to website/archive. In any case, all links pointing to these pages should be changed, and redirections should be placed.

Let me know if you have any questions about any of the comments!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants