Skip to content

refactor(myaccount): merge account info into UserProfileScreen#858

Merged
bmc08gt merged 2 commits into
code/cashfrom
refactor/merge-account-info-into-profile
Jun 5, 2026
Merged

refactor(myaccount): merge account info into UserProfileScreen#858
bmc08gt merged 2 commits into
code/cashfrom
refactor/merge-account-info-into-profile

Conversation

@bmc08gt

@bmc08gt bmc08gt commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

Move account info (Public Key, Account ID, Push Token) from MyAccountScreen into UserProfileScreen, which is already a staff-only destination. This consolidates developer-facing info and removes the hidden gesture.

  • Add account info state, copy events, and clipboard handling to UserProfileViewModel
  • Add Account Info section to UserProfileScreenContent with swipe-to-copy and per-index corner rounding
  • Remove account info state, events, and title-tap gesture from MyAccountScreen/ViewModel
  • Delete AccountInfoHeader.kt (dead code)
  • Add resetOnDismiss to SwipeActionRow for non-destructive swipe actions that animate back after triggering
  • Apply @stable, remember optimization, and KDoc to SwipeAction
  • Update tests for both ViewModels

Move account info (Public Key, Account ID, Push Token) from MyAccountScreen hidden title-tap gesture into UserProfileScreen, which is already a staff-only destination. This consolidates developer-facing info and removes the hidden gesture.

- Add account info state, copy events, and clipboard handling to UserProfileViewModel
- Add Account Info section to UserProfileScreenContent with swipe-to-copy and per-index corner rounding
- Remove account info state, events, and title-tap gesture from MyAccountScreen/ViewModel
- Delete AccountInfoHeader.kt (dead code)
- Add resetOnDismiss to SwipeActionRow for non-destructive swipe actions that animate back after triggering
- Apply @stable, remember optimization, and KDoc to SwipeAction
- Update tests for both ViewModels

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>
@bmc08gt bmc08gt self-assigned this Jun 5, 2026
@github-actions github-actions Bot added type: refactor Code restructuring, no behavior change area: ui Compose UI, theme, components, resources and removed type: refactor Code restructuring, no behavior change labels Jun 5, 2026
Signed-off-by: Brandon McAnsh <git@bmcreations.dev>
@bmc08gt bmc08gt force-pushed the refactor/merge-account-info-into-profile branch from cd5d164 to 401e6c0 Compare June 5, 2026 00:25
@github-actions github-actions Bot added area: network gRPC, connectivity, API, exchange rates type: refactor Code restructuring, no behavior change labels Jun 5, 2026
@bmc08gt bmc08gt merged commit 9c9aeff into code/cash Jun 5, 2026
3 checks passed
@bmc08gt bmc08gt deleted the refactor/merge-account-info-into-profile branch June 5, 2026 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: network gRPC, connectivity, API, exchange rates area: ui Compose UI, theme, components, resources type: refactor Code restructuring, no behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant