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(recommend): introduce TrendingItems #6178

Merged
merged 3 commits into from
May 3, 2024

Conversation

Haroenv
Copy link
Contributor

@Haroenv Haroenv commented May 3, 2024

Summary

Introduces the TrendingItems React widget and its related tests. Note that the tests have the same complexity on the type as is the case in vanilla; as the facet props can only be used if they're both set, but manipulating the props as a whole loses that relation.

Result

FX-2853

@Haroenv Haroenv added the Feature: Recommendations Providing recommendations of records or facets label May 3, 2024
Copy link

codesandbox-ci bot commented May 3, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 0adfb50:

Sandbox Source
react-instantsearch-app Configuration
example-react-instantsearch-default-theme Configuration
example-react-instantsearch-next-app-dir-example Configuration
example-vue-instantsearch-default-theme Configuration

@Haroenv Haroenv force-pushed the feat/recommend-ti-react-widget branch from 534e470 to 1e539fb Compare May 3, 2024 12:53
Copy link
Member

@dhayab dhayab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a nitpicky change request, otherwise that's good for me!

Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com>
@Haroenv Haroenv merged commit 8462748 into feat/map-recommend-results May 3, 2024
0 of 3 checks passed
@Haroenv Haroenv deleted the feat/recommend-ti-react-widget branch May 3, 2024 15:31
Haroenv added a commit that referenced this pull request May 21, 2024
* feat(recommend): introduce TrendingItems

FX-2853

* fix(recommend): correct type for generic base

Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com>

---------

Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com>
dhayab added a commit that referenced this pull request May 21, 2024
* feat(recommend): introduce TrendingItems

FX-2853

* fix(recommend): correct type for generic base

Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com>

---------

Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com>
dhayab added a commit that referenced this pull request May 21, 2024
* feat(recommend): introduce TrendingItems

FX-2853

* fix(recommend): correct type for generic base

Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com>

---------

Co-authored-by: Dhaya <154633+dhayab@users.noreply.github.com>
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

2 participants