From 8464cb92195ad132416b89aa841ed318345d7899 Mon Sep 17 00:00:00 2001 From: null Date: Mon, 6 Jul 2026 20:48:49 -0500 Subject: [PATCH] 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 --- .../app/closer/notifications/PartnerNotificationManager.kt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/app/src/main/java/app/closer/notifications/PartnerNotificationManager.kt b/app/src/main/java/app/closer/notifications/PartnerNotificationManager.kt index b1989e39..34072352 100644 --- a/app/src/main/java/app/closer/notifications/PartnerNotificationManager.kt +++ b/app/src/main/java/app/closer/notifications/PartnerNotificationManager.kt @@ -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(