Skip to content

fix: implement correct usage of History actions#40

Merged
isaacsa51 merged 2 commits into
masterfrom
fix/ui/history_actions
Jun 24, 2026
Merged

fix: implement correct usage of History actions#40
isaacsa51 merged 2 commits into
masterfrom
fix/ui/history_actions

Conversation

@isaacsa51

@isaacsa51 isaacsa51 commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Description

Main branch is to fix issue #37, the merge of this closes it.

Also revamped some UI regarding editing an expense and did a major deocuple/refactor regarding all components and UI for History section.

Change strategy

Made the usage correctly for SwipeableActions per item on the list of transactions and recurring expenses.

Also decoupled a lot of the HistoryScreen, merge and organized using packages like:

  • RecurrenceConfigSheet that lives inside history.edit and some other components like bottom sheets inside history.dialogs

Implemented

New packages edit dialogs section inside the history package to better find and decouple a lot of components inside HistorySections. Which now it doesn't exist and now it only renders each file if needed instead of having the whole thing.

Working demo

Demo of the fix applied:
https://github.com/user-attachments/assets/0ffbd2a3-8742-4d12-a58d-e9a642dfaec1

Demo of the new UX regarding editing an expense
https://github.com/user-attachments/assets/5f1f83bc-f36c-4f89-8d9f-374ec07dfb3e

Testing

Added new file for testing editing a transaction:
image
image
image

  • Ran ./gradlew :app:compileDebugKotlin :wear:compileDebugKotlin :sync-contract:compileKotlin :app:testDebugUnitTest
  • Added/updated tests when applicable
  • Added screenshots, E2E test, or screen recordings for UI changes

Notes

N/A

@isaacsa51 isaacsa51 self-assigned this Jun 24, 2026
@isaacsa51 isaacsa51 added the bug Something isn't working label Jun 24, 2026
@isaacsa51 isaacsa51 merged commit b43ffc3 into master Jun 24, 2026
1 check passed
@isaacsa51 isaacsa51 deleted the fix/ui/history_actions branch June 26, 2026 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant