Skip to content
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.

Sign up for GitHub

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

Jump to bottom

PMM-12070 Add table plugin node overview dashboard #1522

Merged
merged 4 commits into from
Jul 28, 2023
Merged

PMM-12070 Add table plugin node overview dashboard #1522

merged 4 commits into from
Jul 28, 2023

Conversation

YashSartanpara1
Copy link
Contributor

@YashSartanpara1 YashSartanpara1 commented Jul 4, 2023 •
edited
Loading

@YashSartanpara1 YashSartanpara1 mentioned this pull request Jul 4, 2023
Closed
@YashSartanpara1 YashSartanpara1 marked this pull request as ready for review July 5, 2023 07:58
rnovikovP
rnovikovP previously approved these changes Jul 5, 2023
View reviewed changes
Copy link
Contributor

@rnovikovP rnovikovP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The presentation looks good to me, considering the current limitations of the table.

matejkubinec
matejkubinec previously approved these changes Jul 10, 2023
View reviewed changes
@YashSartanpara1 YashSartanpara1 enabled auto-merge (squash) July 28, 2023 10:36
@YashSartanpara1 YashSartanpara1 merged commit 2b104b6 into main Jul 28, 2023
4 checks passed
@YashSartanpara1 YashSartanpara1 deleted the PMM-12070-table-plugin-os-node-overview branch July 28, 2023 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matejkubinec matejkubinec matejkubinec approved these changes

@rnovikovP rnovikovP rnovikovP left review comments

@doracretu3pillar doracretu3pillar Awaiting requested review from doracretu3pillar

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@YashSartanpara1 @rnovikovP @matejkubinec