Closed (fixed)
Project:
Twilio SMS Integration
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
19 Mar 2012 at 15:13 UTC
Updated:
16 Aug 2012 at 19:21 UTC
Jump to comment: Most recent file
Comments
Comment #1
christianchristensen commentedPlease see attached patch.
Comment #2
nmcclain commentedGood work christianchristensen!!
I have successfully tested this, manually reviewed the code, ran it through coder_review to confirm it meets Drupal standards, and am marking it RTBC.
This feature is dependent on http://drupal.org/node/1449494 which is ready pending community review.
Comment #3
mcpuddin commentedThe version in git that the code pulls has an error in it, but that is that open source projects problem, not this one:
Notice: Undefined property: Services_Twilio_Rest_SmsMessage::$IsError in Services_Twilio_InstanceResource->__get() (line 61 of /Users/mcpuddin/Sites/sms/sites/all/libraries/twilio/Services/Twilio/InstanceResource.php).Other than that, I confirm this also works. Thanks for confirming that nmcclain and christianchristensen!
Comment #4
mcpuddin commentedcommitted