Skip to content

Ban a few more email domains#5753

Merged
zachmargolis merged 2 commits intomainfrom
margolis-update-valid-emails
Dec 27, 2021
Merged

Ban a few more email domains#5753
zachmargolis merged 2 commits intomainfrom
margolis-update-valid-emails

Conversation

@zachmargolis
Copy link
Contributor

  • These have been associated with spam accounts at some of our
    partners

- These have been associated with spam accounts at some of our
  partners
Comment on lines +12 to +15
- temp2-login.com
- temp5-login.com
- temp9-login.com
- temp13-login.com
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also I think it's time to move this file to be a key in application.yml so we can roll it out more quickly and with less noise

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zachmargolis zachmargolis merged commit 68700ad into main Dec 27, 2021
@zachmargolis zachmargolis deleted the margolis-update-valid-emails branch December 27, 2021 20:17
peggles2 pushed a commit that referenced this pull request Jan 4, 2022
- These have been associated with spam accounts at some of our
  partners
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.

2 participants