Merged
Conversation
* Fixed transactions order * Added changelog.
#147) * Refactor ledger addresses component into address-select and update related types. Expose `ledger-addresses` as `address-select` in the visual components. Update CHANGELOG.md to reflect these changes. * Fix border variable syntax in address-select component styles. * Update CHANGELOG.md to reflect the correct pull request number for the exposure of `ledger-addresses` as `address-select`. * Update version to 0.0.11, rename `ledger-addresses` to `addresses-table`, and refactor related components and types. Add new styles for addresses table and update CHANGELOG.md accordingly. * Refactor address-select component to use updated account type. Change `IAccount` to `IIndexedAccount` in address processing logic. * Update account type in getLedgerAddressByIndex helper from `IAccount` to `IIndexedAccount` for consistency with recent refactor. * Refactor `ledger-addresses` component to `address-table`, updating related types and styles. Modify imports and event handling to reflect the new naming convention. Update CHANGELOG.md for clarity. * Update version to 0.0.10 and add entry to CHANGELOG.md for recent changes, including fixes and renaming of components. * Refactor address-table component to use updated type `IAddressTableData` instead of `IAccountScreenData`. Update related type definitions for consistency. * Update type definitions in components.d.ts to replace `IAccountScreenData` with `IAddressTableData` for the `MvxAddressTable` interface, ensuring consistency across the project. * Refactor address-table and pagination components to improve class naming consistency and add data-testid attributes for better testing support. Update type definitions to align with recent changes in the ledger-connect types. * Refactor side-panel styles to improve visibility handling. Consolidate class application for better readability and maintainability, ensuring proper pointer events and opacity settings for the visible state.
mgavrila
approved these changes
Jul 5, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.