From 5d30b5eaeeff9b630815c8ecbb5b4d5b2fd94fab Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gon=C3=A7alo=20Rodrigues?= Date: Sat, 13 Jun 2026 18:41:13 +0100 Subject: [PATCH] fix: portfolio current value divided by 100 erroneously currentPrice is already in cents, so currentValue = price * shares. The extra /100 made every holding appear worth 100x less than its cost, producing ~-100% P&L on every position and an empty allocation chart (values too small for Chart.js to render). Co-Authored-By: Claude Sonnet 4.6 --- apps/finance/services/api/main/portfolio.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/finance/services/api/main/portfolio.go b/apps/finance/services/api/main/portfolio.go index e869e18..e23f174 100644 --- a/apps/finance/services/api/main/portfolio.go +++ b/apps/finance/services/api/main/portfolio.go @@ -105,7 +105,7 @@ func computeHoldings(trades []Trade, prices map[string]int64) []Holding { currentPrice := prices[isin] - currentValue := int64(float64(currentPrice) * a.shares / 100) + currentValue := int64(float64(currentPrice) * a.shares) unrealizedPCL := currentValue - a.cost pct := 0.0 if a.cost > 0 {