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

Enhancements and Fixes: Routing Paths, Wallet Components, and Address Checksum #186

Merged
merged 2 commits into from
Jul 16, 2024

Conversation

aeddaqqa
Copy link
Collaborator

@aeddaqqa aeddaqqa commented Jul 16, 2024

Description:

This pull request introduces several key updates, including refactoring and critical bug fixes, to enhance the Camino Wallet and associated components. Below is a summary of the changes:

Refactoring:

  • Update Routing Paths for Wallet Components: Simplified and optimized the routing paths to improve navigation and code readability.
  • ChainTransfer.vue: Refactored to dispatch necessary store actions for better state management.
  • Global Helper Calls: Updated to ensure existence checks before invocation, enhancing stability.

Bug Fixes:

  • Login: Updated the outdated MSIG definition to resolve login issues.
  • Checksum Addresses: Ensured that checksum addresses are used throughout the Camino Wallet UI to prevent address mismatch errors.
  • Deposit Calculation: Added no reward duration to the deposit calculation to improve accuracy.
  • New Features: Wallet Verification: Added wallet verification options to the Verify Wallet page, including a new component for verifying wallet settings.
  • Account Management: Added a setAccount method and implemented delete-account functionality for improved user control.

@aeddaqqa aeddaqqa requested review from Ysrbolles and Ayagoumi July 16, 2024 09:11
@Ysrbolles Ysrbolles changed the title Suite Enhancements and Fixes: Routing Paths, Wallet Components, and Address Checksum Jul 16, 2024
@aeddaqqa aeddaqqa merged commit d1c5b10 into suite-c4t Jul 16, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants