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

fix: refetch settings bundle on locale change #12074

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

LukasHirt
Copy link
Collaborator

Description

To make sure that the server returns settings bundle strings in the correct locale, refetch it after a locale is changed via the UI.

Related Issue

Motivation and Context

Correct translations

How Has This Been Tested?

  • test environment: chrome & 🤖
  • test case 1: switch locale in the account settings
  • test case 2: added unit tests

Screenshots (if appropriate):

Zaznam.obrazovky.2025-01-03.v.11.03.38.mov

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests
  • Documentation
  • Maintenance (e.g. dependency updates or tooling)

To make sure that the server returns settings bundle strings in the correct locale, refetch it after a locale is changed via the UI.
@LukasHirt LukasHirt added the Type:Bug Something isn't working label Jan 3, 2025
@LukasHirt LukasHirt requested a review from saw-jan January 3, 2025 10:13
@LukasHirt LukasHirt self-assigned this Jan 3, 2025
@LukasHirt LukasHirt enabled auto-merge January 3, 2025 10:13
Copy link

sonarqubecloud bot commented Jan 3, 2025

@LukasHirt LukasHirt merged commit 784c246 into master Jan 3, 2025
4 checks passed
@LukasHirt LukasHirt deleted the fix/settings-bundle-locale-change branch January 3, 2025 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:Bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Notifications Settings don't refresh when language is changed
2 participants