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

Add pagination to dataset list/table components #86

Closed
deepakduggirala opened this issue Oct 10, 2023 · 0 comments · Fixed by #97
Closed

Add pagination to dataset list/table components #86

deepakduggirala opened this issue Oct 10, 2023 · 0 comments · Fixed by #97
Assignees

Comments

@deepakduggirala
Copy link
Contributor

deepakduggirala commented Oct 10, 2023

In the /raw_data and /data_products pages, the datasets table, which is managed by a single component (ui/src/components/dataset/DatasetList.vue), currently lacks pagination. This will cause performance issues when there are a large number of datasets to display. Additionally, the absence of pagination impacts the efficiency of searching and sorting, as these operations are currently performed in the browser on the entire dataset array. With pagination enabled, we would need to make API calls for searching and sorting.

For datasets associated with projects, we need to consider a different approach. If there are 10 datasets or fewer (or some suitable threshold defined), it may be preferable to hide the pagination controls, as a cleaner, no-pagination approach might provide a better user experience. However, if there are more than 10 datasets, we should enable pagination.

Additional Feature:
In the project-associated dataset table, we should also introduce a search feature for cases where there are more than 10 datasets. This search feature will help users quickly locate specific datasets within the larger set of project-associated datasets.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants