Remove the show once logic

wordpress-mobile · Mar 13, 2024 · 5ad2182 · 5ad2182
1 parent feafe79
commit 5ad2182
Showing 1 changed file with 2 additions and 9 deletions.
diff --git a/...s/src/main/java/org/wordpress/android/ui/notifications/NotificationsDetailListFragment.kt b/...s/src/main/java/org/wordpress/android/ui/notifications/NotificationsDetailListFragment.kt
@@ -133,10 +133,8 @@ class NotificationsDetailListFragment : ListFragment(), NotificationFragment {
             showErrorToastAndFinish()
         }
 
-        val confetti: LottieAnimationView = requireActivity().findViewById(R.id.confetti)
-        if (note?.isViewMilestoneType == true && !confettiShown) {
-            confetti.playAnimation()
-            confettiShown = true
+        if (note?.isViewMilestoneType == true) {
+            view?.findViewById<LottieAnimationView>(R.id.confetti)?.playAnimation()
         }
     }
 
@@ -161,9 +159,6 @@ class NotificationsDetailListFragment : ListFragment(), NotificationFragment {
             showErrorToastAndFinish()
             return
         }
-        if (noteId != note.id) {
-            confettiShown = false
-        }
         notification = note
     }
 
@@ -625,8 +620,6 @@ class NotificationsDetailListFragment : ListFragment(), NotificationFragment {
         private const val KEY_NOTE_ID = "noteId"
         private const val KEY_LIST_POSITION = "listPosition"
 
-        private var confettiShown = false
-
         @JvmStatic
         fun newInstance(noteId: String?): NotificationsDetailListFragment {
             val fragment = NotificationsDetailListFragment()