Repository navigation
docs(contribute): fix broken links, anchors and redirect targets around the Contribute tab - #2400
Merged
Merged
Conversation
…nd the Contribute tab Review of #2394 found links into and out of the moved pages that open raw markdown, miss their headings or land on a page without the content the old URL had.
palango
force-pushed
the
palango/2394-contribute-link-fixes
branch
from
October 19, 2026 08:19
d93eab0 to
a20f0d4
Compare
palango
enabled auto-merge (squash)
October 19, 2026 08:20
GigaHierz
approved these changes
Oct 19, 2026
GigaHierz
left a comment
Contributor
There was a problem hiding this comment.
Approving. Checked against the branch head:
- The four smart-contracts anchors match existing headings (
When a new release branch is cut,Release management in Git/Github,Deploy the release candidate,Verify Proposed Release Candidate). /learn/protocol/staking/votinghas the Celo Signal signup, so the threecelo-signalredirects now land on the content they used to serve./build/tools/contracts/core-contractslists SortedOracles for both networks, which is what the oracle guide needs.- CI is green, including the broken-links check.
One nit, not blocking: the PR says the landing page is retitled to match the tab, but the tab is "Contribute" and the title is "Contribute to Celo". Fine either way; it reads better as a page title.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reviewing #2394 turned up links in and around the Contribute tab that were already broken before the rename. One opens the raw markdown export, four in-page anchors match no heading, one link is empty, and six old URLs redirect to the Contribute landing page even though the content they used to have lives elsewhere. This PR fixes those, makes the footer link relative, and retitles the landing page to match the tab. It builds on #2394 and targets that branch, so the diff shows only these fixes.
Refs #2261
Changes
Links in the moved pages:
contribute/contributors/code-contributors.mdx:53:./overview.mdserved the raw markdown export. It now links/contribute/contributors/overview.contribute/release-process/smart-contracts.mdx:303,342,390,407: the anchors now point at the headings that exist:#when-a-new-release-branch-is-cut,#release-management-in-git/github,#deploy-the-release-candidateand#verify-proposed-release-candidate.contribute/release-process/blockchain-client.mdx:28: the empty[docs.celo.org]()now links/learn/protocol.Redirects in
docs.json:celo-signalsources go to/learn/protocol/staking/voting, which has the Celo Signal signup. Their siblingcelo-foundation-voting-policyredirects already point there./general/ecosystem/buildersand/what-is-celo/joining-celo/buildersgo to/contribute/builders, and/general/ecosystem/contributorsgoes to/contribute/contributors/code-contributors.git log --followshows these old pages became those files./contributeinstead ofhttps://docs.celo.org/contribute/. Mintlify opens absolute footer links in a new tab, and from a preview build they lead to production.Link targets:
build/tools/oracles/run.mdx:11: "Celo core contract" now links the core contracts page, which lists the SortedOracles address on both networks, instead of the release process.build/tools/libraries-sdks/contractkit/index.mdx:25: removed the ContractKit link that led to the release policy page.operate/operators/faq.mdx:55: the Accordion title is plain text. Mintlify showed the markdown link there as text, andmint broken-linksdoes not check attributes. The accordion before it already links the same page in its body.Landing page:
contribute/index.mdxis titled "Contribute to Celo" with sidebar label "Overview", in place of "Joining Celo".Verification
On the branch head, with
mint dev, the four new anchors match heading ids on the rendered page, and four of the six changed redirects (/validator/celo-signaland the three builders/contributors sources) return 307 to their new destinations. I did not curl the other two celo-signal sources; they have the same destination.Once #2394 merges, I'll rebase onto main so the diff stays clean.