CoreWiki icon indicating copy to clipboard operation
CoreWiki copied to clipboard

Add 'EmailConfirmed' checking before sending notification emails when 'RequireConfirmedEmail' is true

Open erwindevreugd opened this issue 6 years ago • 0 comments

Right now we only check the value of CanNotify on CoreWikiUser if we can send emails to a user.

We should also check the value of EmailConfirmed in the AspNetUsers table to detirmine if we can send emails to a user. This should take into account the value of the RequireConfirmedEmail application setting.

Notification Emails

EmailConfirmed RequireConfirmedEmail CanNotify Action
False False False Don't send
False False True Send
False True False Don't send
False True True Don't send
True False False Don't send
True False True Send
True True False Don't send
True True True Send

Forgot Password Email

EmailConfirmed RequireConfirmedEmail CanNotify Action
False False N/A Send
False True N/A Don't send
True True N/A Send

Confirmation Email

EmailConfirmed RequireConfirmedEmail CanNotify Action
False False N/A Don't send
False True N/A Send
True True N/A N/A

erwindevreugd avatar Jul 17 '18 04:07 erwindevreugd