Remove RTV Update User SMS feature flag - Githubissues

DoSomething / chompy

:card_index: The DoSomething.org third-party importer.

MIT License

1 stars 0 forks source link

Remove RTV Update User SMS feature flag #170

Closed aaronschachter closed 4 years ago

aaronschachter commented 4 years ago

What's this PR do?

This pull request removes the check for the update_user_sms_enabled config as well as the config itself, since this feature has been turned on.

This helps to simplify the import and test coverage, as a predecessor for additional import modifications we need to make in a future PR.

This PR branches off #169 and will need to be edited to merge into master once #169 is approved.

How should this be reviewed?

👀

Any background context you want to provide?

🐸

Relevant tickets

References [Pivotal #172805635](https://www.pivotaltracker.com/story/show/172805635.

Checklist

[ ] This PR has been added to the relevant Pivotal card.
[ ] Documentation added for new features/changed endpoints.
[ ] Added appropriate feature/unit tests.