feat(crossSelling): parametrize groupByProduct
parameter
#460
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What problem is this solving?
groupByProduct
param oncrossSelling
method of thesearch
clientgroupByProduct
param toproductRecommendations
query.quantity
param toRecommendation.similars
resolver.These are necessary for the resolver to return more than 12 similar products.
In the Fashion segments it's common to have different product colours as different (similar) products instead of SKUs due to the limited number of SKUs we can have on the platform. There's a limitation in the current implementation that caps the number of similar products to 12, forcing us to make extra API calls.
This MR parameterize the (currently hardcoded) control variable
groupByProduct
in a backwards compatible way.There's also an MR in
vtex.search-graphql
that updates the necessary schemas.How should this be manually tested?
GraphQL IDE on the following Workspace
Example Queries
Checklist/Reminders
README.md
.CHANGELOG.md
.Screenshots or example usage
Type of changes
Notes