Dev 3362 missing methods#206
Merged
Merged
Conversation
…rden delivery The documented `SDK().initialize()` now registers itself as the companion `instance` that FirebaseMessagingService delivers pushes to, so the documented initialization receives pushes without an UninitializedPropertyAccessException. Guard notificationReceived() before initialize() and wrap onMessageReceived in try/catch so a malformed or early push can never crash the host app. Adds robustness tests covering both paths. Demo app: display pushes with a standard BigPicture notification (matching the React Native demo), request POST_NOTIFICATIONS at runtime, surface the FCM token in a copyable header, and add a crash interceptor for the tester.
Context#getSystemService(Class) requires API 23, but the demo's minSdk is 19, which failed the lint NewApi check on CI. Use ContextCompat.getSystemService instead, which is safe on all supported API levels.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.