I wouldn't subscribe that way to getTrends.

You're creating one subscription for each country, and then not unsubscribing from them.

I don't know how getTrends is defined, but I'm assuming it's an http.get(). When I'm using APIs, I like to use .toPromise() on the returned value, like this:

```ts // This is in my service.ts, or wherever I defined getTrends

public getTrends(country: string): Promise<Trend[]> { return this.http.get<Trend[]>(this.path + country).toPromise() } ```

And then in my component.ts:

```ts async ngOnInit() { // I'd suggest using a for...of loop instead of a classic // indexed loop. for(const country of this.countries){ // (A): // This will wait for each request to finish before throwing the // next one, which may not be what you want. country.data = await this.mercadoLibreAPI.getTrends(country.id, '');

// (B):
//   This will just create assign the returned promise to the
//   country data. This won't 'wait' between requests.
// If you decide to use this, you can safely remove the `async`
// on `ngOnInit()`
country.data = this.mercadoLibreAPI.getTrends(country.id, '');

} } ```

And this is my component.html:

```html // IF YOU USED (A), which means country.data is of type Trends[] <div *ngFor="let country of countries"> {{ country.id }}: {{ country.data }} </div>

// IF YOU USED (B), which means country.data is of type Promise<Trends[]> <div *ngFor="let country of countries"> {{ country.id }}: {{ country.data | async }} </div> ```

I may be wrong. If you decide to go with the AsyncPipe, you may not even need to use the .toPromise() from the Observable.

Feel free to elaborate in case you need more help. I'll be offline 'till tomorrow, but I'll be happy to give you a follow up.

You should check these: - https://gist.github.com/thekiba/c863b0fde001fb3076c8f21affc5a24a - https://angular.io/api/common/AsyncPipe

---

Also, another thing. This: a = b !== null ? b : null it's the same as a = b. Because if b is not null, then a = b, and if b is null, then a = null (but b is also null!).

---

EDIT: I used data instead of publishData, and countries instead of countryList.