-
Notifications
You must be signed in to change notification settings - Fork 43
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
[iOS] Update OneSignal framework #18
Comments
Not fully sure but you should check remote notifications settings in your plist file as that was another feature added to one signal SDK probably later than the original SDK used in the module |
I do have the:
On my tiapp.xml, should be enough I think |
Just out of curiosity are you able to replicate the error without liveview on? |
I'm not using liveview at all |
@williamrijksen any ideas on how we can make this work? I think #13 should get auto resolved along with this and we are missing out on sending pictures and videos with notifications on iOS 10+ |
The error means that the #ifdef USE_TI_NETWORKREGISTERFORPUSHNOTIFICATIONS
- (void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo
{
...
}
...
#endif |
@williamrijksen @chmiiller Any thoughts on @hansemannn input? Ill give it a try if you aren't, just that Im not strong on the iOS skills ... |
When trying to update module's One Signal framework with the last one on their github repo, the app crashes when I send a notification and app is opened, or attempt to open the app from notification.
The message is:
Not sure what they've changed about it =/
Do you guys have a solution for this by any chance?
Thanks
The text was updated successfully, but these errors were encountered: