search

laravel-notification-channels / apn

APN push notifications channel for Laravel
http://laravel-notification-channels.com
MIT License
189 stars 56 forks source link

Send Notification to Log #9

Closed lucadegasperi closed 6 years ago

lucadegasperi commented 8 years ago

In some cases, when just testing the correct payload of the notification, I think it would be useful to have the notification logged instead of actually sent out.

What do you think? Do you have any suggestions on how I could test notifications without actually sending them?

luisdalmolin commented 8 years ago

I have done this in an SMS notification package by adding a pretend config option (https://github.com/luisdalmolin/laravel-zenvia-channel#configuration).

I think it's great to have such option.

barryvdh commented 8 years ago

I like this idea, would it make sense to do something like this across all packages?

dwightwatson commented 6 years ago

Is there any precedent within LNC for supporting logging now?

dwightwatson commented 6 years ago

Happy to implement this if some sort of standard could be agreed upon or if a PR is provided. Part of me is thinking that perhaps any pretend functionality for notifications might better belong in Laravel itself so it's handled at a global level, but I could be wrong there.