docs(FR-2536): add Start From URL and announcement banner to start page docs#6574
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via 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. |
nowgnuesLee
left a comment
There was a problem hiding this comment.
Review: PR #6574 — docs: add Start From URL and announcement banner to start page docs
Summary
Good documentation update that adds missing content for Start From URL, AnnouncementAlert, and card rearrangement. Structure follows the style guide well. A few UI label mismatches against i18n files need fixing.
Findings
[File: packages/backend.ai-webui-docs/src/ko/start/start.md]
[i18n Completeness] Should Fix: The label 폴더 사용 모드 does not match the actual UI label. The i18n file (ko.json) uses "VFolderUsageMode": "폴더 사용 방식". Please use 폴더 사용 방식 to match the UI.
Affected lines:
- "필요한 경우 폴더 사용 모드(일반 또는 모델)를 설정합니다" (appears twice, in GitHub and GitLab sections)
[File: packages/backend.ai-webui-docs/src/ja/start/start.md]
[i18n Completeness] Should Fix: The label フォルダ使用モード does not match the actual UI label. The i18n file (ja.json) uses "VFolderUsageMode": "フォルダ利用モード". Please use フォルダ利用モード to match the UI.
Affected lines:
- "必要に応じてフォルダ使用モード(一般またはモデル)を設定します" (appears twice, in GitHub and GitLab sections)
[File: packages/backend.ai-webui-docs/src/th/start/start.md]
[i18n Completeness] Should Fix: In the GitHub section (step 2), Storage Host is left untranslated. The i18n file (th.json) uses "StorageHost": "โฮสต์ที่เก็บข้อมูล". Please use the Thai label consistently. The GitLab section (step 3) has the same issue.
[i18n Completeness] Nice to Have: ชื่อ Branch ของ GitLab slightly differs from the UI label "GitlabDefaultBranch": "ชื่อ Branch GitLab". Consider dropping "ของ" to match the UI exactly.
[File: packages/backend.ai-webui-docs/src/en/start/start.md]
[Correctness] Nice to Have: The "Customizing Card Layout" section says "Each card has a drag handle at the top-left corner that you can grab to move the card." The old text said "the button at the top-left corner." Could you verify whether the current UI uses a drag handle or a button? The description should match the actual interaction.
Recommendation
Approve with suggestions — The label mismatches (KO 폴더 사용 방식, JA フォルダ利用モード, TH โฮสต์ที่เก็บข้อมูล) should be fixed to match the i18n files before merge. The rest of the content is well-structured and follows the documentation style guide.
0c72ae9 to
e2540db
Compare
1d61e8f to
0a8281b
Compare
e2540db to
2bd36bf
Compare
0a8281b to
5b9a0a9
Compare
5b9a0a9 to
b68cd52
Compare
82073e4 to
15709e3
Compare
15709e3 to
f16e1b9
Compare
7891fa2 to
cd1403c
Compare
cd1403c to
f989516
Compare
f16e1b9 to
a633fae
Compare
f989516 to
f32e9ce
Compare
a633fae to
9d148c9
Compare
f32e9ce to
da8a5f6
Compare
9d148c9 to
38a0fdf
Compare
38a0fdf to
fd1912f
Compare
This reverts commit d3ca954. #6575 was merged with base=main because `gt track --parent main` on a locally untracked branch caused Graphite to record main as its parent. The subsequent `gt submit --publish --force` pushed this base change to GitHub, and the PR was then squash merged, collapsing 29 commits from 6 stacked PRs into a single commit on main: - #6570 docs(FR-2532): share_vfolder labels - #6571 docs(FR-2533): user_settings - #6572 docs(FR-2534): 2FA/TOTP/SSO login - #6573 docs(FR-2535): header session timer & menu - #6574 docs(FR-2536): Start From URL & announcement banner - #6575 docs(FR-2537): dashboard board customization & auto-refresh Reverting so each PR can be merged in its original stack order. After this revert merges: - #6575 will be reopened with its original base (#6574's branch) - #6625's base will be restored to #6611's branch - The other open PRs (#6570, #6571, #6572, #6573, #6574) retain their original bases and will be mergeable in stack order
…docs Resolves #6681 (FR-2537) ## Summary - Document **board customization** via drag-and-drop on the Dashboard page - Document the **auto-refresh** toggle and 15-second refresh interval - Remove references to the Superadmin Dashboard per reviewer feedback (feature is hidden in current UI) ## Changes - `dashboard/dashboard.md` (en/ko/ja/th): board customization section, auto-refresh note, removed Superadmin Dashboard section and cross-references ## Context This PR re-creates the #6575 work after the original PR was accidentally squash merged with 29 commits from its ancestor PRs (reverted in #6780). Now scoped to dashboard.md only, on top of #6574's branch.
Merge activity
|
…ge docs (#6574) Resolves #6680 (FR-2536) ## Summary - Add **Start From URL** section documenting the Import Notebook, Import GitHub Repository, and Import GitLab Repository tabs - Add **Announcement Banner** section explaining the admin-configured banner at the top of the Start page - Add **Customizing Card Layout** section documenting the drag-and-drop card arrangement feature - Add screenshots for all new sections (start_from_url_notebook, start_from_url_github, start_from_url_gitlab, start_announcement_banner) - Apply reviewer fixes for card titles and i18n accuracy ## Changes - `start/start.md` (en/ko/ja/th): Start From URL section, Announcement Banner section, card layout customization section - `start/images/`: 4 new screenshots added ## Test plan - [ ] Verify Start From URL tab descriptions match actual UI tabs - [ ] Verify announcement banner screenshot and description are accurate - [ ] Verify drag-and-drop card customization is described correctly 🤖 Generated with [Claude Code](https://claude.com/claude-code)
4e4820e to
14ce31c
Compare
fd1912f to
bfeea9f
Compare
…docs Resolves #6681 (FR-2537) ## Summary - Document **board customization** via drag-and-drop on the Dashboard page - Document the **auto-refresh** toggle and 15-second refresh interval - Remove references to the Superadmin Dashboard per reviewer feedback (feature is hidden in current UI) ## Changes - `dashboard/dashboard.md` (en/ko/ja/th): board customization section, auto-refresh note, removed Superadmin Dashboard section and cross-references ## Context This PR re-creates the #6575 work after the original PR was accidentally squash merged with 29 commits from its ancestor PRs (reverted in #6780). Now scoped to dashboard.md only, on top of #6574's branch.
…docs (#6781) Resolves #6681 (FR-2537) ## Summary - Document **board customization** via drag-and-drop on the Dashboard page - Document the **auto-refresh** toggle and 15-second refresh interval - Remove references to the Superadmin Dashboard per reviewer feedback (feature is hidden in current UI) ## Changes - `dashboard/dashboard.md` (en/ko/ja/th): board customization section, auto-refresh note, removed Superadmin Dashboard section and cross-references ## Context This PR re-creates the #6575 work after the original PR was accidentally squash merged with 29 commits from its ancestor PRs (reverted in #6780). Now scoped to dashboard.md only, stacked on top of #6574's branch. ## Test plan - [ ] Verify drag-and-drop board customization behavior is accurately described - [ ] Verify auto-refresh interval (15 seconds) matches the actual UI - [ ] Confirm Superadmin Dashboard removal is correct across all 4 languages

Resolves #6680 (FR-2536)
Summary
Changes
start/start.md(en/ko/ja/th): Start From URL section, Announcement Banner section, card layout customization sectionstart/images/: 4 new screenshots addedTest plan
🤖 Generated with Claude Code