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

feat(vanilla): add algolia-insights code sample #76

Merged
merged 7 commits into from Mar 20, 2019
Merged

feat(vanilla): add algolia-insights code sample #76

merged 7 commits into from Mar 20, 2019

Conversation

tkrugg
Copy link
Contributor

@tkrugg tkrugg commented Mar 19, 2019

No description provided.

Copy link
Member

@francoischalifour francoischalifour left a comment

Choose a reason for hiding this comment

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

The example doesn't work on CodeSandbox but I assume it works in a regular environment:

Uncaught Error: Before calling any methods on the analytics, you first need to call the 'init' function with appId and apiKey parameters

InstantSearch.js/algolia-insights/index.html Show resolved Hide resolved
InstantSearch.js/algolia-insights/index.html Show resolved Hide resolved
InstantSearch.js/algolia-insights/src/app.js Show resolved Hide resolved
InstantSearch.js/algolia-insights/src/app.js Outdated Show resolved Hide resolved
InstantSearch.js/algolia-insights/index.html Outdated Show resolved Hide resolved
Haroenv and others added 2 commits March 19, 2019 16:39
@tkrugg
Copy link
Contributor Author

tkrugg commented Mar 20, 2019

@francoischalifour : The example doesn't work on CodeSandbox but I assume it works in a regular environment:

Yes. As we talked about it the other day when you faced the same problem, there is some kind of bug on codesandbox with global variables that breaks all insights examples.

@tkrugg tkrugg merged commit aa1c2b8 into master Mar 20, 2019
@tkrugg tkrugg deleted the insights branch October 29, 2021 13:28
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.

None yet

3 participants