antonioribeiro / health

Laravel Health Panel
BSD 3-Clause "New" or "Revised" License
1.95k stars 197 forks source link

A bunch of issues with this #145

Open mwkcoding opened 5 years ago

mwkcoding commented 5 years ago

I've found a bunch of issues with this package so far.

The config gives you a "notifier" namespace where you'd assume you can change the namespace and use your own notifier. If you search the entire package, that config is NEVER used.

Also the package breaks if you don't SPECIFICALLY use a model for your notifications which is, in my opinion, not very user-friendly.

The package also needs a revision on the documentation since it just very lightly goes over some of the functionality.

On another note, I'm having issues with registering my own event listener on the RaiseHealthIssue. I've registered it in the EventServiceProvider and it doesn't register at all. (Laravel 5.7)