Skip to content

Add deep linking support for push notifications#2109

Merged
MusabShakeel576 merged 1 commit into
qafrom
claude/epic-mendel-R4oi5
Jun 1, 2026
Merged

Add deep linking support for push notifications#2109
MusabShakeel576 merged 1 commit into
qafrom
claude/epic-mendel-R4oi5

Conversation

@MusabShakeel576
Copy link
Copy Markdown
Contributor

Summary

Implemented dynamic deep linking for push notifications based on the notification type sent by the backend, replacing the hardcoded navigation to the home screen.

Key Changes

  • Added getNotificationRoute() helper function that maps notification types to in-app routes
    • card-transaction notifications now navigate to the card screen
    • Unknown types default to the home screen
  • Updated the notification response listener to extract the type from notification data and use it for routing
  • Improved code comments to document the deep linking behavior and fallback logic
  • Added proper TypeScript typing for the notification data object

Implementation Details

  • The getNotificationRoute() function uses a switch statement for extensibility, making it easy to add new notification types in the future
  • The notification data is safely typed as { type?: string } | undefined to handle cases where the backend may not include type information
  • The implementation maintains backward compatibility by defaulting to home navigation for unrecognized types

https://claude.ai/code/session_01Wxoh4LSUztzSkDBgb66n67

Route notification taps by the `type` the backend sets in the payload.
Card transaction notifications now open the card screen instead of
always landing on home, completing the existing deep-link TODO in the
tap handler.

https://claude.ai/code/session_01Wxoh4LSUztzSkDBgb66n67
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
solid-app Ignored Ignored Jun 1, 2026 7:46am
solid-app-staging Ignored Ignored Jun 1, 2026 7:46am

Request Review

@MusabShakeel576 MusabShakeel576 merged commit 0486a7a into qa Jun 1, 2026
4 of 5 checks passed
@MusabShakeel576 MusabShakeel576 deleted the claude/epic-mendel-R4oi5 branch June 1, 2026 07:55
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.

2 participants