Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consolidate SMS messages #29

Open
lizzyaustad opened this issue Sep 19, 2019 · 0 comments
Open

Consolidate SMS messages #29

lizzyaustad opened this issue Sep 19, 2019 · 0 comments
Labels

Comments

@lizzyaustad
Copy link
Member

Instead of repeating the logic and error catching each time we need to send an SMS, we should have one method called send_sms that accepts parameters for the phone number to send to and the message to be sent to that number. It should follow the try/except pattern currently in the test_text, initialize the Africa's Talking and SMS services, and print the response message. We should also add a validation check that the number we are trying to send a message to starts with a + character and add one in if it doesn't.

@jonm8116 jonm8116 self-assigned this Aug 16, 2020
@jonm8116 jonm8116 removed their assignment Mar 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants