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

feat(ui): display sha's revision in every history release (#19962) #19963

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

Tchoupinax
Copy link
Contributor

Hello,

Following the regression reported by this issue, I made the fix to hide only the params table.

I created a sub component to have the state of the panel (open or closed) isolated for each history.

Closes #19962

@Tchoupinax Tchoupinax requested a review from a team as a code owner September 17, 2024 11:25
Copy link

bunnyshell bot commented Sep 17, 2024

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

  • 🚀 /bns:deploy to deploy the environment

Copy link

bunnyshell bot commented Sep 17, 2024

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

  • 🚀 /bns:deploy to deploy the environment

Copy link
Member

@reggie-k reggie-k left a comment

Choose a reason for hiding this comment

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

Tested locally, now revision, authored by and GPG sig are visible at all times, so the revisions are searchable at all times without clicking on the show details button.

@reggie-k reggie-k added component:ui User interfaces bugs and enhancements regression Bug is a regression, should be handled with high priority labels Sep 17, 2024
Copy link
Member

@gdsoumya gdsoumya left a comment

Choose a reason for hiding this comment

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

@Tchoupinax can you just add some screenshots of before and after?

@Tchoupinax
Copy link
Contributor Author

Sure!

Before:
image

After:
image

Copy link
Contributor

@todaywasawesome todaywasawesome left a comment

Choose a reason for hiding this comment

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

Looks great and is a good feature.

@gdsoumya gdsoumya merged commit f4c519a into argoproj:master Sep 19, 2024
32 of 34 checks passed
@Tchoupinax
Copy link
Contributor Author

Hello @reggie-k,

Could this fix be part of v2.13.0-rc4 or v2.13.0 please? The feature has been introduced in v2.13.0-rc1 and requires this fix for the final release.
Tell me if you need anything 🙏

Thanks!

@pasha-codefresh
Copy link
Member

/cherry-pick release-2.13

1 similar comment
@pasha-codefresh
Copy link
Member

/cherry-pick release-2.13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:ui User interfaces bugs and enhancements regression Bug is a regression, should be handled with high priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

In history screen, sha1 revision is not displayed by default
5 participants