Add failed notification event

ankurk91 commented 4 years ago

Hi @chrisbjr,

This PR brings many improvements.

Use dependency injection for FCM Messaging client, so that we can mock it during testing 💪
De-dupe the try/catch logic in channel class 🧰
Dispatch Illuminate\Notifications\Events\NotificationFailed event on failure so that package consumers can take actions on it. 🚀

MessagingClient and Dispatcher implementations will be automatically resolved by Laravel IOC.
This is no breaking change in this PR, Every thing should work as before.

The repo does not have any test cases yet, i am planning to submit tests once this PR gets merged. 😄

ankurk91 commented 4 years ago

cc @atymic

Can you look into this PR.

ankurk91 commented 3 years ago

Anyone there?

laravel-notification-channels / fcm