-
Notifications
You must be signed in to change notification settings - Fork 355
Minor fixes #2125
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
Minor fixes #2125
Conversation
GigaHierz
commented
Jan 22, 2026
- move netowrk overview to build on celo section
- remove ai tutorials
- add goldrush to indexers, Closes Add GoldRush (powered by Covalent) details #2099 , Enhance overview with GoldRush indexer details #2123, Add 'goldrush' indexer to documentation #2124
- update exchanges page for clearer separation, Closes Update exchanges.mdx #2116
- add 1inch wallet to wallet page
- remove ai tutorials - add goldrush to indexers - update exchanges page for clearer separation - add 1inch wallet to wallet page
- fix broken links to network
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Review
Thanks for this cleanup PR! The reorganization and additions look good overall. However, I found a couple of issues with the redirect paths that will cause 404s if merged as-is:
1. Broken redirect destination (line ~2848 in docs.json)
{
"source": "/network",
"destination": "/tooling/network-overview"
}This should be /build-on-celo/network-overview since that's where the file was moved to.
2. Inconsistent redirect paths - build-with-celo vs build-on-celo
Several redirects incorrectly point to /build-with-celo/network-overview (which doesn't exist) instead of /build-on-celo/network-overview:
- Line ~1759:
/getting-started/choosing-a-network→/build-with-celo/network-overview❌ - Line ~1767:
/getting-started/mainnet/index→/build-with-celo/network-overview❌ - Line ~1787:
/getting-started/rc1→/build-with-celo/network-overview❌ - Line ~1803:
/important-information/baklava-testnet-disclaimer→/build-with-celo/network-overview❌ - Line ~1839:
/v/master/getting-started/baklava-testnet→/build-with-celo/network-overview❌
All of these should use /build-on-celo/network-overview instead.
Everything else looks good! 👍