Skip to content
Snippets Groups Projects
jhungerford's avatar
commented on issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio

My current thinking is that an SMS hook should be included in civicrm-core, somewhat equivalent to hook_civicrm_alterMailParams:...

jhungerford's avatar
closed issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio
jhungerford's avatar
commented on issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio

Thinking more about this:...

jhungerford's avatar
commented on issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio

Also, just regarding Twilio assuming that the numbers are in the country for which the account is recognized; you're right, they do, and it works c...

jhungerford's avatar
commented on issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio

Looking at this situation a bit further, a hook still seems like the best solution to me....

jhungerford's avatar
commented on issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio

Regarding the lack of bug reports, I'm guessing that most organisations are not using alphanumeric sender ID, or if they are, they've implemented s...

bgm's avatar
commented on issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio

Thinking out loud, and I don't want to over-complicate things, but I feel like we will end up with 50 extensions of "Twilio for Australia", "Twilio...

jhungerford's avatar
commented on issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio

I'm not aware of any valid way to determine the appropriate country code in the general context. This may just be my lack of knowledge, but for exa...

bgm's avatar
commented on issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio

Hi @jhungerford, just curious:...

jhungerford's avatar
opened issue #6 "Feature proposal: dispatch an event to allow extensions to modify the phone number, subject, and message of an SMS before sending" at Extensions / Twilio
bgm's avatar
commented on merge request !3 "Update Twilio SDK to v8" at Extensions / Twilio

Thank you @jtwyman and @jamie - sorry for the delay.

bgm's avatar
pushed to branch master at Extensions / Twilio
bgm's avatar
accepted merge request !3 "Update Twilio SDK to v8" at Extensions / Twilio
bgm's avatar
commented on merge request !4 "Add composer.json and some info.xml tweaks" at Extensions / Twilio

Thanks @ufundo !

bgm's avatar
accepted merge request !4 "Add composer.json and some info.xml tweaks" at Extensions / Twilio
bgm's avatar
pushed to branch master at Extensions / Twilio
ufundo's avatar
opened merge request !4 "Add composer.json and some info.xml tweaks" at Extensions / Twilio
jamie's avatar
commented on merge request !3 "Update Twilio SDK to v8" at Extensions / Twilio

@bgm I haven't tested this exact MR, but we have been running this extension with the 8.2.2 version of the Twilio SDK since early July with no prob...

bgm's avatar
commented on merge request !3 "Update Twilio SDK to v8" at Extensions / Twilio

@jamie Can you confirm if this works for you?

miguelg's avatar
opened issue #5 "Issue with the sending of SMS through twilio" at Extensions / Twilio