Skip to content
This repository has been archived by the owner on Feb 24, 2023. It is now read-only.

Need to provide “lost password?” interface and workflow #478

Open
BlakeDraper opened this issue Dec 14, 2018 · 4 comments
Open

Need to provide “lost password?” interface and workflow #478

BlakeDraper opened this issue Dec 14, 2018 · 4 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@BlakeDraper
Copy link
Contributor

No description provided.

@BlakeDraper BlakeDraper added the enhancement New feature or request label Dec 14, 2018
@BlakeDraper BlakeDraper added this to the Beta release milestone Dec 14, 2018
@BlakeDraper BlakeDraper self-assigned this Dec 14, 2018
@lprivette
Copy link
Contributor

lprivette commented Aug 5, 2019

Made a skeleton for this interface in branch 478.

One thing to note: If user account emails are not unique then this process might be buggy. If a user has 3 different accounts registered with the same email address, then which account do they want to do the password reset for?

@BlakeDraper
Copy link
Contributor Author

The unique emails thing is an important point. It is also an issue for the lookup by email address we currently do for setting event collaborators. @nbaertlein can you comment on the unique email address question? I feel like we have discussed this in the past and may have actively decided to allow duplicates, but that does have some negative implications. What is your thinking? CC: @JChipault

@nbaertlein
Copy link
Collaborator

Perhaps we could only allow 1 active (is_active = true) email? An email can be used to create an account only if there's no matching active email. This was discussed to accommodate situations in which a small organization (like a rehab facility) has a primary email account, but staff may change. To allow for this though, we'd need a mechanism for an individual to "delete" and account, which we'd then only mark as inactive.

@BlakeDraper
Copy link
Contributor Author

Ok, so to restate as a rule:

Email address must be unique for active user accounts (is_active = true). Email addresses may be duplicated among inactive accounts (is_active = false).

To accommodate this we will need to add an interface for a user to Delete an account (or perhaps we call it "Deactivate" since that is actually what is happening). Reactivating an account would require intervention from a WHISPers administrator since there could potentially be a collision of email addresses. But we could create an interface for that re-activation request, or just leave it up to the general whispers email address and list that among the reasons to contact WHISPers staff in the FAQ and/or Contact tabs of the info dialog box.

@nbaertlein @JChipault Sound right?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants