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

Hardcoded labels instead of i18n shortcodes #382

Open
1 of 4 tasks
fthobe opened this issue Nov 21, 2024 · 8 comments · Fixed by #384
Open
1 of 4 tasks

Hardcoded labels instead of i18n shortcodes #382

fthobe opened this issue Nov 21, 2024 · 8 comments · Fixed by #384
Labels
bug Something isn't working

Comments

@fthobe
Copy link
Contributor

fthobe commented Nov 21, 2024

Not all relevant text strings have been integrated with i18 properly.

Solidus Version:
N/A

To Reproduce

Current behavior

Following items need strings in the language master:

  • Open main menu and Close Menu or decide to substitute with more generic open and close as present in en string files
  • New (there's a new string used in a different context, we could use that one)
  • See all results

Expected behavior
Should use correct translation strings such as <%= I18n.t('spree.my_account') %>

Screenshots
N/A

Desktop (please complete the following information):
Not platform related

Smartphone (please complete the following information):
Not platform related

@fthobe
Copy link
Contributor Author

fthobe commented Nov 21, 2024

Moved into issue description

@benjaminwil
Copy link
Contributor

We were discussing this in the Solidus Slack.

I suggested that solidus_starter_frontend should generate a locales file with translation strings for any translations it requires that aren't included in core Solidus gems.

This is much better than hard coding the English strings into the views, as they are now.

@benjaminwil
Copy link
Contributor

Since solidus_starter_frontend is not a gem, these translations should be in the application namespace.

@jarednorman jarednorman changed the title Hardcoded labels instead of i18 shortcodes Hardcoded labels instead of i18n shortcodes Nov 21, 2024
@fthobe
Copy link
Contributor Author

fthobe commented Nov 22, 2024

Since solidus_starter_frontend is not a gem, these translations should be in the application namespace.

Hey, not having massive experience here, if you create one as an example I am willing to add the missing ones and do also German, Italian and Spanish. I have the feeling I will need it down the road.

kennyadsl added a commit that referenced this issue Nov 26, 2024
#382 Convert Hardcoded Text Strings to i18 variable
@benjaminwil
Copy link
Contributor

We should leave this open. There are others.

@kennyadsl kennyadsl reopened this Nov 26, 2024
@fthobe
Copy link
Contributor Author

fthobe commented Nov 26, 2024

@benjaminwil post them and I fix them! 😜

@benjaminwil
Copy link
Contributor

They seem to already be referenced in the initial message here.

@fthobe
Copy link
Contributor Author

fthobe commented Nov 26, 2024

Ok if you find more do not hesitate to comment here, I will fix them gradually where I find them

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants