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

Prepare APNS support #5

Merged
merged 32 commits into from
Jun 11, 2018
Merged

Prepare APNS support #5

merged 32 commits into from
Jun 11, 2018

Conversation

dbrgn
Copy link
Contributor

@dbrgn dbrgn commented Mar 5, 2018

Add new payload field type that may be either gcm or apns.

If no type is specified, it defaults to gcm (for backwards compatibility).

Work in progress.

Add new payload field `type` that may be either `gcm` or `apns`.

If no type is specified, it defaults to `gcm` (for backwards
compatibility).
Extract push related code into separate module.
This way, it's impossible to pass an ApnsToken to the GCM notification
function.
@dbrgn dbrgn force-pushed the apns branch 2 times, most recently from fec0bdc to 9907346 Compare April 25, 2018 08:26
@dbrgn
Copy link
Contributor Author

dbrgn commented Apr 25, 2018

Note: The cargo-audit warning is a false positive: rustsec/rustsec#17 and dtolnay/semver#172

@dbrgn dbrgn force-pushed the apns branch 2 times, most recently from 11c16c6 to 714c9e3 Compare April 25, 2018 15:00
@dbrgn dbrgn merged commit 2cfc201 into master Jun 11, 2018
@dbrgn dbrgn deleted the apns branch June 11, 2018 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants