It would be great, if there is separate function for sending message, so that developer can just invoke that function with message and route.

CommentFileSizeAuthor
#2 2879366-separate-sms-function.patch3.02 KBmanojbisht_drupal
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

manojbisht_drupal created an issue. See original summary.

manojbisht_drupal’s picture

manojbisht_drupal CreditAttribution: manojbisht_drupal as a volunteer and at QED42 commented
Status: Active » Needs review
FileSize
2879366-separate-sms-function.patch3.02 KB

Please find attached patch for separate function for sending SMS.

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor at OpenSense Labs commented
Status: Needs review » Reviewed & tested by the community

Thanks @manojbish_drupal. There was a parse error due to missing ";" and instead of "Please send mobile number" it should be "Please provide mobile number". Will be Fixed will pushing your code. Thanks!!.

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor at OpenSense Labs commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.