Skip to content
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-3653: Adds French Localization #12994

Closed
wants to merge 31 commits into from
Closed

Conversation

fallmo
Copy link

@fallmo fallmo commented Jul 11, 2023

Adding French Localization to Console by:
Creating fr locale folders with JSON translation files for each package.
Including French in supportedLocales map

@spadgett @jhadvig

@openshift-ci openshift-ci bot added the component/ceph Related to ceph-storage-plugin label Jul 11, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 11, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fallmo
Once this PR has been reviewed and has the lgtm label, please assign jerolimov for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot requested a review from jhadvig July 11, 2023 13:11
@openshift-ci openshift-ci bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Jul 11, 2023
@jhadvig jhadvig changed the title Adds French Localization CONSOLE-3653: Adds French Localization Jul 12, 2023
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jul 12, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jul 12, 2023

@fallmo: This pull request references CONSOLE-3653 which is a valid jira issue.

In response to this:

Adding French Localization to Console by:
Creating fr locale folders with JSON translation files for each package.
Including French in supportedLocales map

@spadgett @jhadvig

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.

@jhadvig
Copy link
Member

jhadvig commented Jul 12, 2023

/retest

1 similar comment
@jhadvig
Copy link
Member

jhadvig commented Jul 13, 2023

/retest

@jhadvig
Copy link
Member

jhadvig commented Jul 25, 2023

There is an bug filed for this PR - https://issues.redhat.com/browse/OCPBUGS-16734

@fallmo
Copy link
Author

fallmo commented Jul 25, 2023

There is an bug filed for this PR - https://issues.redhat.com/browse/OCPBUGS-16734

Good catch, addressed it in the latest commit.

@XiyunZhao
Copy link

Hi @fallmo, there are other three issues were found for this PR, please help to take a look
OCPBUGS-16759, OCPBUGS-16761, OCPBUGS-16763

@fallmo
Copy link
Author

fallmo commented Jul 26, 2023

Hi @fallmo, there are other three issues were found for this PR, please help to take a look OCPBUGS-16759, OCPBUGS-16761, OCPBUGS-16763

Addressed in latest commits.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 26, 2023

@fallmo: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-gcp-console 0b6db93 link true /test e2e-gcp-console

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.

@fallmo
Copy link
Author

fallmo commented Aug 8, 2023

Hi @fallmo, there are other three issues were found for this PR, please help to take a look OCPBUGS-16759, OCPBUGS-16761, OCPBUGS-16763

Addressed in latest commits.

@jhadvig @XiyunZhao hello, any updates since?

@jhadvig
Copy link
Member

jhadvig commented Aug 8, 2023

@fallmo thank you for reaching out. We are waiting for the QE team to approve the change. Sorry for the delay.

@XiyunZhao
Copy link

sorry for the delay, all bug has been verified on pre-merge testing, all passed, and no other new issue was found
/label qe-approved

@openshift-ci openshift-ci bot added the qe-approved Signifies that QE has signed off on this PR label Aug 16, 2023
@fallmo
Copy link
Author

fallmo commented Aug 16, 2023

sorry for the delay, all bug has been verified on pre-merge testing, all passed, and no other new issue was found /label qe-approved

Great news!

@ymoisan
Copy link

ymoisan commented Aug 29, 2023

FR in OpenShift would be really nice :-)

@fallmo
Copy link
Author

fallmo commented Aug 29, 2023

FR in OpenShift would be really nice :-)

Agreed!

@fallmo
Copy link
Author

fallmo commented Sep 15, 2023

@jhadvig @spadgett hey there, any new bugs show up? What are the next steps?

@yanpzhan
Copy link
Contributor

@jhadvig this pr is still open, will it be include in 4.15?

@jhadvig
Copy link
Member

jhadvig commented Jan 8, 2024

Unfortunately we need to close this PR since there is no business requirement for this feature anymore.

@jhadvig jhadvig closed this Jan 8, 2024
@pittar
Copy link

pittar commented Feb 9, 2024

Every department in the Government of Canada is waiting on this feature - how can there be no business requirement?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ceph Related to ceph-storage-plugin component/core Related to console core functionality component/dev-console Related to dev-console component/git-service Related to git-service component/gitops Related to gitops-plugin component/helm Related to helm-plugin component/insights Related to insights plugin component/knative Related to knative-plugin component/kubevirt Related to kubevirt-plugin component/lso Related to local-storage-operator-plugin component/metal3 Related to metal3-plugin component/network-attachment-definition Related to network-attachment-definition component/olm Related to OLM component/pipelines Related to pipelines-plugin component/shared Related to console-shared component/topology Related to topology jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. kind/i18n Indicates issue or PR relates to internationalization or has content that needs to be translated ok-to-test Indicates a non-member PR verified by an org member that is safe to test. qe-approved Signifies that QE has signed off on this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants