Skip to content

Conversation

@ebinbellini
Copy link

  • Added translation to all text strings on the website using i18n
  • Language can be changed using the button in the navbar
  • The translation strings are in en.json and sv.json in
    app/i18n/locales
  • Added locale adjusted date handling in utils/dateFormat.ts

- Added translation to all text strings on the website using i18n
- Language can be changed using the button in the navbar
- The translation strings are in en.json and sv.json in
  /app/i18n/locales
- Added locale adjusted date handling
@ebinbellini ebinbellini changed the base branch from main to dev January 19, 2026 12:54
@ebinbellini ebinbellini changed the base branch from dev to feature/login-page January 19, 2026 12:55
@ebinbellini ebinbellini requested review from SebbeBlom and ankurr0y and removed request for ankurr0y January 19, 2026 12:55
Copy link

@SebbeBlom SebbeBlom left a comment

Choose a reason for hiding this comment

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

The added translations are correct.

Translations of error messages on for the log-in page are strictly in swedish. See image:
Skärmbild 2026-01-21 210348

Possibly add the selected language in a cookie so to save the preferred language on a page refresh.

Was not able to access the account page as a logged in user so that has not been reviewd.

Otherwise this looks like a nice solution to translation.

Your thoughts?

@ebinbellini
Copy link
Author

Good idea, I implemented the cookie so now the selected language persists across page reloads.

The error for the email input in your screenshot is actually built in to the browser, so it uses the user's preferred language from the browser settings. That's why it looks completely different for me, for example:
image

@ebinbellini ebinbellini linked an issue Jan 22, 2026 that may be closed by this pull request
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.

Translation (SWE/ENG)

3 participants