-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
ui: Decommissioned node history page - fix follow up issues #44205
ui: Decommissioned node history page - fix follow up issues #44205
Conversation
@koorosh can you rebase on master so we can see with new sidebar? |
@@ -52,7 +52,11 @@ class DecommissionedNodeHistory extends React.Component<DecommissionedNodeHistor | |||
{ | |||
key: "id", | |||
title: "ID", | |||
sorter: true, | |||
sorter: (a, b) => { |
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.
Can you extract these sorter functions and give them names like sortByNodeID
etc.
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.
Done.
}; | ||
|
||
// DecommissionedNodeList displays 5 most recent nodes. | ||
const data = _.chain(decommissionedStatuses) |
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.
Are we limiting to 5? I don't see that here.
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.
Done.
- Updated design was reused to look the same as tables on Cluster Overview page - Pagination buttons are aligned by center - Added empty table state with proper message Release note (admin ui change): Decommissioned Node History view changed
Apply correct sorting comparator function for Node Id and Decommissioned on columns. Release note: None
b48770c
to
9295a0f
Compare
…issioned Nodes table Release note: None
bors r+ |
44205: ui: Decommissioned node history page - fix follow up issues r=dhartunian a=koorosh Decommissioned Node History table has been redesigned according to the design system styles. - It doesn't use Sorted Table now, and was replaced by Table component. Fixes for: #43881 Co-authored-by: Andrii Vorobiov <and.vorobiov@gmail.com>
Build succeeded |
Decommissioned Node History table has been redesigned according to the design system styles.
Fixes for: #43881