Merge 89e611dd16 into 449eaa1baa

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 - Prefilled the form in the account balance management with the current cash balance
+- Removed precious metals (Commodities) from the analysis data by continents, countries, sectors and markets
 
 ## 3.8.0 - 2026-06-07
 

apps/api/src/app/portfolio/portfolio.service.ts

@@ -1453,7 +1453,10 @@ export class PortfolioService {
     for (const [, position] of Object.entries(holdings)) {
       const value = position.valueInBaseCurrency;
 
-      if (position.assetProfile.assetClass !== AssetClass.LIQUIDITY) {
+      if (
+        position.assetProfile.assetClass !== AssetClass.LIQUIDITY &&
+        position.assetProfile.assetSubClass !== AssetClass.COMMODITY
+      ) {
         if (position.assetProfile.countries.length > 0) {
           markets.developedMarkets.valueInBaseCurrency +=
             position.markets.developedMarkets * value;

apps/client/src/app/pages/portfolio/allocations/allocations-page.component.ts

@@ -348,8 +348,11 @@ export class GfAllocationsPageComponent implements OnInit {
         name: position.assetProfile.name
       };
 
-      if (position.assetProfile.assetClass !== AssetClass.LIQUIDITY) {
+      if (
-        // Prepare analysis data by continents, countries, holdings and sectors except for liquidity
+        position.assetProfile.assetClass !== AssetClass.LIQUIDITY &&
+        position.assetProfile.assetSubClass !== AssetClass.COMMODITY
+      ) {
+        // Prepare analysis data by continents, countries, holdings and sectors except for liquidity and commodity
 
         if (position.assetProfile.countries.length > 0) {
           for (const country of position.assetProfile.countries) {