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
More i18n for vm details page #7250
More i18n for vm details page #7250
Conversation
translation file is created in public directory instead of in our local plugin, set to WIP until this is cleared out. |
604357a
to
47cbbbe
Compare
/test e2e-gcp-console |
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.
There are strings in a few files that I think should to be translated, but were in sections that didn't allow me to make per-line comments. I've included the files and lines below.
kubevirt-plugin/src/components/modals/scheduling-modals/dedicated-resources-modal/dedicated-resources-modal.tsx
Line numbers: 75, 85, 98, 102, 105
frontend/packages/kubevirt-plugin/src/components/modals/scheduling-modals/tolerations-modal/tolerations-modal.tsx
Line numbers: 111, 115, 118, 128, 129
frontend/packages/kubevirt-plugin/src/components/vm-templates/vm-template-resource.tsx
Line numbers: 48, 52, 61, 66, 74, 94,
@pcbailey +1 thanks |
e24dffd
to
de329c1
Compare
removed WIP, the json file is back in the plugin dir, |
4c69456
to
6463aa6
Compare
6463aa6
to
18efa51
Compare
@pcbailey added the missing translations +1, please re-review |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pcbailey, yaacov The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
More i18n for virtualization -> virtual machines -> details page