Skip to content

docs(vfolder): document upload-permission gating, invitation entry point, and cascading delete warning#7447

Merged
graphite-app[bot] merged 1 commit into
mainfrom
05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning
May 19, 2026
Merged

docs(vfolder): document upload-permission gating, invitation entry point, and cascading delete warning#7447
graphite-app[bot] merged 1 commit into
mainfrom
05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning

Conversation

@yomybaby
Copy link
Copy Markdown
Member

resolves #NNN (FR-MMM)

Checklist: (if applicable)

  • Documentation
  • Minium required manager version
  • Specific setting for review (eg., KB link, endpoint or how to setup)
  • Minimum requirements to check during review
  • Test case(s) to demonstrate the difference of before/after

Copy link
Copy Markdown
Member Author

yomybaby commented May 16, 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

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Backend.AI WebUI user manual’s Data / storage folder (vfolder) documentation to reflect recent UX behaviors: where invitation management is surfaced, when uploads are disabled due to host-level permissions, and the risk of cascading deletions when a storage folder is associated with model cards.

Changes:

  • Added a section describing the invitation badge and the /data entry points for reviewing invitations.
  • Documented upload gating when the keypair resource policy lacks the host-level upload-file capability.
  • Added a cascading model card deletion warning and an admin-side cross-reference.

Reviewed changes

Copilot reviewed 4 out of 8 changed files in this pull request and generated 16 comments.

File Description
packages/backend.ai-webui-docs/src/en/vfolder/vfolder.md Adds invitation entry point, upload permission gating warning, and cascading delete warning (English).
packages/backend.ai-webui-docs/src/ko/vfolder/vfolder.md Same documentation additions localized to Korean.
packages/backend.ai-webui-docs/src/ja/vfolder/vfolder.md Same documentation additions localized to Japanese.
packages/backend.ai-webui-docs/src/th/vfolder/vfolder.md Same documentation additions localized to Thai.
Comments suppressed due to low confidence (5)

packages/backend.ai-webui-docs/src/en/vfolder/vfolder.md:199

  • This image reference appears to be missing from the repo (no vfolder_cascading_delete_warning.png under packages/backend.ai-webui-docs/src/en/images/), so it will render as a broken image in the built docs. Add the asset (and keep filenames consistent across locales) or remove the reference until it exists.
![](../images/vfolder_cascading_delete_warning.png)
<!-- TODO: Capture screenshot of the cascading delete warning dialog -->

packages/backend.ai-webui-docs/src/ko/vfolder/vfolder.md:163

  • This image reference appears to be missing from the repo (no vfolder_cascading_delete_warning.png under packages/backend.ai-webui-docs/src/ko/images/), so it will render as a broken image in the built docs. Add the asset (and keep filenames consistent across locales) or remove the reference until it exists.
![](../images/vfolder_cascading_delete_warning.png)
<!-- TODO: 연쇄 삭제 경고 대화 상자의 스크린샷을 캡처합니다 -->

packages/backend.ai-webui-docs/src/ja/vfolder/vfolder.md:166

  • This image reference appears to be missing from the repo (no vfolder_cascading_delete_warning.png under packages/backend.ai-webui-docs/src/ja/images/), so it will render as a broken image in the built docs. Add the asset (and keep filenames consistent across locales) or remove the reference until it exists.
![](../images/vfolder_cascading_delete_warning.png)
<!-- TODO: 連鎖削除警告ダイアログのスクリーンショットを撮影します -->

packages/backend.ai-webui-docs/src/th/vfolder/vfolder.md:106

  • This image reference appears to be missing from the repo (no vfolder_upload_disabled.png under packages/backend.ai-webui-docs/src/th/images/), so it will render as a broken image in the built docs. Add the asset (and keep filenames consistent across locales) or remove the reference until it exists.
![](../images/vfolder_upload_disabled.png)
<!-- TODO: ถ่ายภาพหน้าจอของปุ่มอัปโหลดในสถานะปิดใช้งานพร้อมเครื่องมือแนะนำสิทธิ์ -->

packages/backend.ai-webui-docs/src/th/vfolder/vfolder.md:163

  • This image reference appears to be missing from the repo (no vfolder_cascading_delete_warning.png under packages/backend.ai-webui-docs/src/th/images/), so it will render as a broken image in the built docs. Add the asset (and keep filenames consistent across locales) or remove the reference until it exists.
![](../images/vfolder_cascading_delete_warning.png)
<!-- TODO: ถ่ายภาพหน้าจอของกล่องโต้ตอบเตือนการลบแบบลูกโซ่ -->

Comment thread packages/backend.ai-webui-docs/src/en/vfolder/vfolder.md Outdated
Comment thread packages/backend.ai-webui-docs/src/en/vfolder/vfolder.md Outdated
Comment thread packages/backend.ai-webui-docs/src/en/vfolder/vfolder.md
Comment thread packages/backend.ai-webui-docs/src/en/vfolder/vfolder.md Outdated
Comment thread packages/backend.ai-webui-docs/src/ko/vfolder/vfolder.md Outdated
Comment thread packages/backend.ai-webui-docs/src/ja/vfolder/vfolder.md Outdated
Comment thread packages/backend.ai-webui-docs/src/th/vfolder/vfolder.md Outdated
Comment thread packages/backend.ai-webui-docs/src/th/vfolder/vfolder.md Outdated
Comment thread packages/backend.ai-webui-docs/src/th/vfolder/vfolder.md Outdated
Comment thread packages/backend.ai-webui-docs/src/th/vfolder/vfolder.md Outdated
@agatha197 agatha197 force-pushed the 05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning branch from 1bfb609 to 4beee61 Compare May 18, 2026 03:54
@agatha197 agatha197 force-pushed the 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names branch from f161984 to c722f31 Compare May 18, 2026 03:54
@agatha197 agatha197 force-pushed the 05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning branch from 4beee61 to 9e59847 Compare May 18, 2026 05:51
@agatha197 agatha197 force-pushed the 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names branch from c722f31 to 8541193 Compare May 18, 2026 05:51
@agatha197 agatha197 force-pushed the 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names branch from 8541193 to 55cb4cf Compare May 18, 2026 07:17
@agatha197 agatha197 force-pushed the 05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning branch from 9e59847 to 48a8ada Compare May 18, 2026 07:17
@agatha197 agatha197 force-pushed the 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names branch from 55cb4cf to 3aeee5f Compare May 18, 2026 07:30
@agatha197 agatha197 force-pushed the 05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning branch from 48a8ada to cf500ad Compare May 18, 2026 07:30
@ironAiken2 ironAiken2 changed the base branch from 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names to graphite-base/7447 May 19, 2026 08:00
@agatha197 agatha197 changed the base branch from graphite-base/7447 to 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names May 19, 2026 08:39
@agatha197 agatha197 changed the base branch from 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names to 05-18-docs_fr-2909_align_rbac_manual_with_revoke_confirmation_modal_and_refresh_screenshots May 19, 2026 08:41
@agatha197 agatha197 changed the base branch from 05-18-docs_fr-2909_align_rbac_manual_with_revoke_confirmation_modal_and_refresh_screenshots to graphite-base/7447 May 19, 2026 08:43
@agatha197 agatha197 changed the base branch from graphite-base/7447 to 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names May 19, 2026 08:43
@agatha197 agatha197 changed the base branch from 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names to 05-18-docs_fr-2909_align_rbac_manual_with_revoke_confirmation_modal_and_refresh_screenshots May 19, 2026 08:46
Copy link
Copy Markdown
Contributor

@agatha197 agatha197 left a comment

Choose a reason for hiding this comment

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

LGTM

@agatha197 agatha197 changed the base branch from 05-18-docs_fr-2909_align_rbac_manual_with_revoke_confirmation_modal_and_refresh_screenshots to graphite-base/7447 May 19, 2026 14:28
@agatha197 agatha197 force-pushed the graphite-base/7447 branch from 9ba58a2 to 3aeee5f Compare May 19, 2026 14:28
@agatha197 agatha197 changed the base branch from graphite-base/7447 to 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names May 19, 2026 14:28
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented May 19, 2026

Merge activity

…int, and cascading delete warning (#7447)

resolves #NNN (FR-MMM)
<!-- replace NNN, MMM with the GitHub issue number and the corresponding Jira issue number. -->

<!--
Please precisely, concisely, and concretely describe what this PR changes, the rationale behind codes,
and how it affects the users and other developers.
-->

**Checklist:** (if applicable)

- [ ] Documentation
- [ ] Minium required manager version
- [ ] Specific setting for review (eg., KB link, endpoint or how to setup)
- [ ] Minimum requirements to check during review
- [ ] Test case(s) to demonstrate the difference of before/after
@graphite-app graphite-app Bot force-pushed the 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names branch from 3aeee5f to 2a00256 Compare May 19, 2026 15:21
@graphite-app graphite-app Bot force-pushed the 05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning branch from cf500ad to d467e64 Compare May 19, 2026 15:22
Base automatically changed from 05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_names to main May 19, 2026 15:30
@graphite-app graphite-app Bot merged commit d467e64 into main May 19, 2026
4 checks passed
@graphite-app graphite-app Bot deleted the 05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning branch May 19, 2026 15:31
graphite-app Bot pushed a commit that referenced this pull request May 20, 2026
…folder screenshots (#7489)

Resolves #7488 (FR-2924)

## Summary

- Captured the two previously-missing vfolder screenshots in all four locales (en, ko, ja, th), each in its own UI language:
  - `vfolder_upload_disabled.png` — file-browser modal with the disabled Upload button and the "no upload permission for this host" tooltip
  - `vfolder_cascading_delete_warning.png` — model-card delete confirmation dialog with the cascading-delete warning alert shown
- Removed the corresponding placeholder `<!-- TODO: ... -->` comments from `src/{en,ja,ko,th}/vfolder/vfolder.md`

Stacked on top of `05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning` (PR #7447), which introduced these image references with TODO placeholders.

## Test plan

- [ ] Build the manual and confirm `vfolder_upload_disabled.png` and `vfolder_cascading_delete_warning.png` render in all 4 languages
- [ ] Confirm no remaining `TODO` placeholder comments for these two images in any locale
- [ ] Verify each locale's screenshot shows the UI in that language
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100~500 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants