-
Notifications
You must be signed in to change notification settings - Fork 8
Rewrite profile manager dialog #3805
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
Comments
|
Issue update: |
issue update: refactor: removed isStoredCredentials; it doesn't provide any benefit |
Issue update: Refactor: enhanced the state management of the password field by using form and formValidator, to avoid state conflict |
Issue update: TODO: check disable auto complete; and use the created components in profile manager component |
Issue update:
todo: apply BEM class names, and test the flow |
issue update: test and fixed some flows, added the BEM ids Pr ready for review rn |
Issue update:
|
work completed:
|
Make sure the tabs works fine, even on mobile |
WIP
|
agreed to handle the confirm password field on #4108 |
Which package/s are you suggesting this feature for?
Dashboard
Is your feature request related to a problem? Please describe
the current logic is very complex and in some cases it is unbredictable
Describe the solution you'd like
will update the issue with the changes and pattern I will follow once I check the code.
The text was updated successfully, but these errors were encountered: