Skip to content

Add comprehensive routing for coins/fan clubs feature pages#14048

Merged
dylanjeffers merged 3 commits intomainfrom
claude/fix-coin-routing-Iu2Hi
Apr 3, 2026
Merged

Add comprehensive routing for coins/fan clubs feature pages#14048
dylanjeffers merged 3 commits intomainfrom
claude/fix-coin-routing-Iu2Hi

Conversation

@dylanjeffers
Copy link
Copy Markdown
Contributor

Summary

Expanded the SSR route configuration for the Fan Clubs Page to include all coin-related feature pages and their sub-routes.

Key Changes

  • Extended route array from 2 routes to 13 routes covering the complete coins feature flow
  • Added routes for:
    • Coin creation (/coins/create)
    • Coin sorting (/coins/sort)
    • Individual coin pages (/coins/@ticker)
    • Coin purchase flow (/coins/@ticker/buy)
    • Coin redemption flows (/coins/@ticker/redeem and /coins/@ticker/redeem/@code)
    • Exclusive tracks (/coins/@ticker/exclusive-tracks)
    • Coin editing (/coins/@ticker/edit)

Implementation Details

  • Reformatted the route array to multi-line for improved readability and maintainability
  • All routes remain under the same 'Fan Clubs Page' page context
  • This ensures proper server-side rendering for all coin-related pages and their dynamic segments

https://claude.ai/code/session_017qmCwviCkRf65xdCuoZFXq

The SSR coins route only matched /clubs and /coins exactly, so URLs like
/coins/0003HZ had no SSR route handler. This caused them to fall through
to the track route (/@handle/@slug) which treated "coins" as a user handle.

Add all coin sub-routes (/coins/@ticker, /coins/@ticker/buy, etc.) to the
SSR coins route so Vike correctly identifies these as coin pages.

https://claude.ai/code/session_017qmCwviCkRf65xdCuoZFXq
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 3, 2026

⚠️ No Changeset found

Latest commit: ee7d6bd

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

- Add CLUB_DETAIL_PAGE, CLUB_DETAIL_BUY_PAGE, CLUB_REDEEM_PAGE, etc.
  route constants alongside existing COIN_* routes
- Register /clubs/:ticker routes in both mobile and desktop WebPlayer sections
- Update SSR coins route to handle all /clubs/* sub-routes
- Fix FanClubDetailPageRoute and CoinExclusiveTracksLegacyRedirect to
  preserve /clubs vs /coins URL prefix when normalizing ticker case
- Both /coins/TICKER and /clubs/TICKER now render the same fan club
  detail page

https://claude.ai/code/session_017qmCwviCkRf65xdCuoZFXq
@pull-request-size pull-request-size bot added size/L and removed size/S labels Apr 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

🌐 Web preview ready

Preview URL: https://audius-web-preview-pr-14048.audius.workers.dev

Unique preview for this PR (deployed from this branch).
Workflow run

@dylanjeffers dylanjeffers merged commit d2d6408 into main Apr 3, 2026
5 checks passed
@dylanjeffers dylanjeffers deleted the claude/fix-coin-routing-Iu2Hi branch April 3, 2026 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants