Skip to content

Revert "Add feature to send an e-mail to staff when a student enrolls" - #18762

Merged
schenedx merged 1 commit into
masterfrom
schen/revert_email_send
Aug 9, 2018
Merged

Revert "Add feature to send an e-mail to staff when a student enrolls"#18762
schenedx merged 1 commit into
masterfrom
schen/revert_email_send

Conversation

@schenedx

@schenedx schenedx commented Aug 8, 2018

Copy link
Copy Markdown
Contributor

This reverts commit 370501d, which is the PR at https://github.com/edx/edx-platform/pull/16669

@nasthagiri @nedbat Please review

FYI @clemente @pomegranited

@edx-status-bot

Copy link
Copy Markdown

Your PR has finished running tests. There were no failures.

@schenedx

schenedx commented Aug 9, 2018

Copy link
Copy Markdown
Contributor Author

@nedbat Since @nasthagiri is out on vacation, do you mind double check if my revert did not miss anything?
If I don't hear anything from either of you, I will merge by the end of day. Thank you

@nedbat

nedbat commented Aug 9, 2018

Copy link
Copy Markdown
Contributor

This seems to be an accurate reversion of the original PR. It's not exactly the same lines because there have been systemic changes to test sharding in the meantime.

@schenedx
schenedx merged commit c5347af into master Aug 9, 2018
@schenedx
schenedx deleted the schen/revert_email_send branch August 9, 2018 19:28
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production on Friday, August 10, 2018.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants