From 3f168bf1a7b55d52d81ea9f4387b32120970ba88 Mon Sep 17 00:00:00 2001
From: Thomas Kaul <4159106+dtslvr@users.noreply.github.com>
Date: Sun, 28 Jun 2026 09:25:47 +0200
Subject: [PATCH] Bugfix/exclude valueInBaseCurrency of top holdings in public
 portfolio endpoint (#7155)

* Exclude valueInBaseCurrency

* Update changelog
---
 CHANGELOG.md                                           | 1 +
 apps/api/src/app/endpoints/public/public.controller.ts | 5 +++++
 2 files changed, 6 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 4269d69d4..199b0b383 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -16,6 +16,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+- Fixed an issue in the public access for portfolio sharing that exposed absolute values of the top holdings of ETFs
 - Fixed the time zone handling in the `api` test suite for deterministic execution in `UTC`
 
 ## 3.17.0 - 2026-06-26
diff --git a/apps/api/src/app/endpoints/public/public.controller.ts b/apps/api/src/app/endpoints/public/public.controller.ts
index 6e025936d..d0e459f01 100644
--- a/apps/api/src/app/endpoints/public/public.controller.ts
+++ b/apps/api/src/app/endpoints/public/public.controller.ts
@@ -201,6 +201,11 @@ export class PublicController {
             portfolioPosition.assetProfile.assetSubClass === AssetSubClass.CASH
               ? portfolioPosition.assetProfile.assetSubClassLabel
               : undefined,
+          holdings: portfolioPosition.assetProfile.holdings?.map(
+            ({ allocationInPercentage, name }) => {
+              return { allocationInPercentage, name };
+            }
+          ),
           ...(hasDetails
             ? {}
             : {