igniter-labs / ti-ext-smsnotify

Send SMS notifications to both restaurant and customer about an order or reservation, including OTP verification.
MIT License
7 stars 4 forks source link

Send Test Message gets an error #12

Closed mgmaik closed 2 years ago

mgmaik commented 2 years ago

Expected behavior: Sending a test text message to the store phone number

Actual behavior: Getting an error message: "Missing config. You must set either the username & password or SID and auth token"

System log: NotificationChannels\Twilio\Exceptions\InvalidConfigException: Missing config. You must set either the username & password or SID and auth token in /var/www/vhosts/theshakery.nl/order.theshakery.nl/extensions/igniterlabs/smsnotify/vendor/laravel-notification-channels/twilio/src/Exceptions/InvalidConfigException.php:11

Reproduce steps: Installed SMS Notify, signed up to Twillio and configured phone number then added an account SID, Auth Token and Number Set as default and enabled.

Opened up SMS Templates page and pressed the Send Test Message button.

Version: TI version 3.2.0.

Additional Information: SMS Notify v2.1.1

martin-ro commented 2 years ago

Same issue for me

martin-ro commented 2 years ago

I just got it working by adding

TWILIO_AUTH_TOKEN=xxx TWILIO_ACCOUNT_SID=xxx

to my .env

mgmaik commented 2 years ago

That seems to work!

Also not forgetting to add TWILIO_FROM=XXX

sampoyigi commented 2 years ago

Fixed https://github.com/igniter-labs/ti-ext-smsnotify/commit/7a96fc19243d53c00497b49b4fca1b40b2163ef9