Skip to content

Refactor the Transaction History - #3577

Draft
malik1004x wants to merge 26 commits into
devfrom
cw-1632-tx-history-refactor
Draft

malik1004x wants to merge 26 commits into
devfrom
cw-1632-tx-history-refactor

Conversation

@malik1004x

@malik1004x malik1004x commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Issue Number (if Applicable): Fixes #

Description

Refactors the transaction history, as well as any other item shown in the history panel. Removes dependency on MobX, removes duplicated logic across all wallet types and cleans up DashboardViewModel.

The refactor switches from manually recomputing the list on every item change (which caused severe lag issues on slower devices) to a stream-based approach where history sources send list-altering events to the Bloc. This was confirmed to be substantially faster on my low-end testing phone (Redmi Note 12).

Pull Request - Checklist

  • Initial Manual Tests Passed
  • Double check modified code and verify it with the feature/task requirements
  • Format code
  • Look for code duplication
  • Clear naming for variables and methods
  • Manual tests in accessibility mode (TalkBack on Android) passed

@malik1004x
malik1004x force-pushed the cw-1632-tx-history-refactor branch from 94ff037 to d4a44c8 Compare August 28, 2026 21:05
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.

1 participant