Skip to content

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

Merged
dylanjeffers merged 5 commits intomainfrom
cursor/fix-mobile-playlist-deeplink-530b
Apr 10, 2026
Merged

Mobile: fix /playlists/:id deep link routing#14092
dylanjeffers merged 5 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: b8ac815

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

cursoragent and others added 4 commits April 9, 2026 23:22
Co-authored-by: Dylan Jeffers <dylanjeffers@users.noreply.github.com>
Co-authored-by: Dylan Jeffers <dylanjeffers@users.noreply.github.com>
@dylanjeffers dylanjeffers marked this pull request as ready for review April 10, 2026 17:40
@dylanjeffers dylanjeffers merged commit 45ce680 into main Apr 10, 2026
3 checks passed
@dylanjeffers dylanjeffers deleted the cursor/fix-mobile-playlist-deeplink-530b branch April 10, 2026 17:40
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