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

[Admin] Missing translations: Page titles 'Overview', 'Properties', 'Product Import', 'Subscriptions', 'General Settings', 'Enterprise Roles' #12372

Closed
drummer83 opened this issue Apr 12, 2024 · 7 comments · Fixed by #12454
Assignees
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. good first issue

Comments

@drummer83
Copy link
Contributor

Description

The page titles of the following admin pages are always shown in English and they are not translated to the selected locale.

  • 'Overview'
  • 'Properties'
  • 'Product Import'
  • 'Subscriptions'
  • 'General Settings'
  • 'Enterprise Roles'

Expected Behavior

All page titles should be shown in the selected locale.

Actual Behaviour

The listed page titles above are always shown in English - no matter what locale is selected by the user.

Steps to Reproduce

  1. Log in as super admin (to see all of the pages above).
  2. Visit the listed pages.
  3. Check the page title in the browser.

Animated Gif/Screenshot

image

Workaround

None.

Severity

bug-s3: a feature is broken but there is a workaround

Your Environment

  • Version used:
  • Browser name and version:
  • Operating System and version (desktop or mobile):

Possible Fix

@drummer83 drummer83 added bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. good first issue labels Apr 12, 2024
@arunguleria
Copy link
Contributor

Hi @drummer83, I would like to work on this.

@drummer83
Copy link
Contributor Author

Sure, @arunguleria, I will assign it to you!
Thank you and let us know if you run into any issues.

@drummer83
Copy link
Contributor Author

Hi @arunguleria,
I just found that there are even more pages with missing translations of the page titles in the super admin area, e.g. Taxons and Editing Taxons.
If you find any and you want to fix them as well, go ahead please. It's also fine if you just fix the listed ones above.
Thanks again!

@arunguleria
Copy link
Contributor

Hi @drummer83 , Sure I'll check missing translations

@arunguleria
Copy link
Contributor

Hi @drummer83 , I checked and fixed the issue but I have one query here. I found in the tab we're using Import name but on browser title we're using controller name. Please let me know can we add controller name in translation file.
Please check the below screenshot.

Screenshot 2024-04-30 at 1 09 52 PM

@drummer83
Copy link
Contributor Author

@openfoodfoundation/core-devs Could you support please?
Thank you! 🤗

@dacook
Copy link
Member

dacook commented May 1, 2024

Hi @arunguleria , these are generated from the controller class name (here), and may be translated by adding a key of the same name. The keys are lowercase with underscores between words, for example:

  • Products -> products
  • General Settings -> general_settings

They will probably need to be placed under the Spree section in the default locale en.yml.
Once they are all there, translators will be able to update for each locale in Transifex later.

Hmm, spree.general_settings is already there, in en and also de_DE for example: https://github.com/openfoodfoundation/openfoodnetwork/blob/master/config/locales/en.yml#L3983
https://github.com/openfoodfoundation/openfoodnetwork/blob/master/config/locales/de_DE.yml#L3752

But I can see that some others, like enterprise_roles are not there. So that will need adding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. good first issue
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants