feat: v1.9.1+1 - bug fixes

thanhtunguet · thanhtunguet · commit f0656d760c51 · 2025-07-07T16:24:47.000+07:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,14 @@
 # Changelog
 
+## 1.9.1+1
+
+### Fixed
+
+- Fix: Remove Sentry integration due to rendering issues on some Android devices, switch to using only FirebaseCrashlytics for error reporting
+- Fix: Update error handling bloc to log errors exclusively to FirebaseCrashlytics
+- Fix: Push notification BLoC improvements and documentation updates
+
+
 ## 1.9.0+1
 
 ### Added
diff --git a/lib/blocs/error_handling/error_handling_bloc.dart b/lib/blocs/error_handling/error_handling_bloc.dart
@@ -65,6 +65,10 @@ class ErrorHandlingBloc extends Cubit<void> {
   }
 
   void _captureException(dynamic error) {
-    FirebaseCrashlytics.instance.recordError(error, error.stackTrace);
+    if (!kIsWeb) {
+      if (SupaArchitecturePlatform.instance.useFirebase) {
+        FirebaseCrashlytics.instance.recordError(error, error.stackTrace);
+      }
+    }
   }
 }
diff --git a/lib/blocs/push_notification/push_notification_bloc.dart b/lib/blocs/push_notification/push_notification_bloc.dart
@@ -83,7 +83,8 @@ class PushNotificationBloc
 
   /// Set handler for foreground messages.
   void _setForegroundMessageHandler() {
-    _foregroundNotificationSubscription =
+    if (kIsWeb) return;
+    _foregroundNotificationSubscription ??=
         FirebaseMessaging.onMessage.listen(_handleNotification);
   }
 
diff --git a/pubspec.yaml b/pubspec.yaml
@@ -1,6 +1,6 @@
 name: supa_architecture
 description: Architecture library for Supa Flutter applications
-version: 1.9.0+1
+version: 1.9.1+1
 homepage: https://github.supa.vn
 
 environment:

Original file line number	Diff line number	Diff line change
`@@ -65,6 +65,10 @@ class ErrorHandlingBloc extends Cubit<void> {`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`void _captureException(dynamic error) {`
`68`		`- FirebaseCrashlytics.instance.recordError(error, error.stackTrace);`
	`68`	`+ if (!kIsWeb) {`
	`69`	`+ if (SupaArchitecturePlatform.instance.useFirebase) {`
	`70`	`+ FirebaseCrashlytics.instance.recordError(error, error.stackTrace);`
	`71`	`+ }`
	`72`	`+ }`
`69`	`73`	`}`
`70`	`74`	`}`
Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,8 @@ class PushNotificationBloc`
`83`	`83`
`84`	`84`	`/// Set handler for foreground messages.`
`85`	`85`	`void _setForegroundMessageHandler() {`
`86`		`- _foregroundNotificationSubscription =`
	`86`	`+ if (kIsWeb) return;`
	`87`	`+ _foregroundNotificationSubscription ??=`
`87`	`88`	`FirebaseMessaging.onMessage.listen(_handleNotification);`
`88`	`89`	`}`
`89`	`90`