Andras/BoundlessFlowCampus2K

Fork 0

Localization- change all printed text #861

New issue

Closed

opened 2026-05-11 07:48:36 +00:00 by c24danli · 6 comments

c24danli commented

2026-05-11 07:48:36 +00:00

Collaborator

In the test #830 it worked so now we will introduce localization to all files/ all printed text

In the locales dictionary we have a json file for en & sv.
When creating keys make sure that the keys are descriptive and pressent in both the files.
en.json

{
	"language": "Language"
}

sv.json

{
	"language": "Språk"
}

Right now we have one locales file as we dont have that much text to print but it can be a good idea to structure the files with comments to make it easier to break out if the need comes.
ex

{
    // Settings page
	"language": "Language"

    // tempratureView
    "tempTitle": "Temperature"
    "tempDescript": "This is recoreded by sesnosrs around campus"
}

The key is accessed with translate('key')

In the test #830 it worked so now we will introduce localization to all files/ all printed text ![Skärmbild 2026-05-11 085621](/attachments/86001268-23cb-4a51-81be-e23e18914eb6) In the locales dictionary we have a json file for en & sv. When creating keys make sure that the keys are descriptive and pressent in both the files. en.json ``` { "language": "Language" } ``` sv.json ``` { "language": "Språk" } ``` Right now we have one locales file as we dont have that much text to print but it can be a good idea to structure the files with comments to make it easier to break out if the need comes. ex ``` { // Settings page "language": "Language" // tempratureView "tempTitle": "Temperature" "tempDescript": "This is recoreded by sesnosrs around campus" } ``` ![Skärmbild 2026-05-11 093127](/attachments/06528f90-3b95-4358-b911-ac873183ca41) The key is accessed with translate('key')

Skärmbild 2026-05-11 085621.png

3.8 KiB

Skärmbild 2026-05-11 093127.png

36 KiB

c24danli added the

frontEnd

unassigned

labels

2026-05-11 07:48:36 +00:00

f21marfo was assigned by a24julot

2026-05-11 07:58:56 +00:00

a24julot removed the

unassigned

label

2026-05-11 07:59:02 +00:00

f21marfo commented

2026-05-11 10:07:37 +00:00

Collaborator

The documentation here is a little vague, so for future reference:

First, any and all pages that use the translate function, MUST have the following import declaration.
import { useLanguage } from '../features/useLanguage';
And any function invoking translate MUST fetch the following constant.
const { translate } = useLanguage();

Not doing this will generate an "undefined" error for translate, at build-time.

Additionally comments are NOT allowed inside JSON files, and will break them. Keys must also be separated by a comma ",".

The documentation here is a little vague, so for future reference: First, any and all pages that use the translate function, MUST have the following import declaration. `import { useLanguage } from '../features/useLanguage';` And any function invoking translate MUST fetch the following constant. `const { translate } = useLanguage();` Not doing this will generate an "undefined" error for translate, at build-time. Additionally comments are NOT allowed inside JSON files, and will break them. Keys must also be separated by a comma ",".

👍 1

f21marfo referenced this issue

2026-05-11 10:31:33 +00:00

A localization JSON standard is required. #880

f21marfo added reference #861-Localization-change-all-printed-text

2026-05-11 13:06:22 +00:00

f21marfo commented

2026-05-11 13:23:08 +00:00

Collaborator

I have completed localization work on the following pages.

Settings
Support
About us

These changes can be previewed in the corresponding branch #861, if anyone is curious, will be resuming work tomorrow.

I have completed localization work on the following pages. - Settings - Support - About us These changes can be previewed in the corresponding branch #861, if anyone is curious, will be resuming work tomorrow.

🎉 1

f21marfo referenced this issue

2026-05-12 10:19:48 +00:00

#861 First batch of localizations. #914

a24vinla referenced this issue

2026-05-12 11:07:33 +00:00

#861 First batch of localizations. #914

a22erigr referenced this issue from a commit

2026-05-12 11:32:09 +00:00

Merge pull request '#861 First batch of localizations.' (#914) from #861-Localization-change-all-printed-text into team_2_week_5

f21marfo referenced this issue

2026-05-13 13:12:28 +00:00

Report bug after merges. #936

f21marfo changed reference from ~~#861-Localization-change-all-printed-text~~ to #861-Localization-change-all-printed-text

2026-05-15 07:30:00 +00:00

f21marfo changed reference from ~~#861-Localization-change-all-printed-text~~ to #861-localization-change-all-printed-text-Part-2

2026-05-15 07:30:16 +00:00

b25marem referenced this issue from a commit

2026-05-15 08:56:05 +00:00

Merge pull request '#861 First batch of localizations.' (#914) from #861-Localization-change-all-printed-text into team_2_week_5

f21marfo commented

2026-05-15 10:39:17 +00:00

Collaborator

Update:

At present, I "believe" i have now localized everything visible post-merge, with the exception of the cards themselves. There are currently a number of issues open on them, and I think it best that the people working on structuring and re-organizing them, should be the ones to localize them, in order to minimize conflicts.

If this seems agreeable, i will go ahead and make a pull-request for this last batch of changes, and consider my work on this issue completed.

**Update:** At present, I "believe" i have now localized everything visible post-merge, with the exception of the cards themselves. There are currently a number of issues open on them, and I think it best that the people working on structuring and re-organizing them, should be the ones to localize them, in order to minimize conflicts. If this seems agreeable, i will go ahead and make a pull-request for this last batch of changes, and consider my work on this issue completed.

c24danli commented

2026-05-15 10:46:45 +00:00

Author

Collaborator

It seems as a good approach, when your pr is approved and merged will i add to the review template to check localization where we will hopefully get the once left and future instances discovered.

👍 1

f21marfo referenced this issue

2026-05-15 11:01:09 +00:00

#861 Final batch of localizations #970

a22erigr referenced this issue from a commit

2026-05-18 11:53:42 +00:00

Merge pull request '#861 Final batch of localizations' (#970) from #861-localization-change-all-printed-text-Part-2 into team_1_week_6