Skip to content

Conversation

@GigaHierz
Copy link
Contributor

- remove ai tutorials
- add goldrush to indexers
- update exchanges page for clearer separation
- add 1inch wallet to wallet page
@GigaHierz GigaHierz requested review from a team as code owners January 22, 2026 19:15
@GigaHierz GigaHierz requested review from palango, seolaoh and viral-sangani and removed request for a team January 22, 2026 19:15
- fix broken links to network
Copy link
Collaborator

@viral-sangani viral-sangani left a 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! 👍

@GigaHierz GigaHierz merged commit f5e5493 into main Jan 27, 2026
5 checks passed
@GigaHierz GigaHierz deleted the minor-fixes branch January 27, 2026 09:04
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.

3 participants