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

feat(widget:toggle-refinement): allow multiple values for "on" and "off" #780

Merged

Conversation

Kocal
Copy link
Contributor

@Kocal Kocal commented May 28, 2020

Hi 👋

This PR is for the proposal algolia/instantsearch#4420, which allow multiple values for on and off props.

Thanks!

@tristanbes
Copy link

Thanks, can't wait to use it :)

@Haroenv
Copy link
Contributor

Haroenv commented Jun 1, 2020

Thanks for the PR, the next release is still in progress due to server side rendering, but once we have this, we'll make sure to include the PR too

@Haroenv
Copy link
Contributor

Haroenv commented Jun 17, 2020

Hi @Kocal, could you rebase this on the master branch? :)

@Kocal
Copy link
Contributor Author

Kocal commented Jun 17, 2020

Ah great, v3 has been released!
I'm rebasing

EDIT: PR rebased

@Kocal Kocal force-pushed the feat/widget-toggle-refinement-multiple-values branch from 03b4b6b to 423b284 Compare June 17, 2020 14:25
@Kocal Kocal changed the base branch from next to master June 17, 2020 14:26
Copy link
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

This is likely good to go, but I’ll need to check it locally, since Netlify didn’t run

stories/ToggleRefinement.stories.js Outdated Show resolved Hide resolved
Co-authored-by: Haroen Viaene <fingebimus@me.com>
Copy link
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

this behaves great! I recently had to create a custom component to do this exact behaviour, so thanks :)

@Haroenv Haroenv merged commit 6ee1404 into algolia:master Jun 18, 2020
@Kocal Kocal deleted the feat/widget-toggle-refinement-multiple-values branch June 18, 2020 08:45
@Kocal
Copy link
Contributor Author

Kocal commented Jun 18, 2020

No problem :)

Haroenv added a commit to algolia/instantsearch that referenced this pull request Dec 28, 2022
…algolia/vue-instantsearch#780)

* feat(widget:toggle-refinement): allow multiples values for "on" and "off"

* chore: add story

* Update stories/ToggleRefinement.stories.js

Co-authored-by: Haroen Viaene <fingebimus@me.com>

Co-authored-by: Haroen Viaene <fingebimus@me.com>
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