-
Notifications
You must be signed in to change notification settings - Fork 0
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
#145028655 Send alerts via sms #9
base: develop
Are you sure you want to change the base?
Conversation
- Twilio integration for sending sms alerts. - Added sms integration option to the integrations page.
…/andela/Healthchecks_ThunkNalla into ft-send_alerts_via_sms-145028655
- Adding the latest requirements to the requirements file.
- Adding the latest requirements to the requirements file.
- Did some refactoring in the sms alerts configuration to fix the number to send to error that was apparent before.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recommend removing the Twilio configurations from the repo's history as outlined in the link I shared a while back and load the settings through environment variables.
Resource: Twelve-Factor App
requirements.txt
Outdated
@@ -1,9 +1,32 @@ | |||
appdirs==1.4.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file has conflicts, resolve them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let this one through
What does this PR do?
Adds a feature that sends alerts on a user's job via SMS.
Description of Task to be completed?
Any background context you want to provide?
The feature is being developed by @davidmukiibi and @victoriaaoka.
What are the relevant pivotal tracker stories?
The pivotal tracker story for this work can be found here