Merge pull request #13 from strvcom/mu/fix/event_notifier

fix: Force EventNotifier to notify all updates

Author: michalurbanek

README.md

@@ -562,16 +562,7 @@ For the purpose of Fraud prevention, user safety, and compliance the dedicated A
 <!-- ##########            TODOs              ######### -->
 <!-- ################################################## -->
 # Todos
-- Investigate Github action caching
-- Fix crashlytics upload mapping symbols in Github actions
-- Implements secret handling 
-    - API keys
-    - Keystores for Android
-    - Service accounts
-    - Google service json
-    - iOS Xcode properties with .env files
 - Fix Riverpod async gaps - analytics manager (keep live)
-- Fix EventNotifier - change updateShouldNotify to return true (in case user returns back to the screen and emits the same value)
 - Revisit Google and Apple logins - Providers, Cancellation exception, separating Credentials from Sign In. USe Firebase directly for Apple on Android.
 - Refactor Sealed classes - private classes, use generated `when` function instead of switch.
 

lib/core/riverpod/event_notifier.dart

@@ -15,4 +15,11 @@ class EventNotifier<T> extends Notifier<T?> {
   T? build() {
     return state;
   }
+
+  /// Always returns true to ensure listeners are notified even when the same event is emitted multiple times.
+  /// Addresses navigation scenarios where the same event needs to be processed multiple times (e.g. when navigating back and forth).
+  @override
+  bool updateShouldNotify(T? previous, T? next) {
+    return true;
+  }
 }