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

Admin only - Whitelist specific email addresses #17

Open
samsmith opened this issue Mar 27, 2015 · 4 comments
Open

Admin only - Whitelist specific email addresses #17

samsmith opened this issue Mar 27, 2015 · 4 comments

Comments

@samsmith
Copy link
Contributor

samsmith commented Mar 27, 2015

check whether the email address offered is already in the database, before doing the regex on domains. Email addresses already in the database are automatically valid.

@davidmiller
Copy link
Member

Not sure why we want this ?
Do you have more context?

@samsmith
Copy link
Contributor Author

samsmith commented Jul 1, 2019

as an admin, i can create custom /declare/1234abcd links for people who wish to declare but who don't have addresses in the whitelisted domains; when they wish to make another declaration later, they should be able to do so based on the email already being in the db, rather than asking me again.

@davidmiller
Copy link
Member

I don't think the link creation admin form not checking email suffixes is by design :)

But I can imagine wanting to whitelist specific emails

@davidmiller davidmiller changed the title NHS_EMAIL_SUFFIXES Admin only - Whitelist specific email addresses Jul 2, 2019
@davidmiller davidmiller added this to the Sprint July 19 milestone Jul 2, 2019
@samsmith
Copy link
Contributor Author

samsmith commented Jul 2, 2019 via email

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

No branches or pull requests

2 participants