Browse Source

Refactoring

pull/2958/head
Thomas Kaul 2 years ago
parent
commit
df396b5194
  1. 3
      apps/api/src/app/import/import.controller.ts
  2. 7
      apps/api/src/app/import/import.service.ts
  3. 14
      apps/api/src/services/data-provider/alpha-vantage/alpha-vantage.service.ts

3
apps/api/src/app/import/import.controller.ts

@ -64,13 +64,10 @@ export class ImportController {
maxActivitiesToImport = Number.MAX_SAFE_INTEGER; maxActivitiesToImport = Number.MAX_SAFE_INTEGER;
} }
const userCurrency = this.request.user.Settings.settings.baseCurrency;
try { try {
const activities = await this.importService.import({ const activities = await this.importService.import({
isDryRun, isDryRun,
maxActivitiesToImport, maxActivitiesToImport,
userCurrency,
accountsDto: importData.accounts ?? [], accountsDto: importData.accounts ?? [],
activitiesDto: importData.activities, activitiesDto: importData.activities,
user: this.request.user user: this.request.user

7
apps/api/src/app/import/import.service.ts

@ -139,17 +139,16 @@ export class ImportService {
activitiesDto, activitiesDto,
isDryRun = false, isDryRun = false,
maxActivitiesToImport, maxActivitiesToImport,
userCurrency,
user user
}: { }: {
accountsDto: Partial<CreateAccountDto>[]; accountsDto: Partial<CreateAccountDto>[];
activitiesDto: Partial<CreateOrderDto>[]; activitiesDto: Partial<CreateOrderDto>[];
isDryRun?: boolean; isDryRun?: boolean;
maxActivitiesToImport: number; maxActivitiesToImport: number;
userCurrency: string;
user: UserWithSettings; user: UserWithSettings;
}): Promise<Activity[]> { }): Promise<Activity[]> {
const accountIdMapping: { [oldAccountId: string]: string } = {}; const accountIdMapping: { [oldAccountId: string]: string } = {};
const userCurrency = user.Settings.settings.baseCurrency;
if (!isDryRun && accountsDto?.length) { if (!isDryRun && accountsDto?.length) {
const [existingAccounts, existingPlatforms] = await Promise.all([ const [existingAccounts, existingPlatforms] = await Promise.all([
@ -406,9 +405,9 @@ export class ImportService {
} }
} }
}, },
userId: user.id,
updateAccountBalance: false, updateAccountBalance: false,
User: { connect: { id: user.id } } User: { connect: { id: user.id } },
userId: user.id
}); });
} }

14
apps/api/src/services/data-provider/alpha-vantage/alpha-vantage.service.ts

@ -36,12 +36,6 @@ export class AlphaVantageService implements DataProviderInterface {
return !!this.configurationService.get('ALPHA_VANTAGE_API_KEY'); return !!this.configurationService.get('ALPHA_VANTAGE_API_KEY');
} }
public getDataProviderInfo(): DataProviderInfo {
return {
isPremium: false
};
}
public async getAssetProfile( public async getAssetProfile(
aSymbol: string aSymbol: string
): Promise<Partial<SymbolProfile>> { ): Promise<Partial<SymbolProfile>> {
@ -51,6 +45,12 @@ export class AlphaVantageService implements DataProviderInterface {
}; };
} }
public getDataProviderInfo(): DataProviderInfo {
return {
isPremium: false
};
}
public async getDividends({}: GetDividendsParams) { public async getDividends({}: GetDividendsParams) {
return {}; return {};
} }
@ -121,7 +121,7 @@ export class AlphaVantageService implements DataProviderInterface {
return { return {
items: result?.bestMatches?.map((bestMatch) => { items: result?.bestMatches?.map((bestMatch) => {
return <LookupItem>{ return {
assetClass: undefined, assetClass: undefined,
assetSubClass: undefined, assetSubClass: undefined,
currency: bestMatch['8. currency'], currency: bestMatch['8. currency'],

Loading…
Cancel
Save