Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Attached is a draft D7 port. The section of the code that checks for missing country codes produces an error, but I am able to successfully receive SMSs through Twilio using this early port of the SMS Framework: http://drupal.org/node/795202
I have not tested incoming SMSs yet. I will update this issue once I have done more testing, but I figured I'd go ahead and get this posted since it is at least mostly working.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | sms_twilio-D7port-1194626-02.patch | 16.86 KB | nmcclain |
| #1 | sms_twilio-D7port-1194626.patch | 7.75 KB | nmcclain |
| sms_twilio.module.txt | 11.9 KB | nmcclain | |
| sms_twilio.info | 228 bytes | nmcclain |
Comments
Comment #1
nmcclain commentedHere's a patch for the D7 port against 6.x-1.0.
Comment #2
nmcclain commentedI tested this more carefully, fixed an error with empty country codes, and ran it through Coder review. I think it's ready for a 7.x-dev branch.
This patch is against 6.x-1.0.
Comment #3
nmcclain commentedComment #4
univate commentedOk, committed this with a few changes (like removing the country list).
Wondering:
* should attempt to upgrade to latest twilio api at the same time here?
* whether we include a copy of the twilio php code here with this module so there is no confusion which version you need to use or continue to rely on a external library?