Merge branch 'main' into task/migrate-seed-execution

3 months ago · 3866d3a334
10 changed files with 105 additions and 27 deletions
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@ -10,6 +10,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed

 - Extended the import functionality by tags
+- Improved the dynamic numerical precision for various values in the holding detail dialog
+- Shortened the date in the activities table on mobile
 - Introduced the fuzzy search for the accounts endpoint
 - Refactored the fuzzy search for the holdings of the assistant
 - Eliminated the warnings of the database seeding process
--- a/apps/client/src/app/components/holding-detail-dialog/holding-detail-dialog.component.ts
+++ b/apps/client/src/app/components/holding-detail-dialog/holding-detail-dialog.component.ts
@ -3,7 +3,11 @@ import { GfDialogFooterModule } from '@ghostfolio/client/components/dialog-foote
 import { GfDialogHeaderModule } from '@ghostfolio/client/components/dialog-header/dialog-header.module';
 import { DataService } from '@ghostfolio/client/services/data.service';
 import { UserService } from '@ghostfolio/client/services/user/user.service';
-import { NUMERICAL_PRECISION_THRESHOLD } from '@ghostfolio/common/config';
+import {
+  NUMERICAL_PRECISION_THRESHOLD_3_FIGURES,
+  NUMERICAL_PRECISION_THRESHOLD_5_FIGURES,
+  NUMERICAL_PRECISION_THRESHOLD_6_FIGURES
+} from '@ghostfolio/common/config';
 import { DATE_FORMAT, downloadAsFile } from '@ghostfolio/common/helper';
 import {
  DataProviderInfo,
@ -101,6 +105,7 @@ export class GfHoldingDetailDialogComponent implements OnDestroy, OnInit {
  public assetClass: string;
  public assetSubClass: string;
  public averagePrice: number;
+  public averagePricePrecision = 2;
  public benchmarkDataItems: LineChartItem[];
  public benchmarkLabel = $localize`Average Unit Price`;
  public countries: {
@ -122,11 +127,15 @@ export class GfHoldingDetailDialogComponent implements OnDestroy, OnInit {
  public marketDataItems: MarketData[] = [];
  public marketPrice: number;
  public marketPriceMax: number;
+  public marketPriceMaxPrecision = 2;
  public marketPriceMin: number;
+  public marketPriceMinPrecision = 2;
+  public marketPricePrecision = 2;
  public netPerformance: number;
  public netPerformancePrecision = 2;
  public netPerformancePercent: number;
  public netPerformancePercentWithCurrencyEffect: number;
+  public netPerformancePercentWithCurrencyEffectPrecision = 2;
  public netPerformanceWithCurrencyEffect: number;
  public netPerformanceWithCurrencyEffectPrecision = 2;
  public quantity: number;
@ -274,6 +283,14 @@ export class GfHoldingDetailDialogComponent implements OnDestroy, OnInit {
          value
        }) => {
          this.averagePrice = averagePrice;
+
+          if (
+            this.data.deviceType === 'mobile' &&
+            this.averagePrice >= NUMERICAL_PRECISION_THRESHOLD_6_FIGURES
+          ) {
+            this.averagePricePrecision = 0;
+          }
+
          this.benchmarkDataItems = [];
          this.countries = {};
          this.dataProviderInfo = dataProviderInfo;
@ -281,7 +298,8 @@ export class GfHoldingDetailDialogComponent implements OnDestroy, OnInit {

          if (
            this.data.deviceType === 'mobile' &&
-            this.dividendInBaseCurrency >= NUMERICAL_PRECISION_THRESHOLD
+            this.dividendInBaseCurrency >=
+              NUMERICAL_PRECISION_THRESHOLD_6_FIGURES
          ) {
            this.dividendInBaseCurrencyPrecision = 0;
          }
@ -320,19 +338,42 @@ export class GfHoldingDetailDialogComponent implements OnDestroy, OnInit {
          if (
            this.data.deviceType === 'mobile' &&
            this.investmentInBaseCurrencyWithCurrencyEffect >=
-              NUMERICAL_PRECISION_THRESHOLD
+              NUMERICAL_PRECISION_THRESHOLD_6_FIGURES
          ) {
            this.investmentInBaseCurrencyWithCurrencyEffectPrecision = 0;
          }

          this.marketPrice = marketPrice;
          this.marketPriceMax = marketPriceMax;
+
+          if (
+            this.data.deviceType === 'mobile' &&
+            this.marketPriceMax >= NUMERICAL_PRECISION_THRESHOLD_6_FIGURES
+          ) {
+            this.marketPriceMaxPrecision = 0;
+          }
+
          this.marketPriceMin = marketPriceMin;
+
+          if (
+            this.data.deviceType === 'mobile' &&
+            this.marketPriceMin >= NUMERICAL_PRECISION_THRESHOLD_6_FIGURES
+          ) {
+            this.marketPriceMinPrecision = 0;
+          }
+
+          if (
+            this.data.deviceType === 'mobile' &&
+            this.marketPrice >= NUMERICAL_PRECISION_THRESHOLD_6_FIGURES
+          ) {
+            this.marketPricePrecision = 0;
+          }
+
          this.netPerformance = netPerformance;

          if (
            this.data.deviceType === 'mobile' &&
-            this.netPerformance >= NUMERICAL_PRECISION_THRESHOLD
+            this.netPerformance >= NUMERICAL_PRECISION_THRESHOLD_6_FIGURES
          ) {
            this.netPerformancePrecision = 0;
          }
@ -342,13 +383,21 @@ export class GfHoldingDetailDialogComponent implements OnDestroy, OnInit {
          this.netPerformancePercentWithCurrencyEffect =
            netPerformancePercentWithCurrencyEffect;

+          if (
+            this.data.deviceType === 'mobile' &&
+            this.netPerformancePercentWithCurrencyEffect >=
+              NUMERICAL_PRECISION_THRESHOLD_3_FIGURES
+          ) {
+            this.netPerformancePercentWithCurrencyEffectPrecision = 0;
+          }
+
          this.netPerformanceWithCurrencyEffect =
            netPerformanceWithCurrencyEffect;

          if (
            this.data.deviceType === 'mobile' &&
            this.netPerformanceWithCurrencyEffect >=
-              NUMERICAL_PRECISION_THRESHOLD
+              NUMERICAL_PRECISION_THRESHOLD_5_FIGURES
          ) {
            this.netPerformanceWithCurrencyEffectPrecision = 0;
          }
--- a/apps/client/src/app/components/holding-detail-dialog/holding-detail-dialog.html
+++ b/apps/client/src/app/components/holding-detail-dialog/holding-detail-dialog.html
@ -77,6 +77,7 @@
                [colorizeSign]="true"
                [isPercent]="true"
                [locale]="data.locale"
+                [precision]="netPerformancePercentWithCurrencyEffectPrecision"
                [value]="netPerformancePercentWithCurrencyEffect"
              >
                @if (
@ -95,6 +96,7 @@
                size="medium"
                [isCurrency]="true"
                [locale]="data.locale"
+                [precision]="averagePricePrecision"
                [unit]="SymbolProfile?.currency"
                [value]="averagePrice"
                >Average Unit Price</gf-value
@ -106,6 +108,7 @@
                size="medium"
                [isCurrency]="true"
                [locale]="data.locale"
+                [precision]="marketPricePrecision"
                [unit]="SymbolProfile?.currency"
                [value]="marketPrice"
                >Market Price</gf-value
@ -122,6 +125,7 @@
                    marketPriceMin?.toFixed(2) === marketPrice?.toFixed(2) &&
                    marketPriceMax?.toFixed(2) !== marketPriceMin?.toFixed(2)
                }"
+                [precision]="marketPriceMinPrecision"
                [unit]="SymbolProfile?.currency"
                [value]="marketPriceMin"
                >Minimum Price</gf-value
@ -138,6 +142,7 @@
                    marketPriceMax?.toFixed(2) === marketPrice?.toFixed(2) &&
                    marketPriceMax?.toFixed(2) !== marketPriceMin?.toFixed(2)
                }"
+                [precision]="marketPriceMaxPrecision"
                [unit]="SymbolProfile?.currency"
                [value]="marketPriceMax"
                >Maximum Price</gf-value
@ -208,6 +213,7 @@
              <gf-value
                i18n
                size="medium"
+                [deviceType]="data.deviceType"
                [isDate]="true"
                [locale]="data.locale"
                [value]="firstBuyDate"
--- a/apps/client/src/app/components/home-overview/home-overview.component.ts
+++ b/apps/client/src/app/components/home-overview/home-overview.component.ts
@ -3,7 +3,7 @@ import { LayoutService } from '@ghostfolio/client/core/layout.service';
 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 { NUMERICAL_PRECISION_THRESHOLD } from '@ghostfolio/common/config';
+import { NUMERICAL_PRECISION_THRESHOLD_6_FIGURES } from '@ghostfolio/common/config';
 import {
  AssetProfileIdentifier,
  LineChartItem,
@ -143,7 +143,7 @@ export class GfHomeOverviewComponent implements OnDestroy, OnInit {
        if (
          this.deviceType === 'mobile' &&
          this.performance.currentValueInBaseCurrency >=
-            NUMERICAL_PRECISION_THRESHOLD
+            NUMERICAL_PRECISION_THRESHOLD_6_FIGURES
        ) {
          this.precision = 0;
        }
--- a/apps/client/src/locales/messages.pl.xlf
+++ b/apps/client/src/locales/messages.pl.xlf
@ -7660,7 +7660,7 @@
      </trans-unit>
      <trans-unit id="rule.currencyClusterRiskCurrentInvestment" datatype="html">
        <source>Investment</source>
-        <target state="new">Investment</target>
+        <target state="translated">Inwestycja</target>
        <context-group purpose="location">
          <context context-type="sourcefile">apps/client/src/app/pages/i18n/i18n-page.html</context>
          <context context-type="linenumber">82</context>
@ -7668,7 +7668,7 @@
      </trans-unit>
      <trans-unit id="rule.currencyClusterRiskCurrentInvestment.false" datatype="html">
        <source> Over ${thresholdMax}% of your current investment is in ${currency} (${maxValueRatio}%) </source>
-        <target state="new"> Over ${thresholdMax}% of your current investment is in ${currency} (${maxValueRatio}%) </target>
+        <target state="translated">Ponad ${thresholdMax}% Twojej obecnej inwestycji znajduje się w ${currency} (${maxValueRatio}%)</target>
        <context-group purpose="location">
          <context context-type="sourcefile">apps/client/src/app/pages/i18n/i18n-page.html</context>
          <context context-type="linenumber">84</context>
@ -7676,7 +7676,7 @@
      </trans-unit>
      <trans-unit id="rule.currencyClusterRiskCurrentInvestment.true" datatype="html">
        <source> The major part of your current investment is in ${currency} (${maxValueRatio}%) and does not exceed ${thresholdMax}% </source>
-        <target state="new"> The major part of your current investment is in ${currency} (${maxValueRatio}%) and does not exceed ${thresholdMax}% </target>
+        <target state="translated">Główna część Twojej obecnej inwestycji znajduje się w ${currency} (${maxValueRatio}%) i nie przekracza ${thresholdMax}%</target>
        <context-group purpose="location">
          <context context-type="sourcefile">apps/client/src/app/pages/i18n/i18n-page.html</context>
          <context context-type="linenumber">88</context>
@ -7684,7 +7684,7 @@
      </trans-unit>
      <trans-unit id="routes.start" datatype="html">
        <source>start</source>
-        <target state="new">start</target>
+        <target state="translated">rozpocznij</target>
        <note priority="1" from="description">kebab-case</note>
        <context-group purpose="location">
          <context context-type="sourcefile">libs/common/src/lib/routes/routes.ts</context>
@ -7697,7 +7697,7 @@
      </trans-unit>
      <trans-unit id="5608465303990699628" datatype="html">
        <source>Do you really want to generate a new security token?</source>
-        <target state="new">Do you really want to generate a new security token?</target>
+        <target state="translated">Czy na pewno chcesz wygenerować nowy token bezpieczeństwa?</target>
        <context-group purpose="location">
          <context context-type="sourcefile">apps/client/src/app/components/user-account-access/user-account-access.component.ts</context>
          <context context-type="linenumber">174</context>
@ -7705,7 +7705,7 @@
      </trans-unit>
      <trans-unit id="5193539160604294602" datatype="html">
        <source>Generate</source>
-        <target state="new">Generate</target>
+        <target state="translated">Generuj</target>
        <context-group purpose="location">
          <context context-type="sourcefile">apps/client/src/app/components/user-account-access/user-account-access.html</context>
          <context context-type="linenumber">43</context>
@ -7713,7 +7713,7 @@
      </trans-unit>
      <trans-unit id="2732382861635368484" datatype="html">
        <source>Stocks</source>
-        <target state="new">Stocks</target>
+        <target state="translated">Akcje</target>
        <context-group purpose="location">
          <context context-type="sourcefile">apps/client/src/app/components/markets/markets.component.ts</context>
          <context context-type="linenumber">52</context>
@ -7725,7 +7725,7 @@
      </trans-unit>
      <trans-unit id="1419479195323304896" datatype="html">
        <source>Cryptocurrencies</source>
-        <target state="new">Cryptocurrencies</target>
+        <target state="translated">Kryptowaluty</target>
        <context-group purpose="location">
          <context context-type="sourcefile">apps/client/src/app/components/markets/markets.component.ts</context>
          <context context-type="linenumber">53</context>
@ -7745,7 +7745,7 @@
      </trans-unit>
      <trans-unit id="4022507644751907059" datatype="html">
        <source>Manage Asset Profile</source>
-        <target state="new">Manage Asset Profile</target>
+        <target state="translated">Zarządzaj profilem aktywów</target>
        <context-group purpose="location">
          <context context-type="sourcefile">apps/client/src/app/components/holding-detail-dialog/holding-detail-dialog.html</context>
          <context context-type="linenumber">437</context>
@ -7753,7 +7753,7 @@
      </trans-unit>
      <trans-unit id="2978009302056542263" datatype="html">
        <source>Alternative Investment</source>
-        <target state="new">Alternative Investment</target>
+        <target state="translated">Inwestycja alternatywna</target>
        <context-group purpose="location">
          <context context-type="sourcefile">libs/ui/src/lib/i18n.ts</context>
          <context context-type="linenumber">45</context>
@ -7761,7 +7761,7 @@
      </trans-unit>
      <trans-unit id="5795124554973640871" datatype="html">
        <source>Collectible</source>
-        <target state="new">Collectible</target>
+        <target state="translated">Kolekcjonerskie</target>
        <context-group purpose="location">
          <context context-type="sourcefile">libs/ui/src/lib/i18n.ts</context>
          <context context-type="linenumber">55</context>
@ -7769,7 +7769,7 @@
      </trans-unit>
      <trans-unit id="5053948923184155554" datatype="html">
        <source>Average Unit Price</source>
-        <target state="new">Average Unit Price</target>
+        <target state="translated">Średnia cena jednostkowa</target>
        <context-group purpose="location">
          <context context-type="sourcefile">apps/client/src/app/components/holding-detail-dialog/holding-detail-dialog.component.ts</context>
          <context context-type="linenumber">105</context>
--- a/libs/common/src/lib/config.ts
+++ b/libs/common/src/lib/config.ts
@ -136,7 +136,9 @@ export const HEADER_KEY_SKIP_INTERCEPTOR = 'X-Skip-Interceptor';

 export const MAX_TOP_HOLDINGS = 50;

-export const NUMERICAL_PRECISION_THRESHOLD = 100000;
+export const NUMERICAL_PRECISION_THRESHOLD_3_FIGURES = 100;
+export const NUMERICAL_PRECISION_THRESHOLD_5_FIGURES = 10000;
+export const NUMERICAL_PRECISION_THRESHOLD_6_FIGURES = 100000;

 export const PROPERTY_API_KEY_GHOSTFOLIO = 'API_KEY_GHOSTFOLIO';
 export const PROPERTY_API_KEY_OPENROUTER = 'API_KEY_OPENROUTER';
--- a/libs/ui/src/lib/activities-table/activities-table.component.html
+++ b/libs/ui/src/lib/activities-table/activities-table.component.html
@ -171,7 +171,12 @@
      </th>
      <td *matCellDef="let element" class="px-1" mat-cell>
        <div class="d-flex">
-          {{ element.date | date: defaultDateFormat }}
+          <gf-value
+            [deviceType]="deviceType"
+            [isDate]="true"
+            [locale]="locale"
+            [value]="isLoading ? undefined : element.date"
+          />
        </div>
      </td>
    </ng-container>
--- a/libs/ui/src/lib/activities-table/activities-table.component.ts
+++ b/libs/ui/src/lib/activities-table/activities-table.component.ts
@ -3,7 +3,7 @@ import { ConfirmationDialogType } from '@ghostfolio/client/core/notification/con
 import { NotificationService } from '@ghostfolio/client/core/notification/notification.service';
 import { GfSymbolModule } from '@ghostfolio/client/pipes/symbol/symbol.module';
 import { DEFAULT_PAGE_SIZE } from '@ghostfolio/common/config';
-import { getDateFormatString, getLocale } from '@ghostfolio/common/helper';
+import { getLocale } from '@ghostfolio/common/helper';
 import { AssetProfileIdentifier } from '@ghostfolio/common/interfaces';
 import { TAG_ID_EXCLUDE_FROM_ANALYSIS } from '@ghostfolio/common/tag-ids.mjs';
 import { OrderWithAccount } from '@ghostfolio/common/types';
@ -126,7 +126,6 @@ export class GfActivitiesTableComponent
  @ViewChild(MatPaginator) paginator: MatPaginator;
  @ViewChild(MatSort) sort: MatSort;

-  public defaultDateFormat: string;
  public displayedColumns = [];
  public endOfToday = endOfToday();
  public hasDrafts = false;
@ -173,8 +172,6 @@ export class GfActivitiesTableComponent
  }

  public ngOnChanges() {
-    this.defaultDateFormat = getDateFormatString(this.locale);
-
    this.displayedColumns = [
      'select',
      'importStatus',
--- a/libs/ui/src/lib/value/value.component.stories.ts
+++ b/libs/ui/src/lib/value/value.component.stories.ts
@ -11,7 +11,13 @@ export default {
    moduleMetadata({
      imports: [NgxSkeletonLoaderModule]
    })
-  ]
+  ],
+  argTypes: {
+    deviceType: {
+      control: 'select',
+      options: ['desktop', 'mobile']
+    }
+  }
 } as Meta<GfValueComponent>;

 type Story = StoryObj<GfValueComponent>;
@ -31,6 +37,16 @@ export const Currency: Story = {
  }
 };

+export const DateValue: Story = {
+  args: {
+    deviceType: 'desktop',
+    isDate: true,
+    locale: 'en-US',
+    value: new Date().toISOString()
+  },
+  name: 'Date'
+};
+
 export const Label: Story = {
  args: {
    locale: 'en-US',
--- a/libs/ui/src/lib/value/value.component.ts
+++ b/libs/ui/src/lib/value/value.component.ts
@ -22,6 +22,7 @@ import { NgxSkeletonLoaderModule } from 'ngx-skeleton-loader';
 })
 export class GfValueComponent implements OnChanges {
  @Input() colorizeSign = false;
+  @Input() deviceType: string;
  @Input() icon = '';
  @Input() isAbsolute = false;
  @Input() isCurrency = false;
@ -118,7 +119,7 @@ export class GfValueComponent implements OnChanges {
            {
              day: '2-digit',
              month: '2-digit',
-              year: 'numeric'
+              year: this.deviceType === 'mobile' ? '2-digit' : 'numeric'
            }
          );
        } else {