mirror of
https://github.com/abh/ntppool.git
synced 2026-04-26 03:55:52 +03:00
[GH-ISSUE #185] Allow users to change their email address. #66
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ntppool#66
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Jas0n99 on GitHub (Jan 23, 2020).
Original GitHub issue: https://github.com/abh/ntppool/issues/185
There needs to be a main account page for a user to update their email address (globally) that would also update all team accounts.
The new system allows for multiple accounts, and a person can invite their new email address to each, but that seems a very roundabout way to do it, and also requires an admin to remove the old emails.
Just something for the future to-do...
@Jas0n99 commented on GitHub (Jan 23, 2020):
Nevermind, just saw it was a dupe of #154
@abh commented on GitHub (Jan 23, 2020):
#154 is just the password, no?
Changing the email is tricky if your login was via one of the 3rd party logins (because then the email address comes from there). Inviting the new address works around that (and I think when the admin does it, it's a little more explicit ...). Shrug.
@Jas0n99 commented on GitHub (Jan 23, 2020):
Oh yeah, I didn't think about the interaction of accounts that authenticate with the 3rd parties.
You're right, the current invite method is probably the best for now, it's not that often someone requests to have their email address changed.
@abh commented on GitHub (Feb 21, 2020):
Workaround live on beta site: https://manage-beta.grundclock.com/manage/account
@Jas0n99 commented on GitHub (Feb 22, 2020):
I'm not sure what the workaround is? I saw the text on the Account Information page, "To change your email address, add the new address to the team and login to remove the old address."
I used my non-admin account to test, and even after adding a second email address I do not see the boxes to remove any email addresses.
I also noticed on the Account Management page it says, "To remove a user from the account, please email ..." So that seems a little conflicting?
@abh commented on GitHub (Mar 2, 2020):
@Jas0n99 thank you -- I totally forgot that regular users can't remove email addresses from the account. Fixed now, on the beta site. I forgot to fix the conflicting text though.