Skip to content
This repository has been archived by the owner on Dec 30, 2022. It is now read-only.

feat(ais-relevant-sort): add widget #918

Merged
merged 13 commits into from
Mar 3, 2021
Merged

feat(ais-relevant-sort): add widget #918

merged 13 commits into from
Mar 3, 2021

Conversation

eunjae-lee
Copy link
Contributor

@eunjae-lee eunjae-lee commented Feb 25, 2021

Summary

This PR adds ais-smart-sort widget.

Related task:

  • stats widget

In other flavors:

@codesandbox-ci
Copy link

codesandbox-ci bot commented Feb 25, 2021

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 fbf330b:

Sandbox Source
vue-instantsearch-e-commerce Configuration

package.json Outdated Show resolved Hide resolved
@Haroenv Haroenv marked this pull request as ready for review March 2, 2021 16:04
@Haroenv Haroenv requested review from a team, yannickcr and shortcuts and removed request for a team March 2, 2021 16:17
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

LGTM! We should probably wait until the final name has been decided to merge it, wdyt?

@Haroenv Haroenv changed the title feat(smartSort): add widget feat(ais-relevant-sort): add widget Mar 3, 2021
@Haroenv Haroenv merged commit 4fe5745 into master Mar 3, 2021
@Haroenv Haroenv deleted the feat/smart-sort branch March 3, 2021 13:25
Haroenv added a commit to algolia/instantsearch that referenced this pull request Dec 28, 2022
* feat(smartSort): add widget

* add story

* add isSmartSorted to text slot

* fix story

* fix(smartsort): don't render if not virtual replica

* test(smartsort): add tests

* chore(deps): update instantsearch

* chore(deps): update

* chore: update bundlesize

* update to relevantSort

* feat(stats): apply relevant sort

* chore: update instantsearch

Co-authored-by: Haroen Viaene <hello@haroen.me>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants