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.

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

chore(match expression): visualizer should have title #1060

Merged

Conversation

aali309
Copy link
Contributor

@aali309 aali309 commented Jul 17, 2023

Welcome to Cryostat! 👋

Before contributing, make sure you have:

  • Read the contributing guidelines
  • Linked a relevant issue which this PR resolves
  • Linked any other relevant issues, PR's, or documentation, if any
  • Resolved all conflicts, if any
  • Rebased your branch PR on top of the latest upstream main branch
  • Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
  • Signed the last commit: git commit --amend --signoff

Fixes: #1041

Description of the change:

This change adds a title at the top of the page for the card that contains the Match Expression Visualizer.

Motivation for the change:

This change is helpful as suggested by doc team. The title will help users that comes across the word Match Expression Visualizer in documentations to figure out what the UI element it is referring to.

How to manually test:

  1. Run CRYOSTAT_IMAGE=quay.io... sh smoketest.sh...
  2. ...

@aali309 aali309 self-assigned this Jul 17, 2023
@aali309 aali309 added the chore Refactor, rename, cleanup, etc. label Jul 17, 2023
@aali309 aali309 marked this pull request as ready for review July 17, 2023 18:37
@github-actions
Copy link

Test image available:

CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-web:pr-1060-612c30314d709e77876a435f95f44a3b83476a84 sh smoketest.sh

@github-actions
Copy link

Test image available:

CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-web:pr-1060-b0f89b8c4d7ca003ef9e6e3510978198218a9755 sh smoketest.sh

src/app/Rules/CreateRule.tsx Outdated Show resolved Hide resolved
Copy link
Member

@tthvo tthvo left a comment

Choose a reason for hiding this comment

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

Looks good! One more comment and pls run yarn format:apply.

@github-actions
Copy link

Test image available:

CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-web:pr-1060-e1ad323c495f6513c9bc70c499c0d5927236573c sh smoketest.sh

@aali309 aali309 force-pushed the match_expression_visualizer_title branch 2 times, most recently from 2364ddd to 09048a6 Compare July 17, 2023 22:24
@github-actions
Copy link

Test image available:

CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-web:pr-1060-09048a671aac2e6434781a3bfdfe7296cc458072 sh smoketest.sh

tthvo
tthvo previously approved these changes Jul 17, 2023
Copy link
Member

@tthvo tthvo left a comment

Choose a reason for hiding this comment

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

image

Looks good to me! Integration tests seem flanky on CI (4th time to pass) :))

@github-actions
Copy link

Test image available:

CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-web:pr-1060-eacd093f0769a2724cffab378f6c42a3f793632d sh smoketest.sh

tthvo
tthvo previously approved these changes Jul 17, 2023
@aali309 aali309 force-pushed the match_expression_visualizer_title branch 3 times, most recently from fbf3860 to 87607b9 Compare July 17, 2023 23:54
@aali309 aali309 closed this Jul 17, 2023
@aali309 aali309 force-pushed the match_expression_visualizer_title branch from 87607b9 to 13db028 Compare July 17, 2023 23:58
@github-actions
Copy link

Test image available:

CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-web:pr-1060-13db02830ebfdb6f884d8b71406c648eb3b4202a sh smoketest.sh

@aali309 aali309 reopened this Jul 18, 2023
@github-actions
Copy link

Test image available:

CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-web:pr-1060-4a854b22fe672b691a8b1bb6e7d36a5a3f3a7ab3 sh smoketest.sh

Copy link
Member

@maxcao13 maxcao13 left a comment

Choose a reason for hiding this comment

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

LGTM too.

@andrewazores andrewazores merged commit 9ed8cb1 into cryostatio:main Jul 25, 2023
1 check passed
mergify bot pushed a commit that referenced this pull request Jul 25, 2023
@github-actions
Copy link

Test image available:

CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-web:pr-1060-4a854b22fe672b691a8b1bb6e7d36a5a3f3a7ab3 sh smoketest.sh

andrewazores pushed a commit that referenced this pull request Jul 25, 2023
@aali309 aali309 deleted the match_expression_visualizer_title branch July 25, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport chore Refactor, rename, cleanup, etc. safe-to-test
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Task] Match Expression Visualizer should have title
4 participants