Skip to content

docs(wallets): split the wallet docs into an end-user page and a developer integration page - #2350

Merged
GigaHierz merged 2 commits into
mainfrom
GigaHierz/2256-split-wallet-docs
Oct 2, 2026
Merged

GigaHierz merged 2 commits into
mainfrom
GigaHierz/2256-split-wallet-docs

Conversation

@GigaHierz

@GigaHierz GigaHierz commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Closes #2256

Each wallet page now says in its first paragraph who it is for, and the native-wallet list exists once. The wallet content moved between two pages; no paths move (#2258 / #2259 do that), but holder-facing redirects now point at the holder page.

What changed

Page Before After
home/wallets "Wallets", audience in an Info box Wallets for users: two-sentence intro for holders, plain one-line gas note with a link, sentence-case headings, Ledger and MetaMask guides as - Guides: bullets, MetaMask guides point at the add-network button page (mainnet) and the testnet guide instead of the developer setup page, closing ## Related
tooling/wallets/index "Celo Wallets": native list duplicated from the user page, then providers Wallet integration: two-sentence intro, CIP-64 note, neutral provider list with factual one-line descriptions (marketing copy and the typo removed), links to the in-repo Reown, Dynamic, Portal and JAW pages, a Ledger and MetaMask guides list (including the EIP-712 workaround), closing ## Related; description matches the page
build-on-celo/fee-abstraction/overview Related link "Celo Wallets" Two bullets, one page each

Dynamic is added to the provider list. sidebarTitle is removed where it equalled title.

Redirects

Seventeen holder-facing sources that pointed at /tooling/wallets now point at /home/wallets (no chains):

  • /celo-codebase/wallet, /celo-codebase/wallet/README, /celo-codebase/wallet/intro
  • /celo-codebase/wallet/how-the-wallet-works/{invitations,README,sending-and-requesting-payments,ultralight-node-sync,verification}
  • /blog/2022/05/19/3 Simple Steps to Get Started with Valora on Celo
  • /getting-started/using-the-mobile-wallet, /getting-started/using-the-wallet
  • /getting-started/wallets, /getting-started/wallets/index
  • /getting-started/wallets#celo-compatible-wallets, #celo-native-wallets, #celowalletapp
  • /wallet

Kept on /tooling/wallets as developer-facing: /developer/walletconnect, /developer-resources/walkthroughs/wallet-test.

Acceptance criteria

  • One end-user page, first paragraph says it is for people holding CELO
  • One developer page, neutral list
  • Native-wallet list exists once (home/wallets)
  • Titles state the audience
  • Holder-facing redirects point at the holder page
  • mint broken-links green

Verification

$ mint broken-links --check-redirects
success no broken links found
$ bash scripts/check-orphans.sh
No orphan pages found.

Both pages render under mint dev; the in-page anchors on the holder page resolve (#celo-native-wallets and #celo-compatible-wallets keep their slugs after the sentence-case change). No other page links to the removed /tooling/wallets anchors.

Overlap

#2349 also edits home/wallets.mdx (adds a Troubleshooting section and a Related section, and rewrites the description). Whichever merges second must reconcile the description and the ## Related section.

🤖 Generated with Claude Code

…loper integration page

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@palango palango left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Splitting this into a holder page and a developer page is the right move. Inline comments below. These points sit on lines the diff doesn't touch:

Redirects still send holders to the developer page. docs.json has several user-facing sources pointing at /tooling/wallets: /getting-started/wallets (2094), /getting-started/wallets/index (2098), /getting-started/using-the-mobile-wallet (2086), /getting-started/using-the-wallet (4218), /wallet (2906) and the old "3 Simple Steps to Get Started with Valora on Celo" blog URL (1490). On production they all return 308 to /tooling/wallets:

$ curl -sI https://docs.celo.org/getting-started/wallets | grep -i location
location: /tooling/wallets

After this PR that page lists SDKs (Privy, Openfort, ...) and no wallet a holder can install. The Valora blog URL lands on a page that never mentions Valora. The PR says no redirects are needed because no directories moved, but the user-facing content did move, so these sources should point at /home/wallets. The /celo-codebase/wallet/* sources (1110-1134, 4122) are also about the old Valora app and probably belong there too.

Holder page (home/wallets.mdx):

  • The CIP-64 <Note> at line 16 is copied word for word from the developer page ("drops the app's feeCurrency"). It's jargon for a holder, and two copies will drift apart. One plain sentence would do for holders, something like "some wallets charge gas in CELO even if you hold stablecoins", plus the link.
  • The headings are still in title case: "Choosing a Wallet" (9), "Celo Native Wallets" (21), "Hardware Wallets" (61), "Celo Compatible Wallets" (75). AGENTS.md §3 asks for sentence case on pages being rewritten. Changing them changes the #celo-native-wallets and #celo-compatible-wallets anchors the intro links to, so update those links in the same edit.
  • The page doesn't end with ## Related (AGENTS.md §8.7).

Developer page (tooling/wallets/index.mdx):

  • The list is called neutral, but the vendor blurbs are still marketing copy: "Privy makes it easy…" (21), "enterprise-grade wallets" (40), "effortless, intuitive, and secure" (59), "enterprise-grade security and flexibility" (68), "feature-rich … with ease" (98). There's also a typo, "multisg" (88). AGENTS.md says no promotional framing.
  • The page ends on an external vendor link with no ## Related section, so an agent reaching the bottom has nowhere to go.

Comment thread home/wallets.mdx Outdated
MetaMask is a self-custody wallet with support for Celo.

You can learn more about connecting MetaMask to the Celo network [here](/tooling/wallets/metamask/use).
Guides: [configure MetaMask for Celo](/tooling/wallets/metamask/setup) and [use MetaMask with Celo](/tooling/wallets/metamask/use).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/tooling/wallets/metamask/setup is a developer page. Its title is "Programmatic Setup", its description is "How app developers can use MetaMask…", and the body is a wallet_addEthereumChain JavaScript snippet a holder can't run. For holders, the add-network button on /build-on-celo/network-overview (mainnet) or /tooling/wallets/metamask/add-celo-testnet-to-metamask fits better.

Nit: Ledger puts its guides in a - Guides: bullet (line 71), but here they're a loose paragraph above the list. Make it a bullet after - [Homepage] so the two match.

Comment thread tooling/wallets/index.mdx Outdated
## Wallet providers

Server wallets are a crucial part of building accessible applications. In the future of applications, you don't want to ask suers to login with a wallet but rather using their social logins like email account etc.
These providers supply embedded or server wallets, so users can sign in with an email or social login instead of bringing a wallet of their own. The list is neutral and not a recommendation.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This doesn't hold for several of the entries below:

  • Safe Wallet (86) is a multisig smart account with no email or social login.
  • Zerion (96) links to a data API reference.
  • Reown/WalletConnect (57) mostly connects wallets users already have, the opposite of "instead of bringing a wallet of their own".
  • JAW (116) uses passkeys.

A reader (or an agent) will take these as email-login SDKs. Either describe the list more loosely ("providers for adding wallets to an app") or give each entry a short line on what it actually does.

Comment thread tooling/wallets/index.mdx Outdated
Celo is designed to work seamlessly with a range of wallets, each offering features to meet different user needs.

The [Celo Native Wallets](#celo-native-wallets) section provides an overview of wallets that are optimized for the Celo network. These wallets allow users to fully benefit from Celo’s native functionalities, such as [phone number mapping](/build-on-celo/build-on-socialconnect) and [fee abstraction](/build-on-celo/fee-abstraction/overview).
This page is for app developers adding wallets to an app on Celo. If you hold CELO and want to choose a wallet to use, see [Wallets for users](/home/wallets). To list your Mini App inside MiniPay, see [Build for MiniPay](/build-on-celo/build-on-minipay/overview).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This page is the index of the Tooling > Wallets nav group, but after this PR it links none of the pages under it. Before, the Ledger entry linked setup, to-celo-terminal, to-celo-web, to-celo-cli and eip712-workaround. That last one matters to developers integrating Safe over WalletConnect with a Ledger. The MetaMask guides are unlinked from here too.

The in-repo integration guides for providers on this page are also unlinked: /tooling/libraries-sdks/reown, /portal and /jaw exist in nav, but Reown, Portal and JAW link only to vendor sites. Dynamic (/tooling/libraries-sdks/dynamic) is a wallet SDK and isn't on the page at all.

Also, AGENTS.md §3 asks for one or two sentences before the first ##. This paragraph has three, and the same goes for home/wallets.mdx:7.

Comment thread tooling/wallets/index.mdx
---

## Wallet Infrastructure
## Wallet providers

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Renaming ## Wallet Infrastructure and deleting the native, hardware, MiniPay, Valora and Ledger sections removes anchors such as #wallet-infrastructure and #celo-native-wallets from /tooling/wallets. mint broken-links doesn't check anchors (AGENTS.md §6), so CI won't catch this. Worth a quick search for inbound deep links.

Comment thread tooling/wallets/index.mdx Outdated
description: Overview of digital wallets available to send, spend, and earn Celo assets.
sidebarTitle: "Overview"
title: Wallet integration
description: Wallet providers an app developer can integrate on Celo, and what to know about CIP-64 fee-currency support before relying on gas paid in stablecoins

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The description promises "what to know about CIP-64 fee-currency support", but the page only links out to it. The holder page's description says wallets are "compared", and that page only lists them. AGENTS.md §8.1 treats the description as what an agent reads before deciding to open the page, so it should match the content.

Also, sidebarTitle is the same as title on both pages, so it can go. Mintlify falls back to title, and the frontmatter rule is to set it only when the nav label needs to be shorter. Here the value is also quoted, though it has no colon.

Comment thread home/wallets.mdx
- [Homepage](https://www.ledger.com/)
- Platforms: Ledger Live (desktop, mobile) with a Ledger device
- [Set up a Ledger with Celo](/tooling/wallets/ledger/setup)
- Guides: [set up a Ledger with Celo](/tooling/wallets/ledger/setup), connect it to [Celo Terminal](/tooling/wallets/ledger/to-celo-terminal), the [Celo web wallet](/tooling/wallets/ledger/to-celo-web) or the [Celo CLI](/tooling/wallets/ledger/to-celo-cli), and the [EIP-712 signing workaround](/tooling/wallets/ledger/eip712-workaround)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure about this one: celowallet.app now has the page title "Othello | Use Celo on the web or on your desktop", but the to-celo-web guide's screenshots show "Celo Web Wallet". If it's the same product under a new name, the link text and guide need updating. If it's a different product, the guide may be stale. Moving this link onto the holder page makes that more visible.

- [Using Fee Abstraction](/build-on-celo/fee-abstraction/using-fee-abstraction) — How to pay gas with alternate fee currencies in your transactions
- [Adding Fee Currencies](/build-on-celo/fee-abstraction/add-fee-currency) — How to implement and register a new fee currency
- [Celo Wallets](/tooling/wallets) — The wallets Celo documents, including the ones in the table above
- [Wallets for users](/home/wallets) and [Wallet integration](/tooling/wallets) — The wallets Celo documents, including the ones in the table above, and the providers an app can integrate

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nit: every other entry under ## Related is one page with a short half-sentence on why. This one has two pages and a full clause, so the reader has to work out which part goes with which link. Split it into two bullets.

@palango palango left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One follow-up to my review from yesterday, since nothing has been pushed yet.

I compared the deleted blocks in tooling/wallets/index.mdx against home/wallets.mdx, and nothing was lost. MiniPay, Valora, Celo Terminal and the Ledger guides all survive on the holder page, and the "Build for MiniPay" link moved into the intro. Nothing in the content links to the removed /tooling/wallets anchors either, so you can ignore my comment about deep links.

The redirects are the one thing I'd block on. Nineteen redirect entries still point at /tooling/wallets, including /getting-started/wallets, /wallet and the Valora blog URL. After this PR they all land holders on the SDK list. Please re-point the holder-facing ones at /home/wallets.

On #2349: the two PRs conflict only in the home/wallets.mdx frontmatter. #2349 adds the ## Related section this page is missing. It also links holders to /tooling/wallets/metamask/setup, the same developer page I flagged here, so it's worth fixing that once in whichever PR merges second.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@GigaHierz

Copy link
Copy Markdown
Contributor Author

Review points addressed in 292a292.

Redirects: 17 of the 19 entries pointing at /tooling/wallets now point at /home/wallets (full list in the PR body), with no chains. /developer/walletconnect and /developer-resources/walkthroughs/wallet-test stay on the developer page.

Holder page: one plain sentence replaces the CIP-64 note; headings in sentence case (slugs unchanged, so the intro anchors still resolve); two-sentence intro; closing ## Related; MetaMask links go to the network overview and the testnet page; Ledger guides in a - Guides: bullet; sidebarTitle removed; description matches the page.

Developer page: two-sentence intro and a matching description; vendor blurbs cut to facts I could support (marketing claims dropped), typo fixed; Dynamic added; links to the in-repo Reown, Dynamic, Portal and JAW pages; a Ledger and MetaMask guides list including the EIP-712 workaround; closing ## Related; sidebarTitle removed. The fee-abstraction ## Related entry is split into two bullets.

Not changed: the celowallet.app link. The site now titles itself "Othello" but nothing says it replaces the Celo Web Wallet, so I could not confirm a rename. Nothing linked the removed /tooling/wallets anchors; the only anchor hits were redirects, which now go to /home/wallets.

mint broken-links --check-redirects and the orphan check pass; both pages render with no console errors.

@GigaHierz
GigaHierz requested a review from palango October 2, 2026 09:48

@palango palango left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, this addresses everything I asked for. All 17 holder-facing redirects now go to /home/wallets with no chains, and the two left on /tooling/wallets are developer URLs. mint broken-links --check-redirects and the orphan check pass on 292a292. The CIP-64 note, headings, Related sections, vendor blurbs, dev page intro and index links all read well now, and adding Dynamic and the in-repo Reown, Portal and JAW pages fills the gap I mentioned.

One optional tweak: /developer/walletconnect could point at /tooling/libraries-sdks/reown rather than the wallet index. The old Valora WalletConnect blog URLs already go there.

On merging: #2349 conflicts with this in home/wallets.mdx (frontmatter, intro and the end of the page). Whichever lands second should keep this PR's title and intro, keep #2349's AddNetworkButton import and Troubleshooting section, and end with one merged Related list. #2351, #2352 and #2353 merge cleanly on top of this.

Comment thread home/wallets.mdx
Ledger is a hardware wallet. Ledger Live supports CELO and Celo stablecoins, and its send flow can pay gas in a Celo fee currency of the user's choice. App transactions signed with a Ledger through MetaMask are not CIP-64, because MetaMask drops `feeCurrency`; see [wallet support for CIP-64](/build-on-celo/fee-abstraction/overview#wallet-support-for-cip-64) for the supported versions.

- [Homepage](https://www.ledger.com/)
- Guides: [set up a Ledger with Celo](/tooling/wallets/ledger/setup), connect it to [Celo Terminal](/tooling/wallets/ledger/to-celo-terminal), the [Celo web wallet](/tooling/wallets/ledger/to-celo-web) or the [Celo CLI](/tooling/wallets/ledger/to-celo-cli), and the [EIP-712 signing workaround](/tooling/wallets/ledger/eip712-workaround)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

On the Othello question from my last review: it is a rename. github.com/celo-tools/celo-web-wallet (homepage celowallet.app) now describes itself as "Othello: A lightweight, self-sovereign wallet for the Celo network. Runs on the web and on desktop." Not for this PR, but the link text here and on the dev page, plus the to-celo-web guide and its screenshots, should move to Othello in a follow-up.

@GigaHierz
GigaHierz merged commit 22bbed6 into main Oct 2, 2026
5 checks passed
@GigaHierz
GigaHierz deleted the GigaHierz/2256-split-wallet-docs branch October 2, 2026 10:58
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.

story: Split wallet docs into an end-user page and a developer wallet-integration page

2 participants