Skip to content

feat: implement internationalization (GECO-125)#380

Open
lzueva wants to merge 14 commits into
mainfrom
implement-internationalization
Open

feat: implement internationalization (GECO-125)#380
lzueva wants to merge 14 commits into
mainfrom
implement-internationalization

Conversation

@lzueva
Copy link
Copy Markdown
Contributor

@lzueva lzueva commented May 27, 2026

close #118

@lzueva lzueva self-assigned this May 27, 2026
@lzueva lzueva requested a review from choffmann as a code owner May 27, 2026 06:38
@choffmann choffmann changed the title feat: implement internationalization feat: implement internationalization (GECO-125) May 27, 2026
@lzueva lzueva force-pushed the implement-internationalization branch from 99b0730 to 33697cb Compare May 27, 2026 09:19
Copy link
Copy Markdown
Member

@choffmann choffmann left a comment

Choose a reason for hiding this comment

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

The lang="de" key is hardcoded in index.html. Please remove it.

The release Markdown files are not translated. In this case, it makes sense to create separate Markdown files, e.g. v0.1.2.de.md and v0.1.2.en.md

The feature is not yet complete. We need to implement a language switcher before merging this PR.

Comment thread src/i18n/config.ts

void i18next.use(initReactI18next).init({
lng: 'de',
debug: true,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please set debug in production to false

Comment on lines 115 to 134
@@ -129,8 +123,7 @@ function Faq() {
>
Hochschule Flensburg
</a>
&nbsp;ins Leben gerufen. Nach dem erfolgreichen Abschluss des Forschungsprojekts wird
die Weiterentwicklung des Systems nun von&nbsp;
&nbsp;{t('sections.faq.items.behind.answer.beforeProgeek')}&nbsp;
<a
href="https://progeek.de/"
target="_blank"
@@ -139,7 +132,7 @@ function Faq() {
>
PROGEEK
</a>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please do not split sentence at link boundary. Use the <Trans> component with named placeholders like this:

<Trans
    i18nKey="sections.faq.items.behind.answer"
    components={{ hs: <a href="https://hs-flensburg.de/" .../>, pg: <a href="https://progeek.de/" .../> }}
  />
 "Hinter dem Projekt stehen die <hs>Hochschule Flensburg</hs>, <pg>PROGEEK</pg>, …".

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do not translate Datenschutzerklärung

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do not translate Impressum

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The translation files are huge. Please consider namespacing them with the i18next namespace feature

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.

Add localization

2 participants