Skip to content

Conversation

@mhisham
Copy link
Contributor

@mhisham mhisham commented Apr 11, 2017

I faced a problem when sending GCM Message and want to update my database to remove invalid Tokens.

I receive a response like this:
{ messageId: undefined,
regId: undefined,
error: Error: NotRegistered ..
}

so i replaced undefined token with regToken by order of tokens

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 305d955 on mhisham:master into d5db2a0 on appfeel:master.

@appfeel
Copy link
Owner

appfeel commented Apr 12, 2017

Hey! That's magic, thanks! Are you sure that the results order corresponds to the sent order? I'm not really sure about it as ... Could you confirm before merging?

@mhisham
Copy link
Contributor Author

mhisham commented Apr 12, 2017

Yes I'm sure and if I face anything else I'll make pull request to you 👍

@appfeel appfeel merged commit 148a2a3 into appfeel:master Apr 12, 2017
@mhisham
Copy link
Contributor Author

mhisham commented Apr 13, 2017

Could you please update npm version as patch version 1.0.18 to make me able to update it on my main project.

Thank you

mhisham added a commit to mhisham/node-pushnotifications that referenced this pull request Apr 17, 2017
Match attach registrationToken if registration_id undefined (appfeel#45)
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.

3 participants