-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Notifier] Add Firebase bridge #33968
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
Conversation
src/Symfony/Component/Notifier/Bridge/Firebase/FirebaseTransport.php
Outdated
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/Notification/IOSNotification.php
Outdated
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/FirebaseOptions.php
Outdated
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/FirebaseOptions.php
Outdated
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/FirebaseTransport.php
Outdated
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/FirebaseTransport.php
Outdated
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/Notification/AndroidNotification.php
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/Notification/AndroidNotification.php
Outdated
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/Notification/IOSNotification.php
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/Notification/WebNotification.php
Show resolved
Hide resolved
src/Symfony/Component/Notifier/Bridge/Firebase/FirebaseTransportFactory.php
Outdated
Show resolved
Hide resolved
Thank you @Jeroeny. |
After seeing fabpot's talk today talking about the Firebase notifier coming in 5.1 I looked at the repo and noticed that this Firebase notifier is based on the old Http API, that allows quite a lot less options than the new one, specifically, the new one allows you to send in 1 message different options for different platforms (iOS, Android, Web) instead of having to send separate messages for each of the platforms. Not sure @Jeroeny if this is something that should be changed before the 5.1 release? |
@acasademont I would recommend to open a new issue. |
This would add Firebase integration for the Notifier component. With Firebase you can send push notifications to the users of you Android and iOS app and website (formerly known as Google Cloud messaging).
I'm not sure if it's possible to have this merged, like the other bridges. Or if I should create a stand-alone repository? That'd be fine too.
Also it's now using the
ChatMessage
as implementation ofSymfony\Component\Notifier\Message\MessageInterface
, but I feel like this component could use aPushMessage
or something similar. Although I'm not sure if it would contain more thansubject
that theChatMessage
does.