marvinlabs / laravel-discord-logger

Logging to a discord channel in Laravel
MIT License
176 stars 41 forks source link

[Feat] Add enabled true/false in config file #10

Closed neoteknic closed 4 years ago

neoteknic commented 4 years ago

Hello, awesome package ! But can you jsut add an enabled feature in config ? Thank you !

vpratfr commented 4 years ago

I don't understand why you would need that.

You can simply remove that logger from the configuration if not needed

neoteknic commented 4 years ago

easier to have it in a .env , for dev/prod website for example bug it's ok with empty webhook url

neoteknic commented 4 years ago

Can you just remove the exception when no webhook url ? Because it spam the laravel.log (emergency logger). Or add and "enabled=>true" configuration (like laravel-debugbar) ?

vpratfr commented 4 years ago

If you want to change that from a .env file, simply change the logger (have a stack logger in production which includes discord + a stack logger in dev which does not). You can provide multiple log channels in the logging configuration file.

I do not want to add a setting to enable the package, sorry.