openhealthcare / whopaysthisdoctor.org

whopaysthisdoctor.org website
2 stars 6 forks source link

Admin only - Whitelist specific email addresses #17

Open samsmith opened 9 years ago

samsmith commented 9 years ago

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 commented 5 years ago

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

samsmith commented 5 years ago

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 commented 5 years ago

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

samsmith commented 5 years ago

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

in that case, consider email addresses in the database already as the whitelist, and the admin interface being how you things get added to the whitelist...