-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Pagination #1419
Pagination #1419
Conversation
@fadingNA is attempting to deploy a commit to the Arc53 Team on Vercel. A member of the Team first needs to authorize it. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1419 +/- ##
==========================================
- Coverage 34.03% 34.00% -0.03%
==========================================
Files 69 69
Lines 3376 3405 +29
==========================================
+ Hits 1149 1158 +9
- Misses 2227 2247 +20 ☔ View full report in Codecov by Sentry. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Fixed data fetching, pre calculate pagination data on backend, and passing along with original document for using in another components |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
There are some minor type errors here on build
|
Got it let me fix this |
change routes /combine name, add route /api/source/paginated add new endpoint source/paginated fixing table responsive create new function to handling api/source/paginated
Create new endpoint /api/sources/paginated, change api/combine to api/sources fallback situational calling api/combine will redirect to api/sources paginated instead of manually calculate using mongodb function with sort,skip,limit Thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love the changes @fadingNA
Thanks!
Thank you so much Manish !! |
New feature add pagination to Setting / Document for user that have uploaded multiple document have pagination to align the documents
@ManishMadan2882 thanks for this.