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

Lumen compatibility #72

Closed ssi-anik closed 3 years ago

ssi-anik commented 3 years ago

I'm using Lumen instead of Laravel. And by default the facade is disabled. Now, if I just want to use this package to send a notification, do I have to enable facade? I searched in https://laravel-notification-channels.com/fcm/#usage and your doc for Facade and Lumen, didn't show up any result. But the marked code is imported from the facade namespace.

Would you consider changing it to app()? Why force developer to use facade while it can be avoided? Even the https://github.com/kreait/laravel-firebase package didn't even force to use.

https://github.com/laravel-notification-channels/fcm/blob/8b95cd06b884529e6b14f0e32174d0470e548df1/src/FcmChannel.php#L108