Skip to content

Handle notification clicks and actionId routing - #19

Merged
Joseph-Sameh-0 merged 1 commit into
developfrom
fix-Handle-notification-clicks-and-actionId-routing
Jul 8, 2026
Merged

Joseph-Sameh-0 merged 1 commit into
developfrom
fix-Handle-notification-clicks-and-actionId-routing

Conversation

@Joseph-Sameh-0

Copy link
Copy Markdown
Member

Add notification click handling and route changes for FinancialAction flows. Key changes:

  • Replace FinancialActionRoute.payload with actionId and update related screens, ViewModel, and CategoriesFeatureApiImpl.
  • App: navigate to FinancialActionRoute when push payload contains actionId; otherwise map notification "type" (via NotificationType.fromStringOrDefault) to appropriate feature routes (Achievements, Goals, Profile, ScheduledPayments, Statistics, Notifications).
  • Add NotificationClickState to buffer and emit notification click payloads and consume pending payloads on startup.
  • Register KMPNotifier listener in initKoin to forward clicks to NotificationClickState.
  • Add NotificationType.fromStringOrDefault helper.
  • Set Android MainActivity launchMode to singleTask.

Add notification click handling and route changes for FinancialAction flows. Key changes:

- Replace FinancialActionRoute.payload with actionId and update related screens, ViewModel, and CategoriesFeatureApiImpl.
- App: navigate to FinancialActionRoute when push payload contains actionId; otherwise map notification "type" (via NotificationType.fromStringOrDefault) to appropriate feature routes (Achievements, Goals, Profile, ScheduledPayments, Statistics, Notifications).
- Add NotificationClickState to buffer and emit notification click payloads and consume pending payloads on startup.
- Register KMPNotifier listener in initKoin to forward clicks to NotificationClickState.
- Add NotificationType.fromStringOrDefault helper.
- Set Android MainActivity launchMode to singleTask.
@Joseph-Sameh-0
Joseph-Sameh-0 merged commit bb600c7 into develop Jul 8, 2026
4 checks passed
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.

1 participant