fix(recommend): update TrendingFacetHit facetValue type to string

[raed667]

Why

The trending-facets API only returns type string for facetValue.

A follow-up on #1494 as other models return items/hits which are of type RecordWithObjectID

However, trending-facets model returns a list of TrendingFacetHit which (by definition) don't have an objectID property.

How

To simplify the code, we can remove the type argument TObject for trending-facets

Impact

If you're not using TypeScript or the trending-facets model, there is nothing to change. You can ignore the following.

Otherwise, you will need to remove the type argument passed to getTrendingFacets and the code goes from this:

type FacetType = {
  facetName: string;
  facetValue: string;
}
const { results } = await client.getTrendingFacets<FacetType>(/**/)

To this:

const { results } = await client.getTrendingFacets(/**/)

[Haroenv]

if this type is exported, it's a breaking change, someone who would be using TrendingFacetHit<string> will get a typescript error. Instead all generics should stay, defaulted to string

[raed667]

IMO it is fine to have a build step breaking change in this case as it is a bug fix in typings.

Plus the usage for this model is very small and the update will be a one liner documented in the release notes.

[Haroenv]

BC ok because of low impact

[shortcuts]

BC ok because of low impact

might still worth a minor right?