-
Notifications
You must be signed in to change notification settings - Fork 125
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
locArgs on Android not working anymore with v1.6.2 #155
Comments
@pascaljung for my understanding: with version 1.6.0 you passed an array to |
I also checked the docs of node-gcm and Firebase again and it appears that passing a stringified array should be correct? Not sure if maybe the update to node-gcm 1.0.4 broke things somehow? But judging from their commit history, it shouldn't have. But just to make sure, could you test version 1.6.1, where we still use node-gcm 1.0.3 ? |
@pascaljung Any update on this? Do you still have the same issue with 1.6.1? |
Hi @alex-friedl, sorry didn't get to it yet :(. I'll give you an update once I've tried it |
@pascaljung Any update? :) |
Closing this for inactivity and because v3.1.0 now allows to use the new firebase-admin lib, which may render this issue obsolete as well |
Hey,
we just updated from v.1.6.0 to 1.6.2 and found that our notifications on iOS were not working anymore (didn't get the locArgs correctly). We found that according to your latest changes we had to provide a JSON string instead of an Array as parameter. Fixing that resolved the issue on iOS, however, Android suddenly stopped working (no notifications delivered to the device anymore). Honestly, I'm not quite sure why this is the case. According to the documentation of
node-gcm
and what has been discussed in #150, I can understand the change. However, it's not working for us with the new version. Btw: the non-legacy FCM API requires an usual Array instead of a stringified JSON array (see https://firebase.google.com/docs/reference/fcm/rest/v1/projects.messages#AndroidNotification.FIELDS.body_loc_args). Maybe it has something to do with that.For now we fixed the version to
1.6.0
and that's working fine. @alex-friedl do you have any ideas?Thanks and best,
Pascal
The text was updated successfully, but these errors were encountered: