search

ulsdevteam / sms-send-receive

0 stars 0 forks source link

Reuse sending code across both tools #2

Open ctgraham opened 3 years ago

ctgraham commented 3 years ago

The sending code here: https://github.com/ulsdevteam/sms-send-receive/blob/3a7fc2d31654a698966d1af8d20c94c9ab3bc690/reply.php#L59

Parallels the sending code here: https://github.com/ulsdevteam/sms-send-receive/blob/3a7fc2d31654a698966d1af8d20c94c9ab3bc690/send.php#L25-L39

but with less error handling.

Abstract the sending logic into a function shared by both tools.

wopsononock commented 3 years ago

Addressed in bf4b634d9d9eec79aeb4e1919dd8a698cef3a210