[FEAT/#384] 이벤트 중복되지 않게 구현

Author: leeeyubin

core/firebase/src/main/java/com/terning/core/firebase/messageservice/TerningMessagingService.kt

@@ -37,7 +37,7 @@ class TerningMessagingService : FirebaseMessagingService() {
     lateinit var navigatorProvider: NavigatorProvider
 
     @Inject
-    lateinit var tracker: AmplitudeTracker
+    lateinit var amplitudeTracker: AmplitudeTracker
 
     override fun onNewToken(token: String) {
         super.onNewToken(token)
@@ -48,23 +48,27 @@ class TerningMessagingService : FirebaseMessagingService() {
     override fun handleIntent(intent: Intent?) {
         super.handleIntent(intent)
 
-        extractInformation(
-            title = intent?.getStringExtra(TITLE),
-            body = intent?.getStringExtra(BODY),
-            type = intent?.getStringExtra(TYPE),
-            imageUrl = intent?.getStringExtra(IMAGE_URL)
-        )
+        if (!isAppInForeground()) {
+            extractInformation(
+                title = intent?.getStringExtra(TITLE),
+                body = intent?.getStringExtra(BODY),
+                type = intent?.getStringExtra(TYPE),
+                imageUrl = intent?.getStringExtra(IMAGE_URL)
+            )
+        }
     }
 
     override fun onMessageReceived(message: RemoteMessage) {
         super.onMessageReceived(message)
 
-        extractInformation(
-            title = message.data[TITLE],
-            body = message.data[BODY],
-            type = message.data[TYPE],
-            imageUrl = message.data[IMAGE_URL]
-        )
+        if (isAppInForeground()) {
+            extractInformation(
+                title = message.data[TITLE],
+                body = message.data[BODY],
+                type = message.data[TYPE],
+                imageUrl = message.data[IMAGE_URL]
+            )
+        }
     }
 
     private fun extractInformation(
@@ -75,7 +79,7 @@ class TerningMessagingService : FirebaseMessagingService() {
     ) {
         if (title.isNullOrEmpty() || !userRepository.getAlarmAvailable()) return
 
-        tracker.track(
+        amplitudeTracker.track(
             type = EventType.PUSH_NOTIFICATION,
             name = "received"
         )
@@ -101,6 +105,7 @@ class TerningMessagingService : FirebaseMessagingService() {
             action = Intent.ACTION_VIEW
             data = deeplink.toUri()
             flags = Intent.FLAG_ACTIVITY_NEW_TASK or Intent.FLAG_ACTIVITY_CLEAR_TOP
+            putExtra(FROM_NOTIFICATION, true)
         }
         val pendingIntent = PendingIntent.getActivity(
             this@TerningMessagingService,
@@ -169,5 +174,6 @@ class TerningMessagingService : FirebaseMessagingService() {
         private const val BODY: String = "body"
         private const val TYPE: String = "type"
         private const val IMAGE_URL: String = "imageUrl"
+        const val FROM_NOTIFICATION: String = "fromNotification"
     }
 }

feature/main/src/main/java/com/terning/feature/main/MainActivity.kt

@@ -14,6 +14,7 @@ import com.terning.core.analytics.EventType
 import com.terning.core.analytics.LocalTracker
 import com.terning.core.designsystem.theme.TerningPointTheme
 import com.terning.core.designsystem.util.DeeplinkDefaults.REDIRECT
+import com.terning.core.firebase.messageservice.TerningMessagingService.Companion.FROM_NOTIFICATION
 import dagger.hilt.android.AndroidEntryPoint
 import javax.inject.Inject
 
@@ -45,20 +46,30 @@ class MainActivity : ComponentActivity() {
 
     private fun handleDeeplink(intent: Intent?): Pair<String?, String?> {
         val uri = intent?.data
-        val host = uri?.host
-        val redirect = uri?.getQueryParameter(REDIRECT)
+        val uriString = uri?.toString()
 
-        if (uri != null) {
+        if (uriString.isNullOrEmpty()) return null to null
+
+        val host = uri.host
+        val redirect = uri.getQueryParameter(REDIRECT)
+
+        if (!intent.getBooleanExtra(ALREADY_TRACKED, false)
+            && intent.getBooleanExtra(FROM_NOTIFICATION, false)
+        ) {
             tracker.track(
                 type = EventType.PUSH_NOTIFICATION,
                 name = "opened"
             )
         }
 
+        intent.putExtra(ALREADY_TRACKED, true)
+
         return host to redirect
     }
 
     companion object {
+        private const val ALREADY_TRACKED: String = "alreadyTracked"
+
         fun getIntent(
             context: Context,
         ) = Intent(context, MainActivity::class.java)