Skip to content

fix: use proper value set on settings instead of the remember value#38

Merged
isaacsa51 merged 2 commits into
masterfrom
fix/ui/recurring_payments_view
Jun 23, 2026
Merged

fix: use proper value set on settings instead of the remember value#38
isaacsa51 merged 2 commits into
masterfrom
fix/ui/recurring_payments_view

Conversation

@isaacsa51

Copy link
Copy Markdown
Owner

Description

When changin view modes on settings, on master the view wasn't changing.

Change strategy

Use correct value for the settings datastore saved instead of the hardcoded value.

Implemented

  • Implemented coverage of UI Screenshot and E2E tests on History following flows like:
    • Edit items
    • Delete item
    • Tap item to show TicketDialog
    • Open history to see expenses
    • Open history to see recurring expenses (from active period and future periods.)

Working demo

Screen_recording_20260622_221433.mp4

Testing

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

Notes

Changed default viewmode of recurring payments into VERTICAL, idea of a small ticket was nice, but several uses had the value as vertical for convenience.

@isaacsa51 isaacsa51 self-assigned this Jun 23, 2026
@isaacsa51 isaacsa51 added the bug Something isn't working label Jun 23, 2026
@isaacsa51 isaacsa51 merged commit e439ca1 into master Jun 23, 2026
1 check passed
@isaacsa51 isaacsa51 deleted the fix/ui/recurring_payments_view branch June 23, 2026 05:23
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