null
/
Closer
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

feat(analytics): record specific notification type on push-opened 

Foreground notification taps carried only app_route, so their
push_notification_opened event logged the generic "app_route" category. Also
put the notification type on the intent so the event records the specific kind
(partner_answered, reveal_ready, ...). Backgrounded taps already carry type via
the FCM data payload. Completes push-opened coverage across both tap paths.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
null 2026-07-06 20:48:49 -05:00
parent 368d8c455f
commit 8464cb9219
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/src/main/java/app/closer/notifications/PartnerNotificationManager.kt
View File

@ -126,6 +126,10 @@ class PartnerNotificationManager @Inject constructor(
// dual path is what kept re-breaking notification routing — one mechanism for every route.
val intent = Intent(context, MainActivity::class.java).apply {
putExtra("app_route", route)
// Carry the notification type so the push_notification_opened analytics event records the
// specific kind (partner_answered, reveal_ready, …) instead of the generic "app_route".
// Enum name lowercased matches the remote-type strings in fromRemoteType.
putExtra("type", type.name.lowercase())
flags = Intent.FLAG_ACTIVITY_CLEAR_TOP or Intent.FLAG_ACTIVITY_SINGLE_TOP
}
val pendingIntent = PendingIntent.getActivity(