Skip to content
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

Rewrite profile manager dialog #3805

Open
0oM4R opened this issue Jan 12, 2025 · 8 comments
Open

Rewrite profile manager dialog #3805

0oM4R opened this issue Jan 12, 2025 · 8 comments
Assignees
Labels
type_feature New feature or request
Milestone

Comments

@0oM4R
Copy link
Contributor

0oM4R commented Jan 12, 2025

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.

@0oM4R 0oM4R added the type_feature New feature or request label Jan 12, 2025
@0oM4R 0oM4R self-assigned this Jan 12, 2025
@ramezsaeed ramezsaeed added this to the 2.7.0 milestone Jan 14, 2025
@0oM4R
Copy link
Contributor Author

0oM4R commented Jan 15, 2025

  • Basic issues to be resolved:

    - Replaced the conditional fields with separate tabs; this should include moving some common fields to be separate components
    - Separate the connect button and the dialog component
    - Fix the checking if the password is stored; some times it is not functional
    - Move the functions to a utils 

  • Enhancements
    
 - add skeleton loading to handle lazy loading and checking stored passwords
    

- add ids and class names to make the automated test easier and more functional

@0oM4R
Copy link
Contributor Author

0oM4R commented Jan 16, 2025

Issue update:
add the Wallet_KEY to env, also on setting the stacks links will set the isSoredCredentials to env, to be globally accessible, and to avoid lazyloading related issue
working on separate the taps to a components

@0oM4R 0oM4R mentioned this issue Jan 20, 2025
15 tasks
@0oM4R
Copy link
Contributor Author

0oM4R commented Jan 20, 2025

issue update:
completed the login tab, working on the register tab.
note: some functionality may need to be refactored; we will update the issue with those update as well

refactor: removed isStoredCredentials; it doesn't provide any benefit

@0oM4R
Copy link
Contributor Author

0oM4R commented Jan 21, 2025

#1955

@0oM4R
Copy link
Contributor Author

0oM4R commented Jan 21, 2025

Issue update:
(WIP): create connect wallet tab, need to add the activation and create account logic
(TODO): move T&C dialog to a separate component and revisit reload logic, #1955

Refactor: enhanced the state management of the password field by using form and formValidator, to avoid state conflict

@0oM4R
Copy link
Contributor Author

0oM4R commented Jan 22, 2025

Issue update:
completed the create, activate and accept terms&condetinos parts
enhance relaod mnemonic validation to be enabled only if we have a valid mnemonic and got an error while getting the grid

TODO: check disable auto complete; and use the created components in profile manager component

@0oM4R
Copy link
Contributor Author

0oM4R commented Jan 23, 2025

Issue update:

  • applied the new changes to the profile manager
  • added suspense and skeleton loader
  • enhanced the checking of stored credentials

todo: apply BEM class names, and test the flow

@0oM4R
Copy link
Contributor Author

0oM4R commented Jan 29, 2025

issue update: test and fixed some flows, added the BEM ids Pr ready for review rn

@AhmedHanafy725 AhmedHanafy725 modified the milestones: 2.7.0, 2.8.0 Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type_feature New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants