Attached diff file to make changes needed to work with the newest api, 3.2.3

Comments

mcpuddin’s picture

Title: Update to use latest API 3.2.3 » Update to use latest API 3.2.3 / 2010 API
klflote’s picture

This worked great for me, but it added a drupal_set_message statement to print out the account SID; that needs to be removed.

nickgs’s picture

Status: Needs review » Reviewed & tested by the community

This worked for me as well and I applied this to the 7.x-1.x-dev branch.

This should be checked into both branches as soon as possible. Twilio integration will not work out of the box until then.

Thanks.

Nick
www.nickgs.com

nmcclain’s picture

Status: Reviewed & tested by the community » Needs review

It's always a shame to duplicate effort, but at this point it seems like this issue is best addressed by the patch in: http://drupal.org/node/1449494

I propose this issue be closed as a duplicate of that one.

nmcclain’s picture

Status: Needs review » Closed (fixed)

Marking this as closed (fixed) now that the patch in http://drupal.org/node/1449494 was committed (thanks @mcpuddin!) this module supports the new (2010) version of the Twilio API.

attila.fekete’s picture

Hi,
I noticed that the patch in https://drupal.org/node/1449494 was only commited to the 7.x branch. I made a patch that applies against the latest 6.x-dev and aims to bring the 6.x version of the module closer to the current 7.x branch. Also fixed some coding standard issues.

attila.fekete’s picture

Status: Closed (fixed) » Needs review
chrisrockwell’s picture

Status: Needs review » Closed (won't fix)

Closing this since the module doesn't support 6.x anymore