Thanks for a great module. Just wanted to pass on some feedback. I was troubleshooting a twilio configuration issue, and it took me a while to figure out why it wasn't working and giving me a clickatell error. The issue ended up being that even though all of the additional gateways were deleted from admin/smsframework/gateways, a default had not been selected.

I am recommending a validation rule and/or system message that alerts the admin that a default gateway has not been selected.

I a have created a simple patch. Hope this helps.

CommentFileSizeAuthor
sms.admin_.patch458 bytesCProfessionals
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

CProfessionals created an issue. See original summary.

jacob.embree’s picture

Version: 7.x-1.0-rc1 » 7.x-1.x-dev
Status: Active » Needs work

The second argument to variable_get() makes if very unlikely the condition will ever be true.
Also, the spacing and capitalization is wrong.