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

fix(ie11): preact-compat algolia fork with #423 #2471

Closed
wants to merge 3 commits into from

Conversation

bobylito
Copy link
Contributor

This is to fix an issue of preact-compat on IE11. It uses an algolia fork of preact-compat, that we need to replace once the new version is out (3.17.1 or 3.18.0)

Alex S added 2 commits October 11, 2017 11:36
<a name=2.2.1-beta.0></a>
## [2.2.1-beta.0](v2.2.0...v2.2.1-beta.0) (2017-10-11)

### Bug Fixes

* **connectRangeSlider:** only clear the refinement on the current attribute ([#2459](#2459)) ([7cebf58](7cebf58))
* **ie11:** peract-compat algolia fork with [#423](#423) ([c22e436](c22e436))
* **menuSelect:** select in userCssClasses ([#2455](#2455)) ([0eb3dc8](0eb3dc8))
* **menuSelect:** use preact instead of React ([#2460](#2460)) ([35ccae8](35ccae8))
* **test:** correctly reset the wired dependency ([#2461](#2461)) ([1f7f4ed](1f7f4ed))
@bobylito bobylito requested a review from iam4x October 11, 2017 12:58
@algobot
Copy link
Contributor

algobot commented Oct 11, 2017

Deploy preview ready!

Built with commit ee13d6f

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

@@ -1,6 +1,6 @@
{
"name": "instantsearch.js",
"version": "2.2.0",
"version": "2.2.1-beta.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

this should only change with the build script

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's alright, since it's gonna be 2.2.1 anyway.

@iam4x
Copy link
Contributor

iam4x commented Oct 11, 2017

Failling on the commit:

Checking commit message for 2c5abfc3e7315d6a723b96a539a09857d48e101d... KO (format): v2.2.1-beta.0

@bobylito
Copy link
Contributor Author

Well it seems that it might not fix what is supposed to. Will check that tomorrow.

@bobylito bobylito changed the title fix(ie11): peract-compat algolia fork with #423 fix(ie11): preact-compat algolia fork with #423 Oct 12, 2017
@bobylito
Copy link
Contributor Author

Actually the library works with IE11. No need for this PR.

@bobylito bobylito closed this Oct 13, 2017
@bobylito bobylito deleted the fix/preact-423-ie11 branch October 13, 2017 13:07
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.

3 participants