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

[elasticsearch] Use pagination instead for results #194

Merged
merged 1 commit into from
Oct 31, 2021

Conversation

ricoberger
Copy link
Member

Instead if infinite scrolling (via the load more button) we are now
using pagination to display the documents returned by our API. For that
we can remove the depracated scroll API and directly request 1000
documents from the Elasticsearch API.

Using pagination instead of infite scrolling improves the performance of
our react UI when the user wants to view a lot of documents, e.g. when
the user wants to show 500 documents, the UI becomes laggy with infinite
scrolling. This problem can be avoided with pagination.

Instead if infinite scrolling (via the load more button) we are now
using pagination to display the documents returned by our API. For that
we can remove the depracated scroll API and directly request 1000
documents from the Elasticsearch API.

Using pagination instead of infite scrolling improves the performance of
our react UI when the user wants to view a lot of documents, e.g. when
the user wants to show 500 documents, the UI becomes laggy with infinite
scrolling. This problem can be avoided with pagination.
@ricoberger ricoberger merged commit 860697d into main Oct 31, 2021
@ricoberger ricoberger deleted the elasticsearch-use-pagination branch October 31, 2021 09:57
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.

1 participant