Skip to content

Edge UI: Use browser Language as Default#3415

Merged
sfeilmeier merged 11 commits intoOpenEMS:developfrom
da-Kai:feature/local-ui-browser-language
Mar 1, 2026
Merged

Edge UI: Use browser Language as Default#3415
sfeilmeier merged 11 commits intoOpenEMS:developfrom
da-Kai:feature/local-ui-browser-language

Conversation

@da-Kai
Copy link
Copy Markdown
Contributor

@da-Kai da-Kai commented Nov 14, 2025

  • Fallback to browser language when AuthenticateResponse provides no language.
  • Centralize logic for retrieving the current language
  • Change default Language from de to en
  • Replace Fenecon specific no local history notice with a generic OpenEMS version.
  • Set language explicitly to de in Karma tests.

Address language issue in #3411

@da-Kai da-Kai requested review from Copilot and lukasrgr November 14, 2025 20:36
@da-Kai da-Kai changed the title Edge UI: User browser Language as Default Edge UI: Use browser Language as Default Nov 14, 2025

This comment was marked as resolved.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 14, 2025

Codecov Report

❌ Patch coverage is 22.22222% with 21 lines in your changes missing coverage. Please review.

❌ Your patch status has failed because the patch coverage (22.23%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #3415      +/-   ##
=============================================
- Coverage      58.60%   58.59%   -0.01%     
  Complexity       105      105              
=============================================
  Files           3091     3092       +1     
  Lines         134005   134011       +6     
  Branches        9882     9871      -11     
=============================================
- Hits           78523    78512      -11     
- Misses         52583    52591       +8     
- Partials        2899     2908       +9     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

…-browser-language

# Conflicts:
#	ui/src/assets/i18n/de.json
#	ui/src/assets/i18n/en.json
Copy link
Copy Markdown
Contributor

@lukasrgr lukasrgr left a comment

Choose a reason for hiding this comment

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

Thx for this incremental improvement

@da-Kai da-Kai requested a review from lukasrgr December 1, 2025 12:56
@da-Kai da-Kai requested a review from sfeilmeier February 25, 2026 18:36
"YESTERDAY": "Gestern"
"YESTERDAY": "Gestern",
"NO_LOCAL_DATA_INFO": "Historische Daten sind derzeit lokal nicht verfügbar",
"NO_LOCAL_DATA_NOTICE": "Bitte besuchen Sie https://github.com/OpenEMS/openems für Updates zu diesem Thema."
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There should probably be a link in OpenEMS environment for this case and use a variable here.

@sfeilmeier sfeilmeier merged commit 237b3d8 into OpenEMS:develop Mar 1, 2026
6 of 7 checks passed
sfeilmeier added a commit that referenced this pull request Mar 12, 2026
Since #3415 forwarding to Live did not work on local UI

Reviewed-by: Lukas Rieger <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants