Skip to content

Conversation

@francoischalifour
Copy link
Contributor

@francoischalifour francoischalifour commented Sep 19, 2018

@algobot
Copy link
Contributor

algobot commented Sep 19, 2018

Deploy preview for algolia-instantsearch ready!

Built with commit 35edaff

https://deploy-preview-3130--algolia-instantsearch.netlify.com

@francoischalifour francoischalifour changed the title feat(numeric-selector): apply specs feat(numeric-selector): implement InstantSearch.css Sep 19, 2018
@bobylito
Copy link
Contributor

Alternative stories are broken.

@francoischalifour
Copy link
Contributor Author

They were already because we don't accept undefined values: https://community.algolia.com/instantsearch.js/v2/dev-novel/?selectedStory=NumericSelector.default

I'll fix that in this PR.

Copy link
Contributor

@bobylito bobylito left a comment

Choose a reason for hiding this comment

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

Looks good but the stories don't seem to all work.

@francoischalifour
Copy link
Contributor Author

I rebased to get your fix for the stories @bobylito (see story).

@francoischalifour francoischalifour requested a review from a team October 1, 2018 19:03
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.

in the storybook the "with default selected" doesn't work

Copy link
Contributor

@samouss samouss left a comment

Choose a reason for hiding this comment

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

We didn't implemented it in Vue, React and it's not documented on Angular. We can close the PR but keep it until we are sure to remove it.

@francoischalifour
Copy link
Contributor Author

francoischalifour commented Oct 3, 2018

@Haroenv What is broken to you?

@samouss Oh, I wasn't aware. We can remove it in another PR, then.

@francoischalifour
Copy link
Contributor Author

Closing in favor of #3183.

@samouss samouss deleted the feat/3.0-numeric-selector branch October 11, 2018 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants