Skip to content

feat(tui): add a TOTAL row with totals and portfolio history chart - #126

Open
ivysotskyi wants to merge 3 commits into
igoropaniuk:mainfrom
ivysotskyi:feature/totals-in-separate-row
Open

ivysotskyi wants to merge 3 commits into
igoropaniuk:mainfrom
ivysotskyi:feature/totals-in-separate-row

Conversation

@ivysotskyi

@ivysotskyi ivysotskyi commented Nov 11, 2026 •

Copy link
Copy Markdown

Adds portfolio-level totals to the dashboard and a chart of how the
portfolio's value moved over time.

@igoropaniuk

Copy link
Copy Markdown
Owner

Could you check this spec for how commit message should be structured (https://www.conventionalcommits.org/en/v1.0.0/#specification), or just check commit history. thanks!

The separate Total line only showed market value. Move it into the
table as a bold TOTAL row that stays at the bottom when sorting, and
add total cost, daily change (absolute and %) and unrealized P&L, all
in the base currency.

Signed-off-by: Ivan Vysotskyi <ivan.vysotskyy@gmail.com>
There was no way to see how the portfolio as a whole moved over time.
Pressing g on the TOTAL row now charts what the current holdings were
worth each day over 1M to 5Y, against a cost-basis line, with a
summary of change, high, low and max drawdown.

Signed-off-by: Ivan Vysotskyi <ivan.vysotskyy@gmail.com>
@ivysotskyi
ivysotskyi force-pushed the feature/totals-in-separate-row branch from 9ae02e4 to 7cf5abd Compare November 11, 2026 19:16
@ivysotskyi ivysotskyi changed the title totals in separate row feat(tui): add a TOTAL row with totals and portfolio history chart Nov 11, 2026
@ivysotskyi

Copy link
Copy Markdown
Author

Could you check this spec for how commit message should be structured (https://www.conventionalcommits.org/en/v1.0.0/#specification), or just check commit history. thanks!

done and force-pushed

@igoropaniuk
igoropaniuk self-requested a review November 12, 2026 15:09
@igoropaniuk

Copy link
Copy Markdown
Owner

@ivysotskyi thanks! It seems that CI is still failing :(

https://github.com/igoropaniuk/stonks-cli/actions/runs/43085664533/job/131274058703?pr=126

You can ensure that all tests are passing successfully by running make check-all inside the repo (more details in: https://github.com/igoropaniuk/stonks-cli/blob/main/CONTRIBUTING.md#run-all-checks-at-once)

Signed-off-by: Ivan Vysotskyi <ivan.vysotskyy@gmail.com>
@ivysotskyi
ivysotskyi force-pushed the feature/totals-in-separate-row branch from 2e2f502 to 1cf851c Compare November 12, 2026 23:55
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.

2 participants