Andras/BoundlessFlowCampus2K

Fork 0

Fix localization context and language selector #830

Merged

a22erigr merged 1 commit from 772-fix-localization-context into team_2_week_5

2026-05-11 07:01:53 +00:00

b25marem commented

2026-05-08 07:24:22 +00:00

Collaborator

This PR fixes the localization setup using React Context.

Changes made:

Enabled LanguageProvider in App.js
Added the LanguageSelect component to the Settings page
Connected the language dropdown to the localization context
Verified that language selection updates correctly and persists using localStorage

The localization system is now functional for English and Swedish.

This PR fixes the localization setup using React Context. Changes made: Enabled LanguageProvider in App.js Added the LanguageSelect component to the Settings page Connected the language dropdown to the localization context Verified that language selection updates correctly and persists using localStorage The localization system is now functional for English and Swedish.

b25marem added 1 commit

2026-05-08 07:24:23 +00:00

Fix localization context and language selector d7d5dd61f0

b25marem added the

needs review

label

2026-05-08 07:24:44 +00:00

c24danli commented

2026-05-08 13:25:01 +00:00

Collaborator

Add original issue #xxx in name to easier traverse forgejo and traceability

👍 1

a22erigr requested review from c22abdal

2026-05-09 11:31:22 +00:00

c22abdal commented

2026-05-09 12:01:50 +00:00

Collaborator

The dropdown now exists in the settings page, it works, showing both Swedish and English in the dropdown. However, my concern:
Should the selected language apply the localization to the entire website in this PR?
- Because it does not, it just changes the 'language' to 'språk' in the settings page.

If not, then this PR is mergeable.

Code is clean and minimal.

![GIF 2026-05-09 13-42-53](/attachments/2d85d2dd-6caf-4e98-a7a1-e050d3815299) The dropdown now exists in the settings page, it works, showing both Swedish and English in the dropdown. However, my concern: Should the selected language apply the localization to the entire website in this PR? - Because it does not, it just changes the 'language' to 'språk' in the settings page. If not, then this PR is mergeable. Code is clean and minimal.

GIF 2026-05-09 13-42-53.gif

38 KiB

c22abdal added

reviewed

and removed

needs review

labels

2026-05-09 12:01:56 +00:00

c24danli commented

2026-05-11 06:31:39 +00:00

Collaborator

@c22abdal wrote in #830 (comment):

The dropdown now exists in the settings page, it works, showing both Swedish and English in the dropdown. However, my concern: Should the selected language apply the localization to the entire website in this PR? - Because it does not, it just changes the 'language' to 'språk' in the settings page.

If not, then this PR is mergeable.

Code is clean and minimal.

This was just a smaller test we will impliment it on the website as a whole in the next issue

@c22abdal wrote in https://git.webug.se/Andras/BoundlessFlowCampus2K/pulls/830#issuecomment-9232: > [![GIF 2026-05-09 13-42-53](/attachments/2d85d2dd-6caf-4e98-a7a1-e050d3815299)](/Andras/BoundlessFlowCampus2K/attachments/2d85d2dd-6caf-4e98-a7a1-e050d3815299) > > The dropdown now exists in the settings page, it works, showing both Swedish and English in the dropdown. However, my concern: Should the selected language apply the localization to the entire website in this PR? - Because it does not, it just changes the 'language' to 'språk' in the settings page. > > If not, then this PR is mergeable. > > Code is clean and minimal. This was just a smaller test we will impliment it on the website as a whole in the next issue

a22erigr merged commit 4e6398b99c into team_2_week_5

2026-05-11 07:01:53 +00:00

a22erigr referenced this pull request from a commit

2026-05-11 07:01:55 +00:00

Merge pull request 'Fix localization context and language selector' (#830) from 772-fix-localization-context into team_2_week_5