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

Push: ensure all LocalDevice details are deleted on deregistration #710

Merged
merged 1 commit into from
Sep 18, 2019

Conversation

paddybyers
Copy link
Member

Also: check deviceToken validity on each launch; handle failures to obtain a deviceToken.

Partially fixes: https://github.com/ably/realtime/issues/2544

Copy link
Member

@mattheworiordan mattheworiordan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tomczoink
Copy link
Contributor

@paddybyers do you want to go with the version of RSH2d in https://github.com/ably/docs/pull/694/files or here?

@mattheworiordan mattheworiordan temporarily deployed to ably-docs-staging-pr-710 August 8, 2019 10:16 Inactive
@tomczoink tomczoink force-pushed the push-spec-deregistration-update branch from 7dca2b4 to 73a555b Compare September 18, 2019 16:25
@mattheworiordan mattheworiordan requested a deployment to ably-docs-staging-pr-710 September 18, 2019 16:25 Abandoned
…heck deviceToken validity on each launch; handle failures to obtain a deviceToken
@tomczoink tomczoink force-pushed the push-spec-deregistration-update branch from 73a555b to e470940 Compare September 18, 2019 16:26
@mattheworiordan mattheworiordan requested a deployment to ably-docs-staging-pr-710 September 18, 2019 16:26 Abandoned
@tomczoink
Copy link
Contributor

Confirmed with @paddybyers this is the version we'd prefer for RSH2d.

@tomczoink tomczoink merged commit 1d5c56d into master Sep 18, 2019
@tomczoink tomczoink deleted the push-spec-deregistration-update branch September 18, 2019 16:27
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.

3 participants