-
Notifications
You must be signed in to change notification settings - Fork 204
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
Support send to multiple topics #261
Comments
@vibgy Excellent PR! However, could it be better to introduce a brand new recipient key |
@eladnava Thank you. |
Cool that this has been introduced! I think we should use the "condition" parameter. This might complicate the interface for v1 or force us to rethink some things. (See and discuss at #238) |
I should clarify: let's use "condition" as it is closer to the underlyibg interface :) Don't forget to add yourself as contributor in the package :) |
Sure.. I'll update the PR. Thank you @hypesystem ! |
I updated the PR. Please review. Thanks you @hypesystem @eladnava .. |
PR has been merged. Thanks for your hard work! |
FCM has introduced a way to send notification to multiple topics using the "condition" parameter in send API.
Ref: https://firebase.google.com/docs/cloud-messaging/http-server-ref
The text was updated successfully, but these errors were encountered: