[IRRIGOAPP-109] Replace in-app alert UI with Android system notifications #10

Merged
chris merged 2 commits from feature/IRRIGOAPP-109 into main 2026-08-23 21:45:17 -06:00
Owner

Ticket

IRRIGOAPP-109 Replace in-app alert UI with Android system notifications

Summary

  • Changed setNotificationHandler in NotificationsBridge to set all shouldShow* flags to true, so foreground pushes now surface as standard Android system banners instead of being suppressed
  • Removed the in-app PushBanner overlay, its state, and the addNotificationReceivedListener wiring that drove it
  • Kept addNotificationResponseReceivedListener for deep-link routing when the user taps a system notification
  • Deleted the PushBanner component and its tests (5 tests removed, component fully retired)
## Ticket [IRRIGOAPP-109](http://192.168.2.100:7123/home/browse/IRRIGOAPP-109/) Replace in-app alert UI with Android system notifications ## Summary - Changed `setNotificationHandler` in `NotificationsBridge` to set all `shouldShow*` flags to `true`, so foreground pushes now surface as standard Android system banners instead of being suppressed - Removed the in-app `PushBanner` overlay, its state, and the `addNotificationReceivedListener` wiring that drove it - Kept `addNotificationResponseReceivedListener` for deep-link routing when the user taps a system notification - Deleted the `PushBanner` component and its tests (5 tests removed, component fully retired)
chris merged commit f294f37a4f into main 2026-08-23 21:45:17 -06:00
chris deleted branch feature/IRRIGOAPP-109 2026-08-23 21:45:17 -06:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
chris/irrigo!10
No description provided.