Skip to content

Mobile: fix /playlists/:id deep link routing#14092

Draft
dylanjeffers wants to merge 2 commits intomainfrom
cursor/fix-mobile-playlist-deeplink-530b
Draft

Mobile: fix /playlists/:id deep link routing#14092
dylanjeffers wants to merge 2 commits intomainfrom
cursor/fix-mobile-playlist-deeplink-530b

Conversation

@dylanjeffers
Copy link
Copy Markdown
Contributor

What’s going on

On mobile, the deep-link handler was treating opaque playlist ID URLs like /playlists/:id as profile links. It navigated to the Profile screen with id set to the playlist id, which leaves the profile page stuck showing its skeleton.

Fix

  • Route /playlists/:id to the Collection screen with { id }.
  • Extracted the deeplink path parsing into a small utility so it can be unit-tested.

Tests

  • Added a focused Jest test covering /users/:idProfile and /playlists/:idCollection.
    • Run with: npx jest -c packages/mobile/jest.deeplink.config.js
Open in Web Open in Cursor 

Co-authored-by: Dylan Jeffers <dylanjeffers@users.noreply.github.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 9, 2026

⚠️ No Changeset found

Latest commit: f6d401e

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

Co-authored-by: Dylan Jeffers <dylanjeffers@users.noreply.github.com>
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