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

[INLONG-11183][Dashboard] Module audit page indicator items are merged with other items #11185

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[INLONG-11183][Dashboard] Module audit page indicator items are merged with other items #11185

wants to merge 1 commit into from

Conversation

wohainilaodou
Copy link
Contributor

Fixes #11183

Motivation

Module audit page indicator items are merged with other items

Modifications

Separate request logic to reduce the number of requests, merge indicator items and other categories of items

Verifying this change

before:
id,ip
image
metric:
image

after:
id,ip
image
metric:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aloyszhang aloyszhang aloyszhang approved these changes

@leezng leezng Awaiting requested review from leezng

@bluewang bluewang Awaiting requested review from bluewang

@dockerzhang dockerzhang Awaiting requested review from dockerzhang

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
component/dashboard
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Improve][Dashboard] Module audit page indicator items are merged with other items
2 participants
@wohainilaodou @aloyszhang