Skip to content

I18n sync #3523

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

Merged
merged 6 commits into from
Jun 24, 2025
Merged

I18n sync #3523

merged 6 commits into from
Jun 24, 2025

Conversation

ojuschugh1
Copy link
Contributor

@ojuschugh1 ojuschugh1 commented Jun 12, 2025

Progress Towards #3510

Changes:
Here's a summary of the changes made:
Replaced "SubmitSaveAllSettings" with "SaveAccountDetails" in Hindi locale file.
Updated "Settings" to "My Account" (or its translation) in the AccountView section of the Hindi locale file.
The translations are now consistent across the Hindi language and match the English version. The UI should now display the correct text in Hindi.

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #3510

Copy link
Contributor

@yugalkaushik yugalkaushik left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Collaborator

@raclim raclim left a comment

Choose a reason for hiding this comment

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

Thanks so much @ojuschugh1 for your work on this and @yugalkaushik for the review, this is great!

As a quick note, I updated the description as "Progress Towards" instead of "Fixes" so the issue doesn't close, since we have a few other translation files that still need to be addressed.

@raclim raclim merged commit 7200584 into processing:develop Jun 24, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants