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: add pagination to change dataset modal #14844
Conversation
/testenv up |
@junlincc Container image not yet published for this PR. Please try again when build is complete. |
@junlincc Ephemeral environment creation failed. Please check the Actions logs for details. |
/testenv up |
@junlincc Ephemeral environment spinning up at http://35.166.116.182:8080. Credentials are |
please see video before and after your change Screen.Recording.2021-05-26.at.11.17.04.AM.mov |
Codecov Report
@@ Coverage Diff @@
## master #14844 +/- ##
==========================================
- Coverage 77.62% 77.60% -0.02%
==========================================
Files 962 963 +1
Lines 49172 49285 +113
Branches 6184 6209 +25
==========================================
+ Hits 38169 38247 +78
- Misses 10801 10837 +36
+ Partials 202 201 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
/testenv up |
@junlincc Ephemeral environment spinning up at http://34.219.191.205:8080. Credentials are |
@pkdotson could you update the the video in description that reflects the latest? |
Left some comments of varied importance, but there's one thing I noticed while testing. The good news is, we can now see each page of the datasets! The bad news is, the table header sorting only sorts the particular page of results you're looking at, not the whole set For an example approach, the CRUD views work nicely, in that if you are sorting alphabetically, the first items on page one are all the I think it's worth solving this in a followup PR, where clicking the sort header changes the orderBy parameter on the query itself. Side note: (cc @junlincc here) the sort on |
Yes i'll create a ticket to do a follow up for server sorting to keep the scope of pagination with this pr. |
/testenv up |
@pkdotson Ephemeral environment spinning up at http://54.213.161.246:8080. Credentials are |
Ephemeral environment shutdown and build artifacts deleted. |
See #15014 for an alternate solution. |
SUMMARY
this pr adds pagination to change dataset modal so that user can go though all their datasets.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Screen.Recording.2021-05-26.at.11.52.37.AM.mov
TESTING INSTRUCTIONS
will be updating tests.
ADDITIONAL INFORMATION