Skip to content
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(instantsearch.js): introduce relatedProducts widget #6154

Merged
merged 85 commits into from
Apr 26, 2024

Conversation

sarahdayan
Copy link
Member

@sarahdayan sarahdayan commented Apr 23, 2024

This introduces the relatedProducts widget and tests it in the widget common test suite.

This PR changes a few things in how we pass templates to the UI component:

  • The <RelatedProducts> UI component now defines all default templates. This lets us define them once for all flavors instead of passing them manually and risking output drifts.

  • InstantSearch.js doesn't pass default templates in relatedProducts since they're defined in the UI component.

  • All relatedProducts templates are passed as Fragments so we can avoid redefining components in the widget.

  • https://algolia.atlassian.net/browse/FX-2814

  • https://algolia.atlassian.net/browse/FX-2815

@dhayab dhayab added the Feature: Recommendations Providing recommendations of records or facets label Apr 24, 2024
@sarahdayan sarahdayan marked this pull request as ready for review April 25, 2024 08:27
@sarahdayan sarahdayan changed the title feat(recommend): introduce relatedProducts widget feat(instantsearch.js): introduce relatedProducts widget Apr 25, 2024
@sarahdayan sarahdayan requested a review from dhayab April 26, 2024 12:20
@sarahdayan sarahdayan merged commit 58196fe into feat/map-recommend-results Apr 26, 2024
8 checks passed
@sarahdayan sarahdayan deleted the feat/recommend-rp-widget branch April 26, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature: Recommendations Providing recommendations of records or facets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants