-
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
allItems template shouldn't be available in infiniteHits #2543
Labels
Comments
The fact that you can use the allItems is a bug :/ https://community.algolia.com/instantsearch.js/v2/widgets/infiniteHits.html#struct-InfiniteHitsTemplates |
bobylito
changed the title
Infinite Hits with allItems template
allItems template shouldn't be available in infiniteHits
Nov 1, 2017
Oh that’s right, I’m sorry, I read this as a duplicate |
bobylito
pushed a commit
that referenced
this issue
May 29, 2018
The infinite hits widget is based internally on the hits widget which accepts the allItems template. However this template is not usable for the purpose of the infinite hits widget. In order to prevent users from using this buggy and yet accepted option this PR generates a exception if we detect this particular template.
bobylito
pushed a commit
that referenced
this issue
May 29, 2018
The infinite hits widget is based internally on the hits widget which accepts the allItems template. However this template is not usable for the purpose of the infinite hits widget. In order to prevent users from using this buggy and yet accepted option this PR generates a exception if we detect this particular template. Fix #2543
bobylito
added a commit
that referenced
this issue
May 29, 2018
The infinite hits widget is based internally on the hits widget which accepts the allItems template. However this template is not usable for the purpose of the infinite hits widget. In order to prevent users from using this buggy and yet accepted option this PR generates a exception if we detect this particular template. Fix #2543
bobylito
pushed a commit
that referenced
this issue
May 29, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hits allows for the allItems template which allows for greater customization of the templates, for example inside tables and so forth...
When the allItems template is used with infiniteHits, its doesn't give any error, but instead of adding the newly loaded items, it replaces everything with the new page of results.
The text was updated successfully, but these errors were encountered: