Skip to content

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

Open
yomybaby wants to merge 2 commits into
05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_namesfrom
05-15-docs_vfolder_document_upload-permission_gating_invitation_entry_point_and_cascading_delete_warning
Open

docs(vfolder): document upload-permission gating, invitation entry point, and cascading delete warning#7447
yomybaby wants to merge 2 commits into
05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_namesfrom
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

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


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
yomybaby added 2 commits May 18, 2026 16:29
…int, and cascading delete warning

Part of the recent main-branch docs catch-up plan (Work Item 7).

Source PRs:
- FR-2893 (#7412) disable Upload button when host lacks upload-file permission
- FR-2817 (#7261) unclip folder-status invitation badge; /data invitation entry point
- FR-2813 (#7260) folder-create notification with VFolder node card
- FR-2845 (#7339) cascading model card deletion warning when associated vfolder deleted
- FR-2622 family (#6881, #6883, #7135, #7250) delete-folder option + trash notification on model card deletion
- FR-2651 (#6888) list all model-store folders when creating a model card

Changes:
- New H3 "Invitation Badge and Entry Point" — unclipped pending-invitation badge on the Data entry; /data itself usable for invitation review.
- Explore Folder: :::warning explains Upload button + drag-and-drop are disabled when the keypair resource policy denies upload-file on the host.
- Delete Folder: :::warning[Cascading model card deletion] documents "Also delete the associated model folder" option, cascade-warning copy from i18n, and cross-references admin_menu#manage-model-cards.

Updated in all 4 languages (en/ko/ja/th). 3 new screenshots flagged with TODO markers.
Captures 1 of 3 planned screenshots × 4 languages (4 PNGs):
- vfolder_invitation_badge.png

Skipped (TODO markers retained):
- vfolder_upload_disabled.png — requires a keypair resource policy without upload-file on the host; admin/user on isla-sorna both have upload-file granted, disabled state not reachable from WebUI alone.
- vfolder_cascading_delete_warning.png — the cascade warning surfaces on Admin > Manage Model Cards delete flow, not the user Delete VFolder dialog; this is part of PR H's admin_menu coverage (model_card_delete_with_folder.png).

Also surfaces two doc-accuracy follow-ups (not fixed here):
1. The invitation badge wording in vfolder.md describes a sidebar badge; actual UI is a tab-bar "Pending invitations (N)" link.
2. The user-facing Delete VFolder modal does not currently show the cascade warning; the warning is admin-side. The prose should be tightened to reflect that.

Captured at 2× zoom via Portless dev, per-locale via window.switchLanguage().
@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
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