[LA-36685] Document missing supported language codes#52
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Jira
LA-36685
What
Adds five locales that the public API already accepts on the `language` field but were missing from the Supported Language Codes table in `_users.md`:
Each is inserted in its existing position in `I18n.available_locales` (`config/initializers/1_locale.rb`), so the documented table now matches the app's locale whitelist exactly (25 codes).
Why
The table claims to be the authoritative list of values the `language` field accepts on the Create/Update User endpoints, but these five are fully-translated, first-class locales in the app — they were simply overlooked (note `es-CO` Colombian Spanish was listed while plain `es` was not). Customers integrating against the API had no documented way to know `es`/`no`/`sv`/`da`/`sk` were valid.
Follows on from the Cezanne locale work (LA-36685): it covers the Danish (`da-DK` → `da`) and Norwegian Bokmål (`nb-NO` → `no`) cases from the recent locale request; Hungarian (`hu`) and Polish (`pl`) were already documented. Catalan (`ca-ES`) is intentionally not added — the app has no Catalan locale, so that needs platform work, not a docs change.
Screenshots
n/a — Markdown table change only.
Note
Low Risk
Markdown-only documentation update with no runtime or API changes.
Overview
Updates the Supported Language Codes table in
_users.mdso it lists five locales the Create/Update Userlanguagefield already accepts but were omitted from the docs:es,no,sv,da, andsk. Rows are placed in the same order as the app’s locale list so the table matches the full set of valid codes (25 total).No API or validation behavior changes—documentation only for integrators who rely on this table as the authoritative list.
Reviewed by Cursor Bugbot for commit ec8c589. Bugbot is set up for automated code reviews on this repo. Configure here.