docs(FR-2540): fix session type filters in session_page documentation#6578
Conversation
nowgnuesLee
left a comment
There was a problem hiding this comment.
Review: PR #6578 — docs: fix session type filters in session_page documentation
Summary
The PR correctly adds the missing All filter tab but incorrectly replaces Upload Sessions with System. The i18n key is session.System, but the actual label users see in the UI is "Upload Sessions". Documentation should match the displayed UI labels, not the internal key names.
Findings
All 4 files (en, ja, ko, th)
Correctness (Must Fix): The label System does not match the UI. The tab uses t('session.System') which resolves to:
- EN: "Upload Sessions"
- KO: "업로드 세션"
- JA: "アップロードセッション"
- TH: "เซสชันอัปโหลด"
Per the Translation Guide: "Documentation must use the same UI labels that users actually see in the application." Please verify against resources/i18n/{lang}.json.
The correct filter list in EN should be:
`All`, `Interactive`, `Batch`, `Inference`, or `Upload Sessions`
The old documentation was only missing All -- the Upload Sessions label was already correct and should not have been changed to System.
Reference: react/src/pages/ComputeSessionListPage.tsx lines 362-368 and resources/i18n/en.json key session.System.
Recommendation
Request changes -- The All tab addition is correct, but "System" should remain "Upload Sessions" (the user-visible label). Same applies across all 4 language files.
23967a0 to
cb80247
Compare
e729f78 to
f68540a
Compare
cb80247 to
a2ccc83
Compare
f68540a to
e698b5e
Compare
a2ccc83 to
6520e17
Compare
e698b5e to
775f024
Compare
6520e17 to
934d9a9
Compare
775f024 to
5e29246
Compare
9c75e04 to
645fced
Compare
645fced to
c12bf4b
Compare
14026d1 to
945764f
Compare
c12bf4b to
6468147
Compare
945764f to
378e887
Compare
b3e8e0a to
bf4254b
Compare
3b20c1e to
8528fd6
Compare
bf4254b to
92406fb
Compare
8528fd6 to
7be42e6
Compare
Merge activity
|
…#6578) Resolves #6684 (FR-2540) ## Summary - Fix incorrect session type filter label in the session page documentation: **Upload Sessions** (not "System" or "Batch") - Rewrite the JA and TH session_page filter section with correct i18n label translations verified against `resources/i18n/` files - Apply reviewer fixes to KO documentation ## Changes - `session_page/session_page.md` (en/ko/ja/th): corrected session type filter labels ## Test plan - [ ] Verify "Upload Sessions" filter label matches the actual UI in all languages - [ ] Verify JA/TH translations match `resources/i18n/ja.json` and `resources/i18n/th.json` 🤖 Generated with [Claude Code](https://claude.com/claude-code)
7be42e6 to
db98be4
Compare
92406fb to
a1eeb49
Compare

Resolves #6684 (FR-2540)
Summary
resources/i18n/filesChanges
session_page/session_page.md(en/ko/ja/th): corrected session type filter labelsTest plan
resources/i18n/ja.jsonandresources/i18n/th.json🤖 Generated with Claude Code