Skip to content

Remoce deviceToken checking for android on saving installation #305

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

Closed

Conversation

wangmengyan95
Copy link
Contributor

No description provided.

@drew-gross
Copy link
Contributor

Why do we want to allow android devices to have a device token? Parse.com doesn't allow that.

@wangmengyan95
Copy link
Contributor Author

@drew-gross, it is used to store Android GCM registration id. What do you mean Parse.com doesn't allow that?

@drew-gross
Copy link
Contributor

If you go to dasboard.parse.com/apps/_/browser/_Installation and try to create an installation with device type android and with a deviceToken you will get an error message. For installations created by the parse SDK, those are stored in installationId, I think. For imported installations, I'm pretty sure they are stored in GCMSenderID. You may want to double check that though.

@wangmengyan95
Copy link
Contributor Author

I know we can not create an android installation with deviceToken in dashboard although I am not quite sure about the reason. But for android SDK, I am pretty sure we do not use GCMSenderID to store the registration token. Right now all the GCM registration token is stored as deviceToken. I remember InstallationId is used for PPNS, we do not use that for GCM push.
@bnham @fantasist could you guys provide some context to this issue and confirm the behaviour of our backend?

@wangmengyan95 wangmengyan95 assigned bnham and unassigned gfosco Feb 9, 2016
@wangmengyan95 wangmengyan95 force-pushed the wangmengyan.remove_android_deviceToken_checking branch from c06f046 to 66dfaf8 Compare February 9, 2016 06:20
@facebook-github-bot
Copy link

@wangmengyan95 updated the pull request.

@wangmengyan95 wangmengyan95 deleted the wangmengyan.remove_android_deviceToken_checking branch February 10, 2016 20:18
@gdeglin
Copy link
Contributor

gdeglin commented Feb 12, 2016

This is still an issue. Not sure why it was closed? Doesn't seem fixed in any other pull requests.

@drew-gross This is wrong. If you set the deviceToken, deviceType (android), and pushType (gcm) fields then is saves correctly. GCMSenderID is used for the server key used to send the notification (if you're not using Parse's own server key).

@gdeglin
Copy link
Contributor

gdeglin commented Mar 1, 2016

This was fixed and merged in #405

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.

6 participants