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

Suite #190

Merged
merged 14 commits into from
Feb 21, 2024
Merged

Suite #190

merged 14 commits into from
Feb 21, 2024

Conversation

aeddaqqa
Copy link
Contributor

Pull Request Description

This pull request encompasses a variety of enhancements, fixes, and refactoring efforts aimed at improving the overall functionality, user experience, and maintainability of the application. Here's a concise overview of the changes:

  • Refactor app configuration and routing

    • Restructure the app's configuration and routing for improved organization and scalability.
      Fix issues related to changing wallet name or address when switching wallets
  • Addresses bugs causing incorrect updates to wallet name or address upon switching wallets.

  • Resolve issues with switching to the logged-in network

    • Fixes issues encountered when attempting to switch to the logged-in network.
      Introduce methods for updating the wallet name within the wallet component
  • Adds new methods to the wallet component facilitating the update of wallet names.

  • Change theme options to dark/light for improved clarity

    • Enhances user experience by switching theme options from day/night to dark/light.
  • Ensure proper update of wallet name even for single-signature wallets

    • Fixes a bug preventing the correct update of wallet names, particularly for single-signature wallets.
  • Fix header state inconsistency when navigating back to explorer from wallet

    • Addresses a header state issue occurring upon navigating back to the explorer from the wallet.
  • Refactor Footer and Navbar components, revamp Landing Page layout and content

    • Updates styles and structure of Footer and Navbar components, as well as enhances the layout and content of the Landing Page for better presentation and usability.
  • Introduce a new custom hook for handling wallet actions

    • Adds a custom hook dedicated to managing various actions within the wallet, improving code organization and readability.
  • Set default theme and data-theme

    • Establishes default theme settings and data-theme attributes for consistent application appearance.
  • Persist network selection in local storage implements functionality to persist network selection in local storage, enhancing user convenience.

These changes collectively contribute to a more robust, user-friendly, and maintainable application.

Ayagoumi and others added 13 commits December 28, 2023 16:29
* refactor: app configuration and routing
* feat: add loading indicator to when the networks are loading or on network change
add update wallet name to wallet component methods
fix switching to logged in network
* feat: change theme to dark/light instead of day/night
* refactor: transaction list selector cypress test
* Refactor theme switcher and main layout styles
* feat: inability to create a new network with the same infos as a known network
* fix: set default theme & and set a default data-theme when it's not set
* feat(switch network): persist selected network name to localStorage
* feat: change validation rules of adding a new custom network
fix update name of the wallet even if its singlesig
add new custom hook for handling actions in wallet
@aeddaqqa aeddaqqa requested review from Ayagoumi and mo-c4t February 21, 2024 10:19
@aeddaqqa aeddaqqa merged commit 487149e into suite-c4t Feb 21, 2024
8 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