OneSignal Integration #344
Conversation
| @@ -22,6 +23,15 @@ void main() async { | |||
| await SharedPreferencesUtil.init(); | |||
| await MixpanelManager.init(); | |||
| await ObjectBoxUtil.init(); | |||
| if (Env.oneSignalAppId != null) { | |||
There was a problem hiding this comment.
Let's login independently of permissions, and let's not ask for permissions when the app opens, currently notification permissions are being asked at home/page.dart
There was a problem hiding this comment.
I just now actually made this change and after pushing it, realised that the PR has been merged
There was a problem hiding this comment.
Opened a new PR for this change #347
|
App runs seamlessly, but I have not been able to test because of the onesignal account, merging for now as it is, looking forward for the remaining changes :) |
I'm adding your email as an admin in my personal OneSignal account. And also should I send the p8 file through mail or you'll be generating a new on through Apple Developer Console? |
This PR implements the changes suggest by @josancamon19 in the #344 PR (which has been merged, hence a separate PR). It removes the permission request call at the very beginning of the application and also it removes the if condition to check whether the user has given permission for notifications or not.
This PR integrates OneSignal for both Android and iOS.
Fixes #331
Note: For iOS, the OneSignal Notification Service Extension is only set up for the bundle ID
com.friend-app-with-wearable.ios12. This means it won't work with the bundle IDcom.friend-app-with-wearable.ios12.devbecause this bundle ID isn't included in the Development Team.Steps to integrate OneSignal on iOS with xcode 15 or higher:
OneSignal/OneSignal-Flutter-SDK#551 (comment)