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

Fix FcmMessage return types #148

Closed axlon closed 1 year ago

axlon commented 1 year ago

The FcmMessage class always returns $this on its fluent methods, which may be a child class of FcmMessage. This PR updates the annotations to reflect this. Ideally the self return types would also be removed in favor of static but this could be a breaking change for classes extending this one, since this PR only changes annotations there are no breaking changes.

dwightwatson commented 1 year ago

Nice, thanks!