neolikotsi/laravel-smsportal

SMS Portal Notification Channel for laravel

laravel-smsportal's Issues

Laravel 7.* Support

Hey,

Wanting to use this on Laravel 7.* but having issues with the guzzle version as Laravel now requires it at ^6.3 and this package is requiring ~5.0

Delivery Environment

Hey,

It's a little odd ad very counter-intuitive that by default config, this will send smses on all environments except production

I was going to make a PR, but there are two main options here:

1. Simple: Have a single env variable which is just an enabled flag: (SMS_PORTAL_DELIVERY_ENABLED, true)
   It wouldn't care what environment you're on, it's configurable and very easily understood

2. Environment aware: Have an array in the config with they key/val being environment/bool. The implementation then looks at app env, finds the key in the array based on the env and looks up it's enabled state.

Option 2 would mean the least impact for users and by default we can disable sending sms's out on testing environments etc
But it would be a bit more prone to issues (if someone has an environment outside the norms, it would give a key error etc)

Thoughts?