-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Messages V1 Pagination #2862
Messages V1 Pagination #2862
Conversation
@Haarolean, this one is ready for QA |
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.
seems not all messages are displayed within topic, there are 103 messages but displayed only 19. therefore pagination is not right. as we corrected the limitation for each page should be 100
pagination.mov
This is an old bug that is not related to the where you navigate to the page from the topic page the url, actually the code does not get updated before the request, once you refresh the page you will see 100 limited element in the messages But i will see what i can do to fix it , cause the code in that particular is a bit messy. @Haarolean |
697547f
to
7e3b7cd
Compare
add minor test in messages table component Add Messages pagination add next feature in the Advanced Filters Messages Table add Back button navigation remove un-necessary code from the messages table minor code refactors to remove unnecessary codes page parameter fix in the query of the advanced Filters minor change in the test file to keep uniformity of the project set default table limit for first request messages v1
SonarCloud Quality Gate failed. |
Custom deployment removed |
Custom deployment will be available at http://pr2862.internal.kafka-ui.provectus.io |
SonarCloud Quality Gate failed. |
What changes did you make? (Give an overview)
I have added next and prev pagination to the old messages page.
Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)
Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)
Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)