Skip to content

[docs-agent] Add Swap tokens section to wallets nav to fix live 404s - #1650

Merged
dslovinsky merged 1 commit into
mainfrom
docs/remove-token-swaps-links
Sep 24, 2026
Merged

dslovinsky merged 1 commit into
mainfrom
docs/remove-token-swaps-links

Conversation

@alchemy-bot

@alchemy-bot alchemy-bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Follow-up on the Slack thread: the same-chain Swap API is NOT deprecated (@jakehobbs confirmed it's been live for all teams since Oct 2025, ~16 teams using it in the past week). The actual bug @kennethngo-alch reported is that /docs/wallets/transactions/swap-tokens and /docs/wallets/transactions/cross-chain-swap-tokens both return 404 on the live site.

Root cause: the two MDX files exist on disk at content/wallets/pages/transactions/swap-tokens/index.mdx and content/wallets/pages/transactions/cross-chain-swap-tokens/index.mdx, but neither is registered in content/docs.yml, so Fern doesn't route to them even though inbound links (wallets index card, transactions overview table, React quickstart bullet, cross-chain guide inline reference) all point at those URLs.

Verified against live:

GET /docs/wallets/transactions/retry-transactions         → HTTP 200  (in docs.yml)
GET /docs/wallets/transactions/send-parallel-transactions → HTTP 200  (in docs.yml)
GET /docs/wallets/transactions/pay-gas-with-any-token     → HTTP 200  (in docs.yml)
GET /docs/wallets/transactions/swap-tokens                → HTTP 404  (NOT in docs.yml)
GET /docs/wallets/transactions/cross-chain-swap-tokens    → HTTP 404  (NOT in docs.yml)

Fix

Added a Swap tokens subsection under Wallets > Transactions in content/docs.yml, placed right after Send transactions, with two child pages (Same-chain swaps, Cross-chain swaps) mapped to the existing MDX files. No content edits — just nav registration.

Diff

+          - section: Swap tokens
+            contents:
+              - page: Same-chain swaps
+                path: wallets/pages/transactions/swap-tokens/index.mdx
+              - page: Cross-chain swaps
+                path: wallets/pages/transactions/cross-chain-swap-tokens/index.mdx

History

Original PR intent (removing all inbound links) was based on Ava's initial "yes deprecated" note. That was reversed later in the thread once @jakehobbs shared usage numbers, so the removal commit has been force-pushed away and this PR now instead unblocks the 404s.

Requested by

@dslovinsky (via Slack thread)

@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

🔗 Preview Mode

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview Sep 23, 2026, 10:17 PM

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b0e79ce26f

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread content/wallets/pages/index.mdx
The same-chain `swap-tokens` and `cross-chain-swap-tokens` pages under `content/wallets/pages/transactions/` are not listed in `content/docs.yml`, so both live URLs return 404 even though the underlying API is still supported (Alpha, live for all teams since Oct 2025 per @jakehobbs). Add a `Swap tokens` subsection under Wallets > Transactions with Same-chain and Cross-chain child pages so both routes render again.

Requested-by: @dslovinsky
@alchemy-bot
alchemy-bot force-pushed the docs/remove-token-swaps-links branch from b0e79ce to ff47b3e Compare September 23, 2026 22:16
@alchemy-bot
alchemy-bot requested a review from a team as a code owner September 23, 2026 22:16
@alchemy-bot alchemy-bot changed the title [docs-agent] Remove links to the deprecated same-chain token swaps page [docs-agent] Add Swap tokens section to wallets nav to fix live 404s Sep 23, 2026
dslovinsky
dslovinsky previously approved these changes Sep 23, 2026
@github-actions
github-actions Bot dismissed dslovinsky’s stale review September 23, 2026 22:29

@dslovinsky you are listed as the originator of this docs request (via the Requested-by trailer on a docs-agent commit). Per the docs-agent self-review policy, the originator can't approve their own request. Please ask another team member to review.

@dslovinsky
dslovinsky merged commit c0c88bd into main Sep 24, 2026
15 of 18 checks passed
@dslovinsky
dslovinsky deleted the docs/remove-token-swaps-links branch September 24, 2026 14:57
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