fix: Exit early in permission request for iOS and Android in NotificationService

This commit is contained in:
Janez T
2025-10-30 17:05:24 +01:00
parent d77c60db2a
commit 73958c18a4
2 changed files with 13 additions and 4 deletions

View File

@@ -5,22 +5,22 @@
<testcase classname="fastlane.lanes" name="0: default_platform" time="0.000202"> <testcase classname="fastlane.lanes" name="0: default_platform" time="0.000193">
</testcase> </testcase>
<testcase classname="fastlane.lanes" name="1: increment_build_number" time="0.261599"> <testcase classname="fastlane.lanes" name="1: increment_build_number" time="6.996338">
</testcase> </testcase>
<testcase classname="fastlane.lanes" name="2: build_app" time="121.524995"> <testcase classname="fastlane.lanes" name="2: build_app" time="129.479733">
</testcase> </testcase>
<testcase classname="fastlane.lanes" name="3: upload_to_testflight" time="321.749799"> <testcase classname="fastlane.lanes" name="3: upload_to_testflight" time="333.229336">
</testcase> </testcase>

View File

@@ -101,6 +101,7 @@ class NotificationService {
debugPrint( debugPrint(
'📱 [NotificationService] iOS permissions granted: $_permissionGranted', '📱 [NotificationService] iOS permissions granted: $_permissionGranted',
); );
return; // Exit early if on iOS
} }
// Android 13+ permissions // Android 13+ permissions
@@ -114,7 +115,15 @@ class NotificationService {
debugPrint( debugPrint(
'🤖 [NotificationService] Android permissions granted: $_permissionGranted', '🤖 [NotificationService] Android permissions granted: $_permissionGranted',
); );
return; // Exit early if on Android
} }
// If neither platform plugin is available, assume permissions are granted
// This handles older Android versions that don't require runtime permissions
_permissionGranted = true;
debugPrint(
'✅ [NotificationService] No platform plugin found, assuming permissions granted',
);
} catch (e) { } catch (e) {
debugPrint('⚠️ [NotificationService] Error requesting permissions: $e'); debugPrint('⚠️ [NotificationService] Error requesting permissions: $e');
} }