-
Notifications
You must be signed in to change notification settings - Fork 357
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
Set minimum vSphere API version for WebMKS consoles #2538
Set minimum vSphere API version for WebMKS consoles #2538
Conversation
78523f3
to
c9b8803
Compare
c9b8803
to
7f5c13e
Compare
This pull request is not mergeable. Please rebase and repush. |
7f5c13e
to
4bf223b
Compare
@@ -7,8 +7,18 @@ def visible? | |||
end | |||
|
|||
def disabled? | |||
@error_message = _("The web-based WebMKS console is not available because the required libraries aren't installed") unless webmks_assets_provided? | |||
@error_message = _('The web-based WebMKS console is not available because the VM is not powered on') unless on? | |||
canned_msg = _('The web-based WebMKS console is not available because') |
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.
Please, don't compose senteses via string concatenation. This might not translate to other languates :-(
Replacing the "because" with ":" and starting with a capital letter. Might be a bette choice.
ping @mzazrivec
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.
Correct. The message needs to stay together. I get complaints regarding this from translators.
Also changing the strings in in the Fine branch is likely to produce missing translations. I doubt the catalogs are passed throught the translator team for Fine. |
Yes, should the fix land in some of the older branches, we need to avoid changing the strings |
@bmclaughlin : please, reformat this PR is such a way that the strings do not change. Thx! |
4bf223b
to
77f81cc
Compare
Checked commits bmclaughlin/manageiq-ui-classic@35eea08~...77f81cc with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0 |
@martinpovolny, @mzazrivec Reverted the String changes. Not sure if the CC issue needs to be addressed? Thoughts? |
@mzazrivec : can we merge this one? |
Set minimum vSphere API version for WebMKS consoles (cherry picked from commit 499d120) https://bugzilla.redhat.com/show_bug.cgi?id=1527106
Gaprindashvili backport details:
|
Set minimum vSphere API version for WebMKS consoles (cherry picked from commit 499d120) https://bugzilla.redhat.com/show_bug.cgi?id=1531280
Fine backport details:
|
WebMKS is not available when a VM is using a vSphere API less than version 6.
@miq-bot add_labels bug, compute/infrastructure, fine/yes
https://bugzilla.redhat.com/show_bug.cgi?id=1483936