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
CONSOLE-2371: [i18n] Externalize strings in olm root components #7497
CONSOLE-2371: [i18n] Externalize strings in olm root components #7497
Conversation
frontend/packages/operator-lifecycle-manager/src/components/clusterserviceversion.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/clusterserviceversion.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/create-catalog-source.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/install-plan.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/install-plan.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/subscription.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/subscription.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/subscription.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/subscription.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/subscription.tsx
Outdated
Show resolved
Hide resolved
ac6745b
to
55f37af
Compare
/retest |
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.
frontend/packages/operator-lifecycle-manager/locales/en/olm.json
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/locales/en/olm.json
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/catalog-source.tsx
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/catalog-source.tsx
Outdated
Show resolved
Hide resolved
/label docs-approved |
/retest |
frontend/packages/operator-lifecycle-manager/locales/en/olm.json
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/clusterserviceversion.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/install-plan.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/package-manifest.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/subscription.tsx
Outdated
Show resolved
Hide resolved
b55a236
to
77b6517
Compare
@spadgett I've addressed your comments |
frontend/packages/operator-lifecycle-manager/locales/en/olm.json
Outdated
Show resolved
Hide resolved
/label px-approved |
/label px-approved |
77b6517
to
d7ed2ac
Compare
f7f8555
to
e67ffe0
Compare
@spadgett rebased. Needs lgtm again. |
@TheRealJon: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
e67ffe0
to
b95a54d
Compare
frontend/packages/operator-lifecycle-manager/src/components/install-plan.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/install-plan.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/operator-install-page.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/operator-install-page.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/operator-install-page.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/operator-lifecycle-manager/src/components/operator-install-page.tsx
Outdated
Show resolved
Hide resolved
b95a54d
to
0c0e6b2
Compare
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.
/lgtm
Thanks @TheRealJon !
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rhamilto, spadgett, TheRealJon 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 |
Externalize strings in files at root level of
frontend/packages/operator-lifecycle-manager/src/components