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.

Sign up for GitHub

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

Jump to bottom

Cache events #133

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Cache events #133

wants to merge 3 commits into from

Conversation

adammeaney
Copy link

Please take a moment to fill out the following:

Makes notification action consistent between platforms.

Changes Proposed in this pull request:

  • Allow delayed subscription for the opened event for iOS to follow how Android does it.
  • Follows a similar pattern for the OnTokenRefresh. This can be removed if you see a problem with it that I do not, but it allows for auto registering at the beginning and only using it after a sign in or similar approaches.

@rdelrosario rdelrosario added this to the 2.3.1 milestone May 31, 2018
@thomasgalliker
Copy link

You can find an all new implementation with caching of push notifications here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
3.5.0
Development

Successfully merging this pull request may close these issues.
3 participants
@adammeaney @thomasgalliker @rdelrosario