laravel-notification-channels / fcm

Firebase Cloud Messaging (FCM) notifications channel for Laravel

https://laravel-notification-channels.com/

MIT License

495 stars 127 forks source link

Closed akhzarjaved closed 1 year ago

akhzarjaved commented 3 years ago

This PR Fixes #43

Send notification to topic or condition if token is not specified and topic or condition are specified.

This can be used to send a notification to multiple users without any token.

44 is unable to rebase so it can be closed after this PR

dipesh008 commented 3 years ago

When can this be fixed

dipesh008 commented 3 years ago

@akhzarjaved akhzarjaved is this changed merged. I am facing the same issue

dipesh008 commented 3 years ago

@chrisbjr chrisbjr Please accept this PR we are unable to send messages to topic.

akhzarjaved commented 3 years ago

@dipesh008 No it hasn't yet merged. I have forked it to handle topics.

walvespit commented 3 years ago

@dipesh008 No it hasn't yet merged. I have forked it to handle topics.

Thank you a lot man, i just forked it and used your solution, you saved me a lot of work!!!

atymic commented 3 years ago

Minor fix then I can merge

dipesh008 commented 3 years ago

Minor fix then I can merge

Thank you for approving the changes, when do you pan to merge and release ? it will be a big help.

akhzarjaved commented 3 years ago

@atymic Just updated according to your suggestion. Have a look please

ivanwitzke commented 3 years ago

When will this be merged? For now I have to use the fork, but would be best to use the original repository....

eliasjtg commented 3 years ago

Any update?

AhmedNourJamalElDin commented 3 years ago

@atymic Can we merge this, please?

ankurk91 commented 2 years ago

I ended up by creating my own package