New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated Android SDK version #261
Updated Android SDK version #261
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥇 Thanks for the contribution!
Since version 12.5.0, Android 13 is now officially supported by Intercom, I think we should also make some small changes to the README.md file:
- In the optional permissions for Android section, add:
<uses-permission android:name="android.permission.POST_NOTIFICATIONS" />
- In the Push notifications setup section, add extra steps for Android setup:
- Steps for Android notifications setup: https://developers.intercom.com/installing-intercom/docs/android-fcm-push-notifications
- We could also add that for push notifications to work, user might have to grant the notification permission in iOS or Android 13+ with a plugin like permission_handler
Co-authored-by: Cristian Zazo <cristian.zazo@gmail.com>
Thanks, will apply the changes :) |
I updated the README.md and from what I can see right now the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, I didn't notice there was a new update for firebase_messaging
that automatically handled that for Android too. Changes LGTM, thanks for you help!
There was issue with previous version of Android SDK that it used wrong text colors for dark backgrounds. I got response from Intercom support team, that updating to 12.5.1 should solve that.