-
Notifications
You must be signed in to change notification settings - Fork 297
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
Fixes #38113 - Add a link to REX bootc job templates in image mode details card. #11272
Conversation
1adbc6f
to
f24d38c
Compare
Have you run in rails console:
|
@@ -28,6 +34,7 @@ const ImageModeCard = ({ isExpandedGlobal, hostDetails }) => { | |||
masonryLayout | |||
isExpandedGlobal={isExpandedGlobal} | |||
> | |||
<a href={actionUrl(hostDetails.name)}>{__('Bootc action template')}</a> |
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.
I think using "template" is a little confusing for users, perhaps we can find a title that better describes the relationship to running bootc actions on the host:
<a href={actionUrl(hostDetails.name)}>{__('Bootc action template')}</a> | |
<a href={actionUrl(hostDetails.name)}>{__('Modify via remote execution')}</a> |
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.
@jeremylenz you mentioned having some potential input too :)
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.
Right, I was thinking Run bootc action
maybe? but actually I like this better!
f24d38c
to
894c248
Compare
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.
Didn't execute the code but the wording LGTM now. 👍
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.
Works great for me!
What are the changes introduced in this pull request?
Considerations taken when implementing this change?
What are the testing steps for this pull request?
Host - Details - Image mode details