-
Notifications
You must be signed in to change notification settings - Fork 33
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
Missing collapse key #57
Comments
Thanks for creating your first issue on php-fcm |
Thanks @Philipp91 I'll try and implement that. I've not seen Paragraph1/php-fcm so that's handy to see how they've implemented things. |
Thanks. If you prefer, I can also take a stab at it. Btw, is this repository still maintained? I haven't seen a reaction to my (unrelated) PR, there are several PRs open for a while, and @EdwinHoksberg hasn't been active on GitHub for a while. @EdwinHoksberg This may just be a coincidence (like vacation week or sth like that), in which case please disregard my comment. Though if you find yourself being less active on GitHub for whatever reasons, please consider giving @AndyGaskell or someone else who contributes a lot enough write access to this repository to allow merging PRs -- not sure about releases. |
Hi @Philipp91 |
@AndyGaskell - heh.... |
See docs: https://firebase.google.com/docs/cloud-messaging/concept-options#delivery-options
And another PHP implementation: https://github.com/Paragraph1/php-fcm/blob/b8d6fec2c5327ed8fbe09d4d1cc0b53467a340e0/src/Message.php#L151
The text was updated successfully, but these errors were encountered: