Skip to content

Add price/percentage suggestions and asset info to price alert scene#109

Merged
DRadmir merged 6 commits intomainfrom
81-consolidate-set-price-alerts-scene-with-ios
Apr 15, 2026
Merged

Add price/percentage suggestions and asset info to price alert scene#109
DRadmir merged 6 commits intomainfrom
81-consolidate-set-price-alerts-scene-with-ios

Conversation

@DRadmir
Copy link
Copy Markdown
Contributor

@DRadmir DRadmir commented Apr 8, 2026

  • Add PriceSuggestionsFormatter and PercentageSuggestionsFormatter with tests
  • Show suggestion chips (TabsBar) when input is empty, confirm button when filled
  • Move currency symbol to left of input for price type alerts
  • Display asset info with price and 24h change below current price
  • Wire new ViewModel flows for suggestions, asset, and price state

Close: #81

Screenshot_20260408_104207 Screenshot_20260408_104221 Screenshot_20260408_105050 Screenshot_20260408_105056

@DRadmir DRadmir self-assigned this Apr 8, 2026
@DRadmir DRadmir added the Android label Apr 8, 2026
@DRadmir DRadmir marked this pull request as draft April 10, 2026 07:29
DRadmir added 2 commits April 14, 2026 19:34
- Add PriceSuggestionsFormatter and PercentageSuggestionsFormatter with tests
- Show suggestion chips (TabsBar) when input is empty, confirm button when filled
- Move currency symbol to left of input for price type alerts
- Display asset info with price and 24h change below current price
- Wire new ViewModel flows for suggestions, asset, and price state
Replace platform-specific PercentageSuggestionsFormatter and
RoundingFormatter with shared Rust implementation exposed via
Gemstone UniFFI bindings. Both iOS and Android now call
priceAlertPercentageSuggestions and priceAlertRoundedValues directly.
@DRadmir DRadmir force-pushed the 81-consolidate-set-price-alerts-scene-with-ios branch from c9d5949 to a7a696b Compare April 14, 2026 14:41
@DRadmir DRadmir marked this pull request as ready for review April 14, 2026 14:45
@DRadmir DRadmir merged commit f5a2630 into main Apr 15, 2026
1 of 2 checks passed
@DRadmir DRadmir deleted the 81-consolidate-set-price-alerts-scene-with-ios branch April 15, 2026 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consolidate set price alerts scene with iOS

2 participants