Skip to content

feat(slice-007): add holding coin picker and amount input#8

Merged
fredericomozzato merged 13 commits into
mainfrom
feat/007-add-holding-coin-picker-amount-input
Apr 12, 2026
Merged

feat(slice-007): add holding coin picker and amount input#8
fredericomozzato merged 13 commits into
mainfrom
feat/007-add-holding-coin-picker-amount-input

Conversation

@fredericomozzato
Copy link
Copy Markdown
Owner

Summary

  • Implements a key to open a searchable coin picker dialog filtered by available coins not already held
  • Select coin → amount input → upsert holding into the portfolio
  • Right panel shows holdings table with Coin, Ticker, Amount, Price, Value, 24h% columns ordered by value descending, with portfolio total in header

Test Plan

  • a opens coin picker with all non-held coins listed
  • Typing filters the coin list in real-time
  • Selecting a coin opens the amount input dialog
  • Entering a valid amount saves the holding and updates the right panel
  • Entering an invalid amount shows an error message
  • Holdings appear sorted by value descending
  • Portfolio total updates correctly in the header
  • Coin picker correctly excludes already-held coins

🤖 Generated with Claude Code

fredericomozzato and others added 13 commits April 12, 2026 11:46
…empty filtered list

Closes finding I1 in docs/reviews/007-add-holding-coin-picker-amount-input/revision-1.md
Closes finding F1 in docs/reviews/007-add-holding-coin-picker-amount-input/revision-1.md
…puts

Closes finding I2 in docs/reviews/007-add-holding-coin-picker-amount-input/revision-1.md
…en database has no coins

Closes finding F1 in docs/reviews/007-add-holding-coin-picker-amount-input/revision-2.md
… doesn't effectively verify filtering

Closes finding I1 in docs/reviews/007-add-holding-coin-picker-amount-input/revision-2.md
…pens successfully

Closes finding I1 in docs/reviews/007-add-holding-coin-picker-amount-input/revision-3.md
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@fredericomozzato fredericomozzato merged commit b11698c into main Apr 12, 2026
5 checks passed
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.

1 participant