Skip to content

[refactor / bugfix] language - sort JSON keys, find missing keys#709

Merged
accius merged 2 commits intoaccius:mainfrom
MichaelWheeley:feature/language-sort
Mar 12, 2026
Merged

[refactor / bugfix] language - sort JSON keys, find missing keys#709
accius merged 2 commits intoaccius:mainfrom
MichaelWheeley:feature/language-sort

Conversation

@MichaelWheeley
Copy link
Copy Markdown
Contributor

What does this PR do?

I have been working to add TH language (pending) however it is a moving target. Between languages there appeared to be an inconsistent sort of indexing of JSON keys making identification of new or missing keys difficult. For each language I have sorted alphabetically, and where missing keys occur added them using the 'en' language version as master.

Type of change

  • Bug fix
  • New feature
  • Performance improvement
  • Refactor / code cleanup
  • Documentation
  • Translation
  • Map layer plugin

How to test

  1. Performed quick validation by loading each language from settings menu and visually verifying language load occurs.

Checklist

  • App loads without console errors
  • Tested in Dark, Light, and Retro themes
  • Responsive at different screen sizes (desktop + mobile)
  • If touching server.js: caches have TTLs and size caps (we serve 2,000+ concurrent users)
  • If adding an API route: includes caching and error handling
  • If adding a panel: wired into Modern, Classic, and Dockable layouts
  • No hardcoded colors — uses CSS variables (var(--accent-cyan), etc.)
  • No .bak, .old, console.log debug lines, or test scripts included

Screenshots (if visual change)

(none)

@MichaelWheeley MichaelWheeley changed the title language - sort JSON keys, find missing keys [refactor / bugfix] language - sort JSON keys, find missing keys Mar 10, 2026
@accius accius merged commit d9efb88 into accius:main Mar 12, 2026
2 of 3 checks passed
@accius
Copy link
Copy Markdown
Owner

accius commented Mar 12, 2026

Please point your PR's to staging from now on.

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.

2 participants