docs: document the preferred language setting - #583
Merged
Conversation
abdulrafey1
force-pushed
the
refey/docs/lang-05-guide
branch
2 times, most recently
from
August 10, 2026 07:32
6bc11c6 to
c5b20de
Compare
abdulrafey1
force-pushed
the
refey/docs/lang-05-guide
branch
from
August 10, 2026 09:26
c5b20de to
bd99045
Compare
hamza-56
approved these changes
Aug 10, 2026
abdulrafey1
force-pushed
the
refey/docs/lang-05-guide
branch
from
August 11, 2026 23:23
bd99045 to
6167acc
Compare
abdulrafey1
force-pushed
the
refey/docs/lang-05-guide
branch
from
August 11, 2026 23:31
6167acc to
790cd90
Compare
abdulrafey1
force-pushed
the
refey/docs/lang-05-guide
branch
from
August 12, 2026 06:37
790cd90 to
188a52d
Compare
Covers the supported list, how to read and set a user's choice, what null means, and what happens to a stored tag once its language leaves the allowlist. The guide is explicit that generated content does not consume the preference yet, so nothing here promises behaviour the platform does not have. The DEFAULT_LANGUAGE reference entry ships with the setting itself, so this change is guide-only. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Follows the endpoint dropping its auth gate on #582: documenting a Bearer header the endpoint no longer wants would send readers looking for a token before they can render a sign-in page in the right language. The /user/me examples keep theirs — those do need one. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
abdulrafey1
force-pushed
the
refey/docs/lang-05-guide
branch
from
August 12, 2026 08:29
f0004b3 to
4498dae
Compare
abdulrafey1
added a commit
that referenced
this pull request
Aug 12, 2026
#582 and #583 landed on main as squash commits while this branch carried their original commits, so git saw the same files added on both sides and flagged six conflicts where there was one real question: which copy is current. main's is. Its copies include the review fixes made after this branch was cut — the languages endpoint dropped its auth gate, and the User schema moved back to the root sparkth/schemas.py — so every file this branch does not own is resolved to main's version, and the branch keeps only what it exists to change: the api_router assembly moving from sparkth/api/v1/api.py into the package __init__. Two of those needed more than a side chosen: - api.py was modified in main (#582 registered the language router) and deleted here. Kept deleted; the nine include_router calls in __init__.py were verified identical to main's, prefixes and tags included, so the deletion drops no route. - architectural_patterns.md is a union: main's amended shared-models rule, which now names User, plus this branch's rename of api.py to the package __init__. The re-export bullet is reworded because the rename leaves two different __init__.py roles in one section. auth.py auto-merged without a conflict and silently kept this branch's pre-fix import, dropping main's. Restored from main. Verified by diffing the whole tree against main: no file outside the four this branch owns differs from it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
2 participants
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.
What
Documents the preferred-language setting for users and administrators.
Changes
How to Test
make docs— builds with no warnings.site/guides/user-management/index.htmland confirm the new section renders and its link to the configuration reference resolves to theDEFAULT_LANGUAGEanchor.Notes
Documentation only, no code. The guide covers the supported list, reading and setting a user's choice, clearing it with an explicit
null, the exact case-sensitive matching, and what happens to a stored tag that later leaves the allowlist.It states plainly that AI-generated content does not use the preference — the setting is recorded and exposed for clients to read.
This description was written with the assistance of an LLM (Claude).