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

feat(locales): update translations files #1421

Merged

Conversation

lavanya-bmw
Copy link
Contributor

@lavanya-bmw lavanya-bmw commented Jan 7, 2025

Description

Unused translations for locales files are removed and missing translations are added.

Changelog:

  • Locales
    • unused translations for locales files are removed and missing translations are added #1421

Why

Unused translations for locales files to be removed and missing translations to be added

Issue

#1419

Checklist

Please delete options that are not relevant.

  • I have followed the contributing guidelines
  • I have performed a self-review of my own code
  • I have successfully tested my changes locally

Copy link

sonarqubecloud bot commented Jan 7, 2025

@@ -479,7 +481,6 @@
"message": "The Privacy Policy states which data the business applications collects, handles and processes. The details below explicitly show which company / user information are getting collected/stored.",
"notSupportedMessage": "Currently not yet supported",
"COMPANY_DATA": "Company Data",
"COMPANY_DATA_ICON": "ApartmentIcon",
"USER_DATA": "User Data",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some wrong practice has been introduced here. I know this is not part of the PR. Could you check as it is a small fix?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@charmi-v guideline says to use camelCase here

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for highlighting this, @manojava-gk. The issue arose during the implementation of the new design for the company subscription details page. I referenced the same file (locales/en/main.json), utilizing existing key usages such as REGISTRATION_VERIFICATION and IN_PROGRESS to validate it directly as an enum.

Currently, COMPANY_DATA_ICON is accurately mapping icons based on the enum key check.

I will review and consider refactoring this in alignment with the suggested guidelines shortly.

@oyo oyo merged commit e68b81c into eclipse-tractusx:main Jan 13, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: USER READY
Development

Successfully merging this pull request may close these issues.

4 participants