Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 0 additions & 9 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -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.

Expand Down
7 changes: 7 additions & 0 deletions lib/core/riverpod/event_notifier.dart
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
}