-
Notifications
You must be signed in to change notification settings - Fork 516
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
setImmediate is not defined on Meteor + React #1807
Comments
Hi @kolyasya, Currently we are using setImmediate to perform some scheduling and it seems that this function isn't available with meteor. We are going to change this code to be compatible with the largest lib, but in the meantime one thing you can do is using a Also you are going to have another issue because of the Here a working example: import algoliaClient from 'algoliasearch/src/browser/builds/algoliasearch'
<InstantSearch
indexName="xxx"
algoliaClient={algoliaClient('appId', 'apiKey')}
>
</InstantSearch> Let me know if this help :) |
@mthuret thank you for a detailed answer, it helped :-) All works as it should now! |
That's great to hear @kolyasya! |
@mthuret and @kolyasya: I'm trying to use Passing the Is it possible to use InstantSearch in Meteor? Would it be possible to change |
I can confirm that changing |
We are no more using setImmediate in react-instantsearch, @damonmaria what you are experiencing is due to #1024 (comment) This is a meteor.js issue for now. I bet you could also fix it by tuning your build tool to alias algoliasearch to build/algoliasearch somehow? |
Thanks @vvo. Yes, I knew the Meteor deficiency in handling the For the moment I have forked this package and changed the |
We do not yet have good SSR in react-instantsearch so I bet we could ultimately make this change but it won't be 100% future proof because ultimately we will get server side rendering and the fix will have to be reverted. I'd rather see meteor.js take action or someone (maybe you) do the necessary changes in meteor.js to handle the browser field on packages. |
Meteor.js fully implementing use of the |
react-instantsearch: 2.1.0
Latest versions of Chrome and FF.
Hello, I've just started setup react-instantsearch on my Meteor project and have strange bug.
I'm following steps from the React Instant Search Getting Started Tutorial and when I insert first widget (
<Hits />
for example) I have an error in console:It refers to this code:
The text was updated successfully, but these errors were encountered: