diff --git a/CHANGELOG.md b/CHANGELOG.md index 7ebec12b7..93142484a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -9,6 +9,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ### Changed +- Improved the allocations by ETF holding on the allocations page (experimental) +- Improved the language localization for German (`de`) - Upgraded `prisma` from version `5.14.0` to `5.15.0` ## 2.86.0 - 2024-06-07 diff --git a/apps/api/src/app/portfolio/portfolio.controller.ts b/apps/api/src/app/portfolio/portfolio.controller.ts index 5cdaa1641..19f48fa86 100644 --- a/apps/api/src/app/portfolio/portfolio.controller.ts +++ b/apps/api/src/app/portfolio/portfolio.controller.ts @@ -204,6 +204,7 @@ export class PortfolioController { : undefined, countries: hasDetails ? portfolioPosition.countries : [], currency: hasDetails ? portfolioPosition.currency : undefined, + holdings: hasDetails ? portfolioPosition.holdings : [], markets: hasDetails ? portfolioPosition.markets : undefined, marketsAdvanced: hasDetails ? portfolioPosition.marketsAdvanced diff --git a/apps/api/src/services/data-provider/data-enhancer/trackinsight/trackinsight.service.ts b/apps/api/src/services/data-provider/data-enhancer/trackinsight/trackinsight.service.ts index 284caed4f..0e12b8f02 100644 --- a/apps/api/src/services/data-provider/data-enhancer/trackinsight/trackinsight.service.ts +++ b/apps/api/src/services/data-provider/data-enhancer/trackinsight/trackinsight.service.ts @@ -163,6 +163,10 @@ export class TrackinsightDataEnhancerService implements DataEnhancerInterface { response.holdings = []; for (const { label, weight } of holdings?.topHoldings ?? []) { + if (label?.toLowerCase() === 'other') { + continue; + } + response.holdings.push({ weight, name: label diff --git a/apps/client/src/app/pages/portfolio/allocations/allocations-page.component.ts b/apps/client/src/app/pages/portfolio/allocations/allocations-page.component.ts index f8ad447de..18c839d10 100644 --- a/apps/client/src/app/pages/portfolio/allocations/allocations-page.component.ts +++ b/apps/client/src/app/pages/portfolio/allocations/allocations-page.component.ts @@ -3,7 +3,7 @@ import { AccountDetailDialogParams } from '@ghostfolio/client/components/account import { DataService } from '@ghostfolio/client/services/data.service'; import { ImpersonationStorageService } from '@ghostfolio/client/services/impersonation-storage.service'; import { UserService } from '@ghostfolio/client/services/user/user.service'; -import { UNKNOWN_KEY } from '@ghostfolio/common/config'; +import { MAX_TOP_HOLDINGS, UNKNOWN_KEY } from '@ghostfolio/common/config'; import { prettifySymbol } from '@ghostfolio/common/helper'; import { Holding, @@ -85,7 +85,7 @@ export class AllocationsPageComponent implements OnDestroy, OnInit { value: number; }; }; - public topHoldings: Holding[] = []; + public topHoldings: Holding[]; public topHoldingsMap: { [name: string]: { name: string; value: number }; }; @@ -456,21 +456,28 @@ export class AllocationsPageComponent implements OnDestroy, OnInit { for (const holding of position.holdings) { const { name, valueInBaseCurrency } = holding; - if (this.topHoldingsMap[name]?.value) { - this.topHoldingsMap[name].value += - valueInBaseCurrency * - (isNumber(position.valueInBaseCurrency) - ? position.valueInBaseCurrency - : position.valueInPercentage); - } else { - this.topHoldingsMap[name] = { - name, - value: + if ( + !this.hasImpersonationId && + !this.user.settings.isRestrictedView + ) { + if (this.topHoldingsMap[name]?.value) { + this.topHoldingsMap[name].value += valueInBaseCurrency * (isNumber(position.valueInBaseCurrency) - ? this.portfolioDetails.holdings[symbol].valueInBaseCurrency - : this.portfolioDetails.holdings[symbol].valueInPercentage) - }; + ? position.valueInBaseCurrency + : position.valueInPercentage); + } else { + this.topHoldingsMap[name] = { + name, + value: + valueInBaseCurrency * + (isNumber(position.valueInBaseCurrency) + ? this.portfolioDetails.holdings[symbol] + .valueInBaseCurrency + : this.portfolioDetails.holdings[symbol] + .valueInPercentage) + }; + } } } } @@ -505,11 +512,7 @@ export class AllocationsPageComponent implements OnDestroy, OnInit { } } - if ( - this.positions[symbol].assetSubClass === 'ETF' && - !this.hasImpersonationId && - !this.user.settings.isRestrictedView - ) { + if (this.positions[symbol].assetSubClass === 'ETF') { this.totalValueInEtf += this.positions[symbol].value; } @@ -557,19 +560,21 @@ export class AllocationsPageComponent implements OnDestroy, OnInit { this.markets[UNKNOWN_KEY].value = this.markets[UNKNOWN_KEY].value / marketsTotal; - if (!this.hasImpersonationId && !this.user.settings.isRestrictedView) { - this.topHoldings = Object.values(this.topHoldingsMap) - .map(({ name, value }) => { - return { - name, - allocationInPercentage: - this.totalValueInEtf > 0 ? value / this.totalValueInEtf : 0, - valueInBaseCurrency: value - }; - }) - .sort((a, b) => { - return b.valueInBaseCurrency - a.valueInBaseCurrency; - }); + this.topHoldings = Object.values(this.topHoldingsMap) + .map(({ name, value }) => { + return { + name, + allocationInPercentage: + this.totalValueInEtf > 0 ? value / this.totalValueInEtf : 0, + valueInBaseCurrency: value + }; + }) + .sort((a, b) => { + return b.valueInBaseCurrency - a.valueInBaseCurrency; + }); + + if (this.topHoldings.length > MAX_TOP_HOLDINGS) { + this.topHoldings = this.topHoldings.slice(0, MAX_TOP_HOLDINGS); } } diff --git a/apps/client/src/app/pages/portfolio/allocations/allocations-page.html b/apps/client/src/app/pages/portfolio/allocations/allocations-page.html index 3a464b2f7..bf26e1579 100644 --- a/apps/client/src/app/pages/portfolio/allocations/allocations-page.html +++ b/apps/client/src/app/pages/portfolio/allocations/allocations-page.html @@ -264,6 +264,30 @@
+
+ + + By Country + + + + + + +
@@ -306,57 +330,36 @@
-
+
By CountryBy ETF Holding + + + Approximation based on the Top 15 holdings per ETF + -
- @if (topHoldings?.length > 0 && user?.settings?.isExperimentalFeatures) { -
- - - By ETF Holding - - - - Approximation based on the Top 15 holdings per - ETF - - - - - - -
- }
diff --git a/apps/client/src/locales/messages.de.xlf b/apps/client/src/locales/messages.de.xlf index 7864ad19c..648a966d7 100644 --- a/apps/client/src/locales/messages.de.xlf +++ b/apps/client/src/locales/messages.de.xlf @@ -164,6 +164,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -232,6 +236,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -1576,6 +1584,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Performance @@ -1600,6 +1612,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Today @@ -2238,7 +2254,7 @@ Nach Konto apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 @@ -2286,7 +2302,7 @@ Nach Land apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -4130,7 +4146,7 @@ Nach ETF-Anbieter apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 @@ -10628,6 +10644,10 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + Ready to take control of your personal finances? @@ -16129,6 +16149,22 @@ 558 + + By ETF Holding + Nach ETF Position + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + + + Approximation based on the Top 15 holdings per ETF + Annäherung auf Basis der 15 grössten Positionen pro ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + diff --git a/apps/client/src/locales/messages.es.xlf b/apps/client/src/locales/messages.es.xlf index 37d8e19d8..3b9123218 100644 --- a/apps/client/src/locales/messages.es.xlf +++ b/apps/client/src/locales/messages.es.xlf @@ -165,6 +165,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -233,6 +237,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -1577,6 +1585,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Performance @@ -1601,6 +1613,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Today @@ -2239,7 +2255,7 @@ Por cuenta apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 @@ -2287,7 +2303,7 @@ Por país apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -4131,7 +4147,7 @@ By ETF Provider apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 @@ -10629,6 +10645,10 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + Ready to take control of your personal finances? @@ -16130,6 +16150,22 @@ 558 + + By ETF Holding + By ETF Holding + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + + + Approximation based on the Top 15 holdings per ETF + Approximation based on the Top 15 holdings per ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + diff --git a/apps/client/src/locales/messages.fr.xlf b/apps/client/src/locales/messages.fr.xlf index 79d878057..0a084f24d 100644 --- a/apps/client/src/locales/messages.fr.xlf +++ b/apps/client/src/locales/messages.fr.xlf @@ -176,6 +176,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -288,6 +292,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -2814,7 +2822,7 @@ Par Compte apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 @@ -2862,7 +2870,7 @@ Par Pays apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -3352,6 +3360,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Show all @@ -3360,6 +3372,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Account @@ -4130,7 +4146,7 @@ Par Émetteur d'ETF apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 @@ -10628,6 +10644,10 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + Ready to take control of your personal finances? @@ -16129,6 +16149,22 @@ 558 + + By ETF Holding + By ETF Holding + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + + + Approximation based on the Top 15 holdings per ETF + Approximation based on the Top 15 holdings per ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + diff --git a/apps/client/src/locales/messages.it.xlf b/apps/client/src/locales/messages.it.xlf index 1ef095070..652e71af8 100644 --- a/apps/client/src/locales/messages.it.xlf +++ b/apps/client/src/locales/messages.it.xlf @@ -165,6 +165,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -233,6 +237,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -1577,6 +1585,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Performance @@ -1601,6 +1613,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Today @@ -2239,7 +2255,7 @@ Per account apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 @@ -2287,7 +2303,7 @@ Per paese apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -4131,7 +4147,7 @@ Per fornitore di ETF apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 @@ -10629,6 +10645,10 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + Ready to take control of your personal finances? @@ -16130,6 +16150,22 @@ 558 + + By ETF Holding + By ETF Holding + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + + + Approximation based on the Top 15 holdings per ETF + Approximation based on the Top 15 holdings per ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + diff --git a/apps/client/src/locales/messages.nl.xlf b/apps/client/src/locales/messages.nl.xlf index 82767ae43..0ba9ce85a 100644 --- a/apps/client/src/locales/messages.nl.xlf +++ b/apps/client/src/locales/messages.nl.xlf @@ -164,6 +164,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -232,6 +236,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -1576,6 +1584,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Performance @@ -1600,6 +1612,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Today @@ -2238,7 +2254,7 @@ Per rekening apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 @@ -2286,7 +2302,7 @@ Per land apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -4130,7 +4146,7 @@ Per ETF-aanbieder apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 @@ -10628,6 +10644,10 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + Ready to take control of your personal finances? @@ -16129,6 +16149,22 @@ 558 + + By ETF Holding + By ETF Holding + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + + + Approximation based on the Top 15 holdings per ETF + Approximation based on the Top 15 holdings per ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + diff --git a/apps/client/src/locales/messages.pl.xlf b/apps/client/src/locales/messages.pl.xlf index 5e7ae3183..f3f382f9e 100644 --- a/apps/client/src/locales/messages.pl.xlf +++ b/apps/client/src/locales/messages.pl.xlf @@ -1804,6 +1804,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -1900,6 +1904,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -5420,13 +5428,17 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + By Account By Account apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 @@ -5434,7 +5446,7 @@ By ETF Provider apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 @@ -5442,7 +5454,7 @@ By Country apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -14248,6 +14260,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Show all @@ -14256,6 +14272,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Account @@ -16129,6 +16149,22 @@ 558 + + By ETF Holding + By ETF Holding + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + + + Approximation based on the Top 15 holdings per ETF + Approximation based on the Top 15 holdings per ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + diff --git a/apps/client/src/locales/messages.pt.xlf b/apps/client/src/locales/messages.pt.xlf index 93c696418..e996ec68e 100644 --- a/apps/client/src/locales/messages.pt.xlf +++ b/apps/client/src/locales/messages.pt.xlf @@ -176,6 +176,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -288,6 +292,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -1904,6 +1912,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Show all @@ -1912,6 +1924,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Today @@ -2714,7 +2730,7 @@ Por Conta apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 @@ -2762,7 +2778,7 @@ Por País apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -4130,7 +4146,7 @@ Por Prestador de ETF apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 @@ -10628,6 +10644,10 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + Ready to take control of your personal finances? @@ -16129,6 +16149,22 @@ 558 + + By ETF Holding + By ETF Holding + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + + + Approximation based on the Top 15 holdings per ETF + Approximation based on the Top 15 holdings per ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + diff --git a/apps/client/src/locales/messages.tr.xlf b/apps/client/src/locales/messages.tr.xlf index 47e052531..0138931b1 100644 --- a/apps/client/src/locales/messages.tr.xlf +++ b/apps/client/src/locales/messages.tr.xlf @@ -1768,6 +1768,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -1864,6 +1868,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -4908,13 +4916,17 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + By Account Hesaba Göre apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 @@ -4922,7 +4934,7 @@ ETF Sağlayıcısına Göre apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 @@ -4930,7 +4942,7 @@ Ülkeye Göre apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -13648,6 +13660,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Show all @@ -13656,6 +13672,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Account @@ -16129,6 +16149,22 @@ 558 + + By ETF Holding + By ETF Holding + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + + + Approximation based on the Top 15 holdings per ETF + Approximation based on the Top 15 holdings per ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + diff --git a/apps/client/src/locales/messages.xlf b/apps/client/src/locales/messages.xlf index a82f9bc3b..deaf94503 100644 --- a/apps/client/src/locales/messages.xlf +++ b/apps/client/src/locales/messages.xlf @@ -1776,6 +1776,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -1869,6 +1873,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -5047,26 +5055,30 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + By Account apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 By ETF Provider apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 By Country apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -14668,6 +14680,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Show all @@ -14675,6 +14691,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Account @@ -15493,6 +15513,20 @@ 243 + + Approximation based on the Top 15 holdings per ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + + + By ETF Holding + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + diff --git a/apps/client/src/locales/messages.zh.xlf b/apps/client/src/locales/messages.zh.xlf index 896088cf6..b68266037 100644 --- a/apps/client/src/locales/messages.zh.xlf +++ b/apps/client/src/locales/messages.zh.xlf @@ -1813,6 +1813,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 28 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 11 + Total @@ -1909,6 +1913,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 74 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 25 + Edit @@ -5437,13 +5445,17 @@ apps/client/src/app/pages/public/public-page.html 123 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 86 + By Account 按帐户 apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 270 + 294 @@ -5451,7 +5463,7 @@ 按 ETF 提供商 apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 290 + 314 @@ -5459,7 +5471,7 @@ 按国家/地区 apps/client/src/app/pages/portfolio/allocations/allocations-page.html - 313 + 271 @@ -15193,6 +15205,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 98 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 45 + Show all @@ -15201,6 +15217,10 @@ libs/ui/src/lib/holdings-table/holdings-table.component.html 197 + + libs/ui/src/lib/top-holdings/top-holdings.component.html + 79 + Account @@ -16130,6 +16150,22 @@ 558 + + By ETF Holding + By ETF Holding + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 342 + + + + Approximation based on the Top 15 holdings per ETF + Approximation based on the Top 15 holdings per ETF + + apps/client/src/app/pages/portfolio/allocations/allocations-page.html + 350 + + diff --git a/libs/common/src/lib/config.ts b/libs/common/src/lib/config.ts index c89143d9d..cd0f207da 100644 --- a/libs/common/src/lib/config.ts +++ b/libs/common/src/lib/config.ts @@ -89,6 +89,7 @@ export const HEADER_KEY_TIMEZONE = 'Timezone'; export const HEADER_KEY_TOKEN = 'Authorization'; export const MAX_CHART_ITEMS = 365; +export const MAX_TOP_HOLDINGS = 50; export const PROPERTY_BENCHMARKS = 'BENCHMARKS'; export const PROPERTY_BETTER_UPTIME_MONITOR_ID = 'BETTER_UPTIME_MONITOR_ID'; diff --git a/libs/ui/src/lib/holdings-table/holdings-table.component.html b/libs/ui/src/lib/holdings-table/holdings-table.component.html index 680154dc9..2aa01580d 100644 --- a/libs/ui/src/lib/holdings-table/holdings-table.component.html +++ b/libs/ui/src/lib/holdings-table/holdings-table.component.html @@ -169,7 +169,7 @@ }" (click)=" !ignoreAssetSubClasses.includes(row.assetSubClass) && - onOpenPositionDialog({ + onOpenHoldingDialog({ dataSource: row.dataSource, symbol: row.symbol }) @@ -193,7 +193,7 @@ @if (dataSource.data.length > pageSize && !isLoading) {
-
diff --git a/libs/ui/src/lib/holdings-table/holdings-table.component.ts b/libs/ui/src/lib/holdings-table/holdings-table.component.ts index d1964b06f..12f071ef1 100644 --- a/libs/ui/src/lib/holdings-table/holdings-table.component.ts +++ b/libs/ui/src/lib/holdings-table/holdings-table.component.ts @@ -102,7 +102,7 @@ export class GfHoldingsTableComponent implements OnChanges, OnDestroy, OnInit { } } - public onOpenPositionDialog({ dataSource, symbol }: UniqueAsset) { + public onOpenHoldingDialog({ dataSource, symbol }: UniqueAsset) { if (this.hasPermissionToOpenDetails) { this.router.navigate([], { queryParams: { dataSource, symbol, holdingDetailDialog: true } @@ -110,7 +110,7 @@ export class GfHoldingsTableComponent implements OnChanges, OnDestroy, OnInit { } } - public onShowAllPositions() { + public onShowAllHoldings() { this.pageSize = Number.MAX_SAFE_INTEGER; setTimeout(() => { diff --git a/libs/ui/src/lib/portfolio-proportion-chart/portfolio-proportion-chart.component.ts b/libs/ui/src/lib/portfolio-proportion-chart/portfolio-proportion-chart.component.ts index f243f888a..aa0a6cacd 100644 --- a/libs/ui/src/lib/portfolio-proportion-chart/portfolio-proportion-chart.component.ts +++ b/libs/ui/src/lib/portfolio-proportion-chart/portfolio-proportion-chart.component.ts @@ -112,7 +112,7 @@ export class GfPortfolioProportionChartComponent this.positions[symbol][this.keys[0]].toUpperCase() ].value = chartData[ this.positions[symbol][this.keys[0]].toUpperCase() - ].value.plus(this.positions[symbol].value); + ].value.plus(this.positions[symbol].value || 0); if ( chartData[this.positions[symbol][this.keys[0]].toUpperCase()] @@ -124,20 +124,20 @@ export class GfPortfolioProportionChartComponent chartData[ this.positions[symbol][this.keys[0]].toUpperCase() ].subCategory[this.positions[symbol][this.keys[1]]].value.plus( - this.positions[symbol].value + this.positions[symbol].value || 0 ); } else { chartData[ this.positions[symbol][this.keys[0]].toUpperCase() ].subCategory[ this.positions[symbol][this.keys[1]] ?? UNKNOWN_KEY - ] = { value: new Big(this.positions[symbol].value) }; + ] = { value: new Big(this.positions[symbol].value || 0) }; } } else { chartData[this.positions[symbol][this.keys[0]].toUpperCase()] = { name: this.positions[symbol][this.keys[0]], subCategory: {}, - value: new Big(this.positions[symbol].value ?? 0) + value: new Big(this.positions[symbol].value || 0) }; if (this.positions[symbol][this.keys[1]]) { @@ -145,7 +145,7 @@ export class GfPortfolioProportionChartComponent this.positions[symbol][this.keys[0]].toUpperCase() ].subCategory = { [this.positions[symbol][this.keys[1]]]: { - value: new Big(this.positions[symbol].value) + value: new Big(this.positions[symbol].value || 0) } }; } @@ -153,7 +153,7 @@ export class GfPortfolioProportionChartComponent } else { if (chartData[UNKNOWN_KEY]) { chartData[UNKNOWN_KEY].value = chartData[UNKNOWN_KEY].value.plus( - this.positions[symbol].value + this.positions[symbol].value || 0 ); } else { chartData[UNKNOWN_KEY] = { @@ -161,7 +161,7 @@ export class GfPortfolioProportionChartComponent subCategory: this.keys[1] ? { [this.keys[1]]: { value: new Big(0) } } : undefined, - value: new Big(this.positions[symbol].value) + value: new Big(this.positions[symbol].value || 0) }; } } @@ -170,7 +170,7 @@ export class GfPortfolioProportionChartComponent Object.keys(this.positions).forEach((symbol) => { chartData[symbol] = { name: this.positions[symbol].name, - value: new Big(this.positions[symbol].value) + value: new Big(this.positions[symbol].value || 0) }; }); } diff --git a/libs/ui/src/lib/top-holdings/top-holdings.component.html b/libs/ui/src/lib/top-holdings/top-holdings.component.html index b3ce98a14..ec95bddcb 100644 --- a/libs/ui/src/lib/top-holdings/top-holdings.component.html +++ b/libs/ui/src/lib/top-holdings/top-holdings.component.html @@ -11,7 +11,7 @@ Name - {{ element?.name }} + {{ element?.name | titlecase }} @@ -59,3 +59,30 @@ + + + +@if (isLoading) { + +} + +@if (dataSource.data.length > pageSize && !isLoading) { +
+ +
+} + +@if (dataSource.data.length === 0 && !isLoading) { +
+ No data available +
+} diff --git a/libs/ui/src/lib/top-holdings/top-holdings.component.ts b/libs/ui/src/lib/top-holdings/top-holdings.component.ts index aa2b85e2e..6f7695687 100644 --- a/libs/ui/src/lib/top-holdings/top-holdings.component.ts +++ b/libs/ui/src/lib/top-holdings/top-holdings.component.ts @@ -2,6 +2,7 @@ import { getLocale } from '@ghostfolio/common/helper'; import { Holding } from '@ghostfolio/common/interfaces'; import { GfValueComponent } from '@ghostfolio/ui/value'; +import { CommonModule } from '@angular/common'; import { CUSTOM_ELEMENTS_SCHEMA, ChangeDetectionStrategy, @@ -13,14 +14,24 @@ import { ViewChild } from '@angular/core'; import { MatButtonModule } from '@angular/material/button'; +import { MatPaginator, MatPaginatorModule } from '@angular/material/paginator'; import { MatSort, MatSortModule } from '@angular/material/sort'; import { MatTableDataSource, MatTableModule } from '@angular/material/table'; import { get } from 'lodash'; +import { NgxSkeletonLoaderModule } from 'ngx-skeleton-loader'; import { Subject } from 'rxjs'; @Component({ changeDetection: ChangeDetectionStrategy.OnPush, - imports: [GfValueComponent, MatButtonModule, MatSortModule, MatTableModule], + imports: [ + CommonModule, + GfValueComponent, + MatButtonModule, + MatPaginatorModule, + MatSortModule, + MatTableModule, + NgxSkeletonLoaderModule + ], schemas: [CUSTOM_ELEMENTS_SCHEMA], selector: 'gf-top-holdings', standalone: true, @@ -30,8 +41,10 @@ import { Subject } from 'rxjs'; export class GfTopHoldingsComponent implements OnChanges, OnDestroy, OnInit { @Input() baseCurrency: string; @Input() locale = getLocale(); + @Input() pageSize = Number.MAX_SAFE_INTEGER; @Input() topHoldings: Holding[]; + @ViewChild(MatPaginator) paginator: MatPaginator; @ViewChild(MatSort) sort: MatSort; public dataSource: MatTableDataSource = new MatTableDataSource(); @@ -40,6 +53,7 @@ export class GfTopHoldingsComponent implements OnChanges, OnDestroy, OnInit { 'valueInBaseCurrency', 'allocationInPercentage' ]; + public isLoading = true; private unsubscribeSubject = new Subject(); @@ -48,14 +62,26 @@ export class GfTopHoldingsComponent implements OnChanges, OnDestroy, OnInit { public ngOnInit() {} public ngOnChanges() { - if (this.topHoldings) { - this.dataSource = new MatTableDataSource(this.topHoldings); + this.isLoading = true; + + this.dataSource = new MatTableDataSource(this.topHoldings); + this.dataSource.paginator = this.paginator; + this.dataSource.sort = this.sort; + this.dataSource.sortingDataAccessor = get; - this.dataSource.sort = this.sort; - this.dataSource.sortingDataAccessor = get; + if (this.topHoldings) { + this.isLoading = false; } } + public onShowAllHoldings() { + this.pageSize = Number.MAX_SAFE_INTEGER; + + setTimeout(() => { + this.dataSource.paginator = this.paginator; + }); + } + public ngOnDestroy() { this.unsubscribeSubject.next(); this.unsubscribeSubject.complete();