-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix 16805: pagination dropdown issue #18611
Conversation
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
openmetadata-ui/src/main/resources/ui/src/pages/APICollectionPage/APICollectionPage.tsx
Outdated
Show resolved
Hide resolved
openmetadata-ui/src/main/resources/ui/src/pages/APICollectionPage/APICollectionVersionPage.tsx
Outdated
Show resolved
Hide resolved
...adata-ui/src/main/resources/ui/src/pages/DatabaseSchemaPage/DatabaseSchemaPage.component.tsx
Outdated
Show resolved
Hide resolved
...a-ui/src/main/resources/ui/src/pages/DatabaseSchemaVersionPage/DatabaseSchemaVersionPage.tsx
Outdated
Show resolved
Hide resolved
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
const pagingInfo = usePaging(PAGE_SIZE); | ||
|
||
const { paging, pageSize, handlePagingChange } = pagingInfo; |
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.
const pagingInfo = usePaging(PAGE_SIZE); | |
const { paging, pageSize, handlePagingChange } = pagingInfo; | |
const { paging, pageSize, handlePagingChange } = usePaging(PAGE_SIZE); |
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.
pagingInfo
has other properties like showPagination
, handlePageChange
, etc. and these properties are being used in other components where we pass pagingInfo
as a prop.
On line 106, the destructured properties are being used in the component.
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.
need to fix playwright before merging
Quality Gate passed for 'open-metadata-ui'Issues Measures |
Fixes #16805
I worked on pagination dropdown for the pages, DatabaseSchemaPage, APICollectionPage, ServiceDetailsPage, etc.
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>