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
Drop support for Unity 4, to support push notifications. #42
Conversation
Push notifications on Unity 5 were broken due to IL2CPP not supporting PropertyInfo being exported by any type. Fixes #36.
@@ -1023,36 +1023,15 @@ private class SettingsWrapper : IDictionary<string, object> { | |||
/// </summary> | |||
/// <param name="action">Action to be completed when device token is received.</param> | |||
internal static void RegisterDeviceTokenRequest(Action<byte[]> action) { | |||
// Basically what we want to achieve here is: |
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.
I'm glad I left this comment
This looks good to me. Let's punt this to |
Hmm. Seeing as these methods are currently broken in the 1.6 release, do we really want to keep them broken (for both unity 4 and 5) in the next release? |
You're correct. Let's land this then |
Merge this maybe? |
My bad, PR wasn't assigned to me so I didn't see it :) |
Drop support for Unity 4, to support push notifications.
Push notifications on Unity 5 were broken due to IL2CPP not supporting PropertyInfo being exported by any type.
Fixes #36.