Skip to content

Conversation

@rishumehra
Copy link
Contributor

@rishumehra rishumehra commented Mar 10, 2021

This PR has additional Bugfixes for the ODC 4.7.

  • Applies to 4.7
  • Reviewed by SMEs/QEs
  • Aligned team label DevTools

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 10, 2021
@netlify
Copy link

netlify bot commented Mar 10, 2021

Deploy preview for osdocs ready!

Built with commit be92fd0

https://deploy-preview-30299--osdocs.netlify.app

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Previously, in the *YAML view* of the Helm installation form only the YAML schema or code was shown. There was no option to provide and view comments with parameters. Now you can view parameters in defined order with comments. A *Schema* viewer is added to the page, next to *YAML* editor to check all the schema objects present in the Helm Chart. (link:https://bugzilla.redhat.com/show_bug.cgi?id=1886861[*BZ#1886861*])
* Previously, in the *YAML view* of the Helm installation form only the YAML schema or code was shown. There was no option to provide and view comments with parameters. Now you can view parameters in a defined order with comments. A *Schema* viewer is added to the page, next to the *YAML* editor to check all the schema objects present in the Helm Chart. (link:https://bugzilla.redhat.com/show_bug.cgi?id=1886861[*BZ#1886861*])

Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM

@openshift-ci-robot openshift-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Mar 11, 2021

Choose a reason for hiding this comment

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

YAML schema was not shown earlier. Only YAML code was shown. This bug fix adds the schema viewer in yaml editor. Also, the comments are not shown in YAML code. Only the schema and its descriptions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the clarification @rohitkrai03. Will update the content based on this comment.

@rohitkrai03
Copy link

LGTM

Copy link
Contributor

@JStickler JStickler left a comment

Choose a reason for hiding this comment

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

LGTM

@Preeticp Preeticp added this to the Next Release milestone Mar 18, 2021
@Preeticp Preeticp merged commit 1c55791 into openshift:enterprise-4.7 Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.7 size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants