Browse Source

Bugfix/do not remove countries and sectors in yahoo finance data enhancer (#2297)

* Do not remove countries and sectors

* Update changelog
pull/2293/head^2
Thomas Kaul 1 year ago
committed by GitHub
parent
commit
0f8bc7db32
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 4
      CHANGELOG.md
  2. 5
      apps/api/src/services/data-provider/data-enhancer/yahoo-finance/yahoo-finance.service.ts

4
CHANGELOG.md

@ -18,6 +18,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Refreshed the cryptocurrencies list - Refreshed the cryptocurrencies list
- Changed the version in the `docker-compose` files from `3.7` to `3.9` - Changed the version in the `docker-compose` files from `3.7` to `3.9`
### Fixed
- Fixed an issue in the _Yahoo Finance_ data enhancer where countries and sectors have been removed
## 1.305.0 - 2023-09-03 ## 1.305.0 - 2023-09-03
### Added ### Added

5
apps/api/src/services/data-provider/data-enhancer/yahoo-finance/yahoo-finance.service.ts

@ -8,6 +8,7 @@ import {
AssetClass, AssetClass,
AssetSubClass, AssetSubClass,
DataSource, DataSource,
Prisma,
SymbolProfile SymbolProfile
} from '@prisma/client'; } from '@prisma/client';
import { countries } from 'countries-list'; import { countries } from 'countries-list';
@ -102,11 +103,11 @@ export class YahooFinanceDataEnhancerService implements DataEnhancerInterface {
const { countries, sectors, url } = const { countries, sectors, url } =
await this.getAssetProfile(yahooSymbol); await this.getAssetProfile(yahooSymbol);
if (countries) { if ((countries as unknown as Prisma.JsonArray)?.length > 0) {
response.countries = countries; response.countries = countries;
} }
if (sectors) { if ((sectors as unknown as Prisma.JsonArray)?.length > 0) {
response.sectors = sectors; response.sectors = sectors;
} }

Loading…
Cancel
Save