-
Notifications
You must be signed in to change notification settings - Fork 591
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
PRO-4544 User settings group and password change #4238
Conversation
PRO-4544 As a user, I can update my password when logged in (requires implementation of User Setting groups).
This requires the implementation of User Setting groups. In the User Settings, which we started as a way to accommodate the need to allow users to set their admin UI language, we would like to add other settings moving forward. One of them is password. Sketch Tech Design
Acceptance Criteria
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great stuff, but I did ask to change "protected" to "protection" because it will resolve a lot of confusion when reading the source (even for me).
Summary
What are the specific steps to test this change?
Use Testbed branch feature-user-settings, link Apostrophe locally while in the current branch. Boot the app, force rebuild (CI=1 APOS_DEV=1 npx nodemon)
Requirements for Phase 1 and 2 should pass.
What kind of change does this PR introduce?
(Check at least one)
Make sure the PR fulfills these requirements: