-
Notifications
You must be signed in to change notification settings - Fork 152
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
Support profile - move security settings onto separate screen #114
Conversation
@dd32, @pkevan, @tellyworth : this pr creates a new the text is just rough stubs and there's no design, but take a look at the code for both branches and let me know if you think this is a good architecture to build on |
I'm not deeply familiar with the profiles part, but seems like a sensible way to approach this initially. |
This commits the parts of #114 that don't affect production users, and can therefore be deployed now. Adding a link to this screen and removing the email/password settings from the built-in Edit screen will needs to wait until the WPORG Two Factor UI is done. The rewrite rule was added in `r19711-dotorg`. See WordPress/wporg-two-factor#26 git-svn-id: https://meta.svn.wordpress.org/sites/trunk@12321 74240141-8908-4e6f-9713-ba540dce6ec7
3e3bd92
to
e8ea29b
Compare
I've committed the parts that we can deploy now in 4db5f38 and |
e8ea29b
to
45655d3
Compare
I think we may need another path to security settings. Logged in, when I click "Edit Profile", I land on https://profiles.wordpress.org/{username}/profile/edit/ which doesn't have any references to security settings. I would have to already know the settings is in our "Forum profiles". Should we add a section to the General tab that leads directly to 2fa? I'm not sure if that's the best language or link treatment :). |
🤔 Do you have any ideas? It was named "Account" because previously the email/password settings were in a section of the profile page named "Account" Renaming it to "Security" makes sense for the 2FA settings, but I don't think users would expect that for email. Maybe "Account Settings" instead of "Edit Account" ? |
Yeah, that looks good to me. |
It's just labelled |
77f54ea
to
570f31e
Compare
🤔 Changing the menu label would also entail changing the rewrite rule and links to the page. I don't personally think it's worth it, but I'm not opposed to it either. I'd rather not tack that onto this PR, though, do you mind opening another issue for it? |
Closes WordPress#114 git-svn-id: https://meta.svn.wordpress.org/sites/trunk@12896 74240141-8908-4e6f-9713-ba540dce6ec7
WIP
Fixes WordPress/wporg-two-factor#26
Regular Profile:
Forum Profile