-
Notifications
You must be signed in to change notification settings - Fork 22
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
🧼 Fix more help text fields, based on doc team review for the OpenStack provider #876
🧼 Fix more help text fields, based on doc team review for the OpenStack provider #876
Conversation
… provider Reference: kubev2v#646 Reference: kubev2v#861 (comment) Fix all comments mentioned by the doc team in kubev2v#861 and applied it to all relevant providers. Signed-off-by: Sharon Gratch <[email protected]>
Quality Gate failedFailed conditions 33.7% Duplication on New Code (required ≤ 20%) |
<br /> | ||
Insecure migration means that the transferred data is sent over an insecure connection | ||
and potentially sensitive data could be exposed. | ||
Note: If this field is checked/true, migrations from this provider will not be secure, |
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.
"selected" instead of "checked/true"
<br /> | ||
Insecure migration means that the transferred data is sent over an insecure connection | ||
and potentially sensitive data could be exposed. | ||
Note: If this field is checked/true, migrations from this provider will not be secure, |
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.
"selected" instead of "checked/true"
<br /> | ||
Insecure migration means that the transferred data is sent over an insecure connection | ||
and potentially sensitive data could be exposed. | ||
Note: If this field is checked/true, migrations from this provider will not be secure, |
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.
"selected" instead of "checked/true"
<br /> | ||
Insecure migration means that the transferred data is sent over an insecure connection | ||
and potentially sensitive data could be exposed. | ||
Note: If this field is checked/true, migrations from this provider will not be secure, |
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.
"selected" instead of "checked/true"
<br /> | ||
Insecure migration means that the transferred data is sent over an insecure connection | ||
and potentially sensitive data could be exposed. | ||
Note: If this field is checked/true, migrations from this provider will not be secure, |
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.
"selected" instead of "checked/true"
<br /> | ||
Insecure migration means that the transferred data is sent over an insecure connection and | ||
potentially sensitive data could be exposed. | ||
Note: If this field is checked/true, migrations from this provider will not be secure, |
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.
"selected" instead of "checked/true"
Great work, @sgratch! There's one small repeated error which I may have missed before. Otherwise, LGTM! |
Reference: #646
Reference: #861 (comment)
Fix all comments mentioned by the doc team in PR #861 and applied it to all relevant providers if relevant.