Skip to content

feat(notifications): add channel routing, grouping, and tests#770

Merged
bmc08gt merged 2 commits into
code/cashfrom
feat/notification-channels-grouping
May 26, 2026
Merged

feat(notifications): add channel routing, grouping, and tests#770
bmc08gt merged 2 commits into
code/cashfrom
feat/notification-channels-grouping

Conversation

@bmc08gt
Copy link
Copy Markdown
Collaborator

@bmc08gt bmc08gt commented May 26, 2026

No description provided.

bmc08gt added 2 commits May 25, 2026 15:56
Add Hash message to common, enhance push Payload with substitutions,
categories, and group key support.

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>
Route push notifications to dedicated channels based on payload category
(Buys & Sells, Deposits & Withdrawals, Price Alerts, Chat, New Contacts).
Fix notification grouping by using unique IDs per notification with a
summary notification for the group, instead of reusing groupKey.hashCode()
which caused same-group notifications to replace each other. Add Robolectric
tests for channel mapping and grouping behavior, and a test script for
on-device verification.

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>
@bmc08gt bmc08gt self-assigned this May 26, 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 labels May 26, 2026
@bmc08gt bmc08gt merged commit 3abbbbb into code/cash May 26, 2026
3 checks passed
@bmc08gt bmc08gt deleted the feat/notification-channels-grouping branch May 26, 2026 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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