-
Notifications
You must be signed in to change notification settings - Fork 503
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(recommend): introduce connectRelatedProducts
connector
#6142
feat(recommend): introduce connectRelatedProducts
connector
#6142
Conversation
fa726da
to
3121f7a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few questions and changes on how to align both connectors PRs.
I like the outline of the connector common test suite. Are we confident with our "end-user" test suite and dropping more granular tests that might not be possible in CTS (like checking the number of renderings for example) ?
packages/instantsearch.js/src/connectors/related-products/connectRelatedProducts.ts
Show resolved
Hide resolved
packages/instantsearch.js/src/connectors/related-products/connectRelatedProducts.ts
Outdated
Show resolved
Hide resolved
packages/instantsearch.js/src/connectors/related-products/connectRelatedProducts.ts
Outdated
Show resolved
Hide resolved
packages/instantsearch.js/src/connectors/related-products/connectRelatedProducts.ts
Show resolved
Hide resolved
Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com> Co-authored-by: Raed <raed.chammam@algolia.com> Co-authored-by: Aymeric Giraudet <aymeric.giraudet@algolia.com>
Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com> Co-authored-by: Raed <raed.chammam@algolia.com> Co-authored-by: Aymeric Giraudet <aymeric.giraudet@algolia.com>
Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com> Co-authored-by: Raed <raed.chammam@algolia.com> Co-authored-by: Aymeric Giraudet <aymeric.giraudet@algolia.com>
This introduces the
connectRelatedProducts
connector and tests it in the connector common test suite.https://algolia.atlassian.net/browse/FX-2813