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

Display name of Container Pod instead of HTML in Container Build's Textual summary #6575

Merged

Conversation

hstastna
Copy link

@hstastna hstastna commented Jan 2, 2020

Issue: #6572

There was HTML code in the Build Instances table in textual summary of selected Container Build. I've replaced link_to by value.link + value.value in this PR. Making the link look like a link wil be done in a follow up PR (in react-ui-components repo).


Before:
pod_before

After:
container

@hstastna
Copy link
Author

hstastna commented Jan 2, 2020

@miq-bot add_label bug

@hstastna hstastna changed the title [WIP] Display name of Container Pod instead of HTML in Container Build's Textual summary Display name of Container Pod instead of HTML in Container Build's Textual summary Jan 6, 2020
@hstastna
Copy link
Author

hstastna commented Jan 6, 2020

@miq-bot remove_label wip

@hstastna
Copy link
Author

hstastna commented Jan 7, 2020

@ZitaNemeckova @himdel @martinpovolny Could you, please, review this? Thank you 🙏

@hstastna hstastna force-pushed the HTML_ContainerBuild_summary_table branch from 3b62227 to 31caca1 Compare January 7, 2020 11:58
@miq-bot
Copy link
Member

miq-bot commented Jan 7, 2020

Checked commit hstastna@31caca1 with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

@martinpovolny martinpovolny added this to the Sprint 128 Ending Jan 20, 2020 milestone Jan 7, 2020
@martinpovolny martinpovolny self-assigned this Jan 7, 2020
@martinpovolny
Copy link
Member

martinpovolny commented Jan 7, 2020

Changelog entry to be composed with #6577 (something like "Fixed display of 'Build Instances' under Container Pod")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants