Skip to content

docs(FR-2540): fix session type filters in session_page documentation#6578

Merged
graphite-app[bot] merged 1 commit intomainfrom
04-10-docs_fix_session_type_filters_in_session_page_documentation
Apr 17, 2026
Merged

docs(FR-2540): fix session type filters in session_page documentation#6578
graphite-app[bot] merged 1 commit intomainfrom
04-10-docs_fix_session_type_filters_in_session_page_documentation

Conversation

@nowgnuesLee
Copy link
Copy Markdown
Contributor

@nowgnuesLee nowgnuesLee commented Apr 10, 2026

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

Copy link
Copy Markdown
Contributor Author

nowgnuesLee commented Apr 10, 2026


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • flow:merge-queue - adds this PR to the back of the merge queue
  • flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Copy Markdown
Contributor Author

@nowgnuesLee nowgnuesLee left a comment

Choose a reason for hiding this comment

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

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.

@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_update_copy_button_location_in_my_environments_documentation branch from 23967a0 to cb80247 Compare April 10, 2026 09:25
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch from e729f78 to f68540a Compare April 10, 2026 09:25
@github-actions github-actions Bot added size:M 30~100 LoC and removed size:XS ~10 LoC labels Apr 10, 2026
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_update_copy_button_location_in_my_environments_documentation branch from cb80247 to a2ccc83 Compare April 10, 2026 09:33
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch from f68540a to e698b5e Compare April 10, 2026 09:33
@github-actions github-actions Bot added size:XS ~10 LoC and removed size:M 30~100 LoC labels Apr 10, 2026
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_update_copy_button_location_in_my_environments_documentation branch from a2ccc83 to 6520e17 Compare April 11, 2026 03:46
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch from e698b5e to 775f024 Compare April 11, 2026 03:46
@github-actions github-actions Bot added size:M 30~100 LoC and removed size:XS ~10 LoC labels Apr 11, 2026
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_update_copy_button_location_in_my_environments_documentation branch from 6520e17 to 934d9a9 Compare April 12, 2026 14:16
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch from 775f024 to 5e29246 Compare April 12, 2026 14:16
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch from 9c75e04 to 645fced Compare April 14, 2026 05:45
@nowgnuesLee nowgnuesLee changed the title docs: fix session type filters in session_page documentation docs(FR-2540): fix session type filters in session_page documentation Apr 14, 2026
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch from 645fced to c12bf4b Compare April 15, 2026 07:08
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_update_copy_button_location_in_my_environments_documentation branch from 14026d1 to 945764f Compare April 15, 2026 07:08
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch from c12bf4b to 6468147 Compare April 15, 2026 07:38
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_update_copy_button_location_in_my_environments_documentation branch from 945764f to 378e887 Compare April 15, 2026 07:38
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch 2 times, most recently from b3e8e0a to bf4254b Compare April 16, 2026 09:36
@nowgnuesLee nowgnuesLee force-pushed the 04-10-docs_update_copy_button_location_in_my_environments_documentation branch from 3b20c1e to 8528fd6 Compare April 16, 2026 09:36
@agatha197 agatha197 force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch from bf4254b to 92406fb Compare April 16, 2026 14:47
@agatha197 agatha197 force-pushed the 04-10-docs_update_copy_button_location_in_my_environments_documentation branch from 8528fd6 to 7be42e6 Compare April 16, 2026 14:47
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Apr 17, 2026

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)
@graphite-app graphite-app Bot force-pushed the 04-10-docs_update_copy_button_location_in_my_environments_documentation branch from 7be42e6 to db98be4 Compare April 17, 2026 10:57
@graphite-app graphite-app Bot force-pushed the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch from 92406fb to a1eeb49 Compare April 17, 2026 10:58
Base automatically changed from 04-10-docs_update_copy_button_location_in_my_environments_documentation to main April 17, 2026 11:01
@graphite-app graphite-app Bot merged commit a1eeb49 into main Apr 17, 2026
4 checks passed
@graphite-app graphite-app Bot deleted the 04-10-docs_fix_session_type_filters_in_session_page_documentation branch April 17, 2026 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30~100 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: Fix session type filters in session_page documentation (FR-2540)

3 participants