Skip to content
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

Adopt APNs changes with iOS 13 #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

christophhagen
Copy link
Contributor

Adds the apns-push-type option to NotificationPusher, as required for watchOS 6 and recommended for all other platforms.

The initialization of NotificationPusher receives an additional, optional argument of type APNSPushType, which is an enum specifying the type of the notification.

Specifying the type adds the apns-push-type key with the appropriate value to the header of the notification request.

The APNSPushType enum is commented with additional information about the different types, as detailed in Sending Notification Requests to APNs.

Adding the option resolves issues with push notifications being dropped when no alert is included, specifically when sending content-available push updates.

Adds the `apns-push-type` option to `NotificationPusher`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant