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

Fix i18n script #1285

Merged
merged 1 commit into from
May 2, 2023
Merged

Conversation

upalatucci
Copy link
Member

@upalatucci upalatucci commented May 2, 2023

📝 Description

There was something wrong with translations.
the script for no reason adds inexistence keys like.

{
"showFailed": "showFailed",
"showSuccess": "showSuccess"
}

just use i18next which will load automatically our i18next configuration file and restrict the research to the src folder and the console-extensions.json file

@openshift-ci openshift-ci bot requested review from avivtur and metalice May 2, 2023 08:27
@openshift-ci openshift-ci bot added the approved This issue is something we want to fix label May 2, 2023
@upalatucci upalatucci force-pushed the fix-i18n-script branch 2 times, most recently from dab2ea8 to 3ce40e0 Compare May 2, 2023 08:49
@metalice
Copy link
Member

metalice commented May 2, 2023

/lgtm

@openshift-ci openshift-ci bot added the lgtm Passed code review, ready for merge label May 2, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 2, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: metalice, upalatucci

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:
  • OWNERS [metalice,upalatucci]

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

@openshift-merge-robot openshift-merge-robot merged commit 792f130 into kubevirt-ui:main May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved This issue is something we want to fix lgtm Passed code review, ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants