Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I am trying create a site for where authenticated users can register their mobile numbers via a block and then be automatically sent an sms when a content of certain types are created. I have clickatell working ok to send confirmations and send to phone works - I have also created a rule that states that every time a content of type x is saved send an sms.
When I get to the rule to send to a destination number I cannot see an option to add all users with confirmed numbers.
Comment | File | Size | Author |
---|---|---|---|
#4 | smsframework-sms_notification_to-1933034-3.patch | 1.54 KB | almaudoh |
#3 | sms_notification_to-1933034-3.patch | 1.54 KB | alansaviolobo |
Comments
Comment #1
alansaviolobo CreditAttribution: alansaviolobo commentedComment #2
alansaviolobo CreditAttribution: alansaviolobo commentedThis can be achieved with the following lines of code added to the sms_blast.module.
Comment #3
alansaviolobo CreditAttribution: alansaviolobo commentedrolled a patch for the same
Comment #4
almaudoh CreditAttribution: almaudoh commentedRe-uploaded patch in #3 above with "smsframework-" prefix so the testbot can pick it up...
Comment #5
almaudoh CreditAttribution: almaudoh commentedComment #6
almaudoh CreditAttribution: almaudoh commentedComment #7
almaudoh CreditAttribution: almaudoh commentedThis functionality should be incorporated into the sms_user module, not sms_blast
Comment #8
alansaviolobo CreditAttribution: alansaviolobo commentedI believe since this functionality is helping you send sms to multiple users, it would make more sense to be in sms_blast rather than sms_user
Comment #9
almaudoh CreditAttribution: almaudoh commentedIt is creating an unnecessary dependency on sms_blast module because all the functionality is from sms_user module. Some admins may want this functionality but not sms_blast module (sms_blast is just a convenient ui).
Comment #11
almaudoh CreditAttribution: almaudoh commentedCNW for #7.
Comment #12
almaudoh CreditAttribution: almaudoh commented