Skip to content

feat: make autoswap warning values dynamic and reactive#1772

Draft
BullishNode wants to merge 1 commit into
developfrom
feat/autoswap-dynamic-values
Draft

feat: make autoswap warning values dynamic and reactive#1772
BullishNode wants to merge 1 commit into
developfrom
feat/autoswap-dynamic-values

Conversation

@BullishNode
Copy link
Copy Markdown
Contributor

@BullishNode BullishNode commented Jan 11, 2026

Summary

  • Display actual user-configured values in autoswap warning instead of hardcoded strings
  • Update database seeds to match model defaults (1M/2M instead of 500k/1M)
  • Read values from WalletBloc state and convert to BTC/sats for display
  • Respect user unit preference in autoswap warning
  • Trigger WalletRefreshed on settings save to reload values in WalletBloc
  • Apply BlocListener pattern in settings screen and bottom sheet

Changes

  • database_seeds.dart: Updated threshold values
  • autoswap_settings_screen.dart: Added WalletBloc refresh on save
  • autoswap_settings.dart: Added WalletBloc refresh on save
  • autoswap_warning_bottom_sheet.dart: Dynamic values display based on user settings

Test plan

  • Verify autoswap warning shows correct threshold values
  • Change threshold values in settings and verify warning updates
  • Toggle between BTC/sats and verify display format changes

Note: These changes were originally part of PR #1765 but have been split into a separate PR for cleaner review.

@BullishNode BullishNode marked this pull request as draft January 11, 2026 18:32
- Display actual user-configured values in autoswap warning instead of hardcoded strings
- Update database seeds to match model defaults (1M/2M instead of 500k/1M)
- Read values from WalletBloc state and convert to BTC/sats for display
- Respect user unit preference in autoswap warning
- Trigger WalletRefreshed on settings save to reload values in WalletBloc
- Apply BlocListener pattern in settings screen and bottom sheet
@BullishNode BullishNode force-pushed the feat/autoswap-dynamic-values branch 2 times, most recently from 675a817 to 146be07 Compare January 11, 2026 20:11
@BullishNode BullishNode requested a review from kumulynja January 11, 2026 23:33
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.

3 participants