Browse Source

implements code review

pull/5169/head
tobikugel 1 month ago
committed by Thomas Kaul
parent
commit
6b52d88770
  1. 9
      apps/api/src/models/rules/economic-market-cluster-risk/developed-markets.ts
  2. 10
      apps/api/src/models/rules/economic-market-cluster-risk/emerging-markets.ts

9
apps/api/src/models/rules/economic-market-cluster-risk/developed-markets.ts

@ -16,8 +16,8 @@ export class EconomicMarketClusterRiskDevelopedMarkets extends Rule<Settings> {
languageCode: string languageCode: string
) { ) {
super(exchangeRateDataService, { super(exchangeRateDataService, {
key: EconomicMarketClusterRiskDevelopedMarkets.name, languageCode,
languageCode key: EconomicMarketClusterRiskDevelopedMarkets.name
}); });
this.currentValueInBaseCurrency = currentValueInBaseCurrency; this.currentValueInBaseCurrency = currentValueInBaseCurrency;
@ -78,7 +78,10 @@ export class EconomicMarketClusterRiskDevelopedMarkets extends Rule<Settings> {
} }
public getCategoryName() { public getCategoryName() {
return 'Economic Market Cluster Risk'; // TODO: Replace hardcoded text with i18n translation return this.i18nService.getTranslation({
id: 'rule.economicMarketClusterRisk.category',
languageCode: this.getLanguageCode()
});
} }
public getConfiguration() { public getConfiguration() {

10
apps/api/src/models/rules/economic-market-cluster-risk/emerging-markets.ts

@ -16,8 +16,8 @@ export class EconomicMarketClusterRiskEmergingMarkets extends Rule<Settings> {
languageCode: string languageCode: string
) { ) {
super(exchangeRateDataService, { super(exchangeRateDataService, {
key: EconomicMarketClusterRiskEmergingMarkets.name, languageCode,
languageCode key: EconomicMarketClusterRiskEmergingMarkets.name
}); });
this.currentValueInBaseCurrency = currentValueInBaseCurrency; this.currentValueInBaseCurrency = currentValueInBaseCurrency;
@ -60,6 +60,7 @@ export class EconomicMarketClusterRiskEmergingMarkets extends Rule<Settings> {
value: false value: false
}; };
} }
return { return {
evaluation: this.i18nService.getTranslation({ evaluation: this.i18nService.getTranslation({
id: 'rule.economicMarketClusterRiskEmergingMarkets.true', id: 'rule.economicMarketClusterRiskEmergingMarkets.true',
@ -77,7 +78,10 @@ export class EconomicMarketClusterRiskEmergingMarkets extends Rule<Settings> {
} }
public getCategoryName() { public getCategoryName() {
return 'Economic Market Cluster Risk'; // TODO: Replace hardcoded text with i18n translation return this.i18nService.getTranslation({
id: 'rule.economicMarketClusterRisk.category',
languageCode: this.getLanguageCode()
});
} }
public getConfiguration() { public getConfiguration() {

Loading…
Cancel
Save