Skip to content

feat: support phone number chat deeplinks from push notifications#1016

Merged
bmc08gt merged 2 commits into
code/cashfrom
feat/chat-phone-deeplink
Jun 29, 2026
Merged

feat: support phone number chat deeplinks from push notifications#1016
bmc08gt merged 2 commits into
code/cashfrom
feat/chat-phone-deeplink

Conversation

@bmc08gt

@bmc08gt bmc08gt commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Add NavigationTrigger.Chat.ByContact to handle push notifications that target a phone number instead of a chatId. AppRouter now distinguishes phone-number deeplinks (e164 with '+' prefix) from base64-encoded chatId deeplinks, avoiding a URLDecoder double-decode that mangled the '+' into a space. ChatViewModel resolves the contact via in-memory lookup, Room DB fallback, or the raw deeplink stub.

Add NavigationTrigger.Chat.ByContact to handle push notifications that
target a phone number instead of a chatId. AppRouter now distinguishes
phone-number deeplinks (e164 with '+' prefix) from base64-encoded chatId
deeplinks, avoiding a URLDecoder double-decode that mangled the '+' into
a space. ChatViewModel resolves the contact via in-memory lookup, Room
DB fallback, or the raw deeplink stub.
@bmc08gt bmc08gt self-assigned this Jun 29, 2026
@github-actions github-actions Bot added type: feature New functionality area: network gRPC, connectivity, API, exchange rates area: notifications Push notifications, in-app messaging area: deeplinks Deep link handling, URL routing, and link parsing and removed type: feature New functionality labels Jun 29, 2026
@github-actions github-actions Bot added the type: feature New functionality label Jun 29, 2026
@bmc08gt bmc08gt merged commit 76877df into code/cash Jun 29, 2026
2 of 3 checks passed
@bmc08gt bmc08gt deleted the feat/chat-phone-deeplink branch June 29, 2026 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: deeplinks Deep link handling, URL routing, and link parsing area: network gRPC, connectivity, API, exchange rates area: notifications Push notifications, in-app messaging type: feature New functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant