-
Notifications
You must be signed in to change notification settings - Fork 52
apns-push-type header is required on iOS 13 and later #96
apns-push-type header is required on iOS 13 and later #96
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.
You need to add a new error type “InvalidPushType” to errors.go.
https://github.com/RobotsAndPencils/buford/blob/master/push/errors.go
https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/sending_notification_requests_to_apns#2947619
I added InvalidPushType error. |
@RobotsAndPencils @nathany If there is no problem, could you approve and merge this? |
@RobotsAndPencils @nathany |
Thanks. We should probably add something to the README in the headers section. |
with #96 to remove stutter (push.PushType)
There were a number of lint issues:
I've changed the API slightly. |
release tagged: 0.14.0 |
https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/sending_notification_requests_to_apns#2947610