Support defer loading of dataTables data #638
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Closes #629 [skip ci]
[skip ci]
ajstewart
approved these changes
Mar 3, 2022
|
Just thought, not sure if there is any wording in the docs that needs updating. I.e. it implies the results are already there. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds support to defer the loading of dataTables data with AJAX. Applied to the sources table on the source query page, this stops the results table from being populated with all sources. The results table is only populated when the user submits a query using the form. If the user wants to view all sources, they can still do this by submitting an empty query.
Closes #629