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

Internationalization of WebApp Elements #270

Merged
merged 25 commits into from
Apr 10, 2024
Merged

Conversation

PabloGOP
Copy link
Contributor

@PabloGOP PabloGOP commented Apr 10, 2024

Status

As of now, I've updated every page and test except for:

  • Instructions
  • Homepage

As these sections require special attention as they utilize data from an external source. I'll be addressing them promptly.

How to use

Pages with Internationalization:

  • Use t("key_of_jsons_message") to insert strings.
  • Ensure key_of_jsons_message is correctly set in all webapp/src/localize/*.json files.

Pages without Internationalization:

  • Import useTranslation.
  • Define t in the component: const { t } = useTranslation();
  • Refer to the previous entry for usage.

@PabloGOP PabloGOP added enhancement🆕 New feature or request webapp🌐 Related to the front-end labels Apr 10, 2024
@PabloGOP PabloGOP added this to the Deliverable IV milestone Apr 10, 2024
@PabloGOP PabloGOP self-assigned this Apr 10, 2024
@PabloGOP PabloGOP mentioned this pull request Apr 10, 2024
Copy link

sonarcloud bot commented Apr 10, 2024

@PabloGOP PabloGOP changed the title Internacionalization of WebApp Elements Internationalization of WebApp Elements Apr 10, 2024
Copy link
Contributor

@and1na and1na left a comment

Choose a reason for hiding this comment

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

Good work Pablo!

@PabloGOP PabloGOP merged commit b80c556 into develop Apr 10, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement🆕 New feature or request webapp🌐 Related to the front-end
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants