Task/improve type safety in pricing page component (#6806)

* fix(client): resolve type errors

* feat(client): enforce immutability

* fix(client): remove unused variables

* fix(client): enforce encapsulation

* feat(client): replace constructor based DI with inject

apps/client/src/app/pages/pricing/pricing-page.component.ts

@@ -13,6 +13,7 @@ import {
   Component,
   CUSTOM_ELEMENTS_SCHEMA,
   DestroyRef,
+  inject,
   OnInit
 } from '@angular/core';
 import { takeUntilDestroyed } from '@angular/core/rxjs-interop';
@@ -48,35 +49,28 @@ import { catchError } from 'rxjs/operators';
   templateUrl: './pricing-page.html'
 })
 export class GfPricingPageComponent implements OnInit {
-  public baseCurrency: string;
+  protected baseCurrency: string;
-  public coupon: number;
+  protected coupon: number | undefined;
-  public couponId: string;
+  protected durationExtension: StringValue | undefined;
-  public durationExtension: StringValue;
+  protected hasPermissionToCreateUser: boolean;
-  public hasPermissionToCreateUser: boolean;
+  protected hasPermissionToUpdateUserSettings: boolean;
-  public hasPermissionToUpdateUserSettings: boolean;
+
-
+  protected readonly importAndExportTooltipBasic = translate(
-  public importAndExportTooltipBasic = translate(
     'DATA_IMPORT_AND_EXPORT_TOOLTIP_BASIC'
   );
 
-  public importAndExportTooltipOSS = translate(
+  protected readonly importAndExportTooltipOSS = translate(
     'DATA_IMPORT_AND_EXPORT_TOOLTIP_OSS'
   );
 
-  public importAndExportTooltipPremium = translate(
+  protected label: string | undefined;
-    'DATA_IMPORT_AND_EXPORT_TOOLTIP_PREMIUM'
+  protected price: number | undefined;
-  );
-
-  public isLoggedIn: boolean;
-  public label: string;
-  public price: number;
-  public priceId: string;
 
-  public professionalDataProviderTooltipPremium = translate(
+  protected readonly professionalDataProviderTooltipPremium = translate(
     'PROFESSIONAL_DATA_PROVIDER_TOOLTIP_PREMIUM'
   );
 
-  public referralBrokers = [
+  protected readonly referralBrokers = [
     'Alpian',
     'DEGIRO',
     'finpension',
@@ -87,19 +81,22 @@ export class GfPricingPageComponent implements OnInit {
     'Swissquote',
     'VIAC',
     'Zak'
-  ];
+  ] as const;
-
+
-  public routerLinkFeatures = publicRoutes.features.routerLink;
+  protected readonly routerLinkFeatures = publicRoutes.features.routerLink;
-  public routerLinkRegister = publicRoutes.register.routerLink;
+  protected readonly routerLinkRegister = publicRoutes.register.routerLink;
-  public user: User;
+  protected user: User;
-
+
-  public constructor(
+  private couponId: string | undefined;
-    private changeDetectorRef: ChangeDetectorRef,
+  private priceId: string;
-    private dataService: DataService,
+
-    private destroyRef: DestroyRef,
+  private readonly changeDetectorRef = inject(ChangeDetectorRef);
-    private notificationService: NotificationService,
+  private readonly dataService = inject(DataService);
-    private userService: UserService
+  private readonly destroyRef = inject(DestroyRef);
-  ) {
+  private readonly notificationService = inject(NotificationService);
+  private readonly userService = inject(UserService);
+
+  public constructor() {
     addIcons({
       checkmarkCircleOutline,
       checkmarkOutline,
@@ -147,7 +144,7 @@ export class GfPricingPageComponent implements OnInit {
       });
   }
 
-  public onCheckout() {
+  protected onCheckout() {
     this.dataService
       .createStripeCheckoutSession({
         couponId: this.couponId,

libs/ui/src/lib/i18n.ts

@@ -11,7 +11,6 @@ const locales = {
   CLOSE: $localize`Close`,
   DATA_IMPORT_AND_EXPORT_TOOLTIP_BASIC: $localize`Switch to Ghostfolio Premium or Ghostfolio Open Source easily`,
   DATA_IMPORT_AND_EXPORT_TOOLTIP_OSS: $localize`Switch to Ghostfolio Premium easily`,
-  DATA_IMPORT_AND_EXPORT_TOOLTIP_PREMIUM: $localize`Switch to Ghostfolio Open Source or Ghostfolio Basic easily`,
   DATA_SOURCE: $localize`Data Source`,
   EMERGENCY_FUND: $localize`Emergency Fund`,
   EXCLUDE_FROM_ANALYSIS: $localize`Exclude from Analysis`,