fix(FR-2622): add delete-folder option and trash notification on model card deletion by agatha197 · Pull Request #6881 · lablup/backend.ai-webui

agatha197 · 2026-04-22T05:42:47Z

Resolves #6841 (FR-2622)

Summary

Added vfolderId field to AdminModelCardListPageQuery so the model card's associated vfolder ID is available in the deletion flow.
Added AdminModelCardListPageDeleteVFolderMutation (deleteVfolderV2) to soft-delete (move to trash) the associated model folder.
Added alsoDeleteFolder state (default false) that resets when the deletion dialog closes.
Added an opt-in Checkbox (wrapped in Tooltip) to the single-delete BAIDeleteConfirmModal via the extraContent prop, allowing admins to also trash the associated model folder in the same action.
Updated the onOk handler: when checked, deleteVfolderV2 is called after card deletion; both paths show a notification.success with a "Go to Data > Trash" action button navigating to /data?statusCategory=deleted (with folder filter when folder is also deleted).
Added new i18n keys under adminModelCard in en.json: AlsoDeleteModelFolder, AlsoDeleteModelFolderTooltip, ModelCardAndFolderDeleted, ModelCardDeletedFolderKept, GoToTrash.

agatha197 · 2026-04-22T05:43:01Z

fix(FR-2651): list all model-store folders when creating a model card #6888 : 2 dependent PRs (#6887 , #7135 )
chore(FR-2622): add i18n translations for delete-folder option #6883
fix(FR-2622): add delete-folder option and trash notification on model card deletion #6881 👈 (View in Graphite)
main

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.

github-actions · 2026-04-22T05:45:54Z

Coverage report for `./react`

St.^❔	Category	Percentage	Covered / Total
🔴	Statements	8.11% (-0% 🔻)	1859/22932
🔴	Branches	7.34% (-0.01% 🔻)	1187/16168
🔴	Functions	4.82% (+0.01% 🔼)	297/6161
🔴	Lines	7.88% (-0% 🔻)	1750/22217

Test suite run success

865 tests passing in 40 suites.

Report generated by 🧪jest coverage report action from 19752e5

Copilot

Pull request overview

Adds an optional “delete associated model folder” flow when deleting an admin model card, and replaces the plain success toast with a notification that can deep-link users to the Trash view.

Changes:

Added new adminModelCard.* i18n strings for the new checkbox/tooltip and deletion result notifications.
Updated AdminModelCardListPage delete confirmation modal to include an “also delete folder” checkbox and (optionally) run deleteVfolderV2, followed by a success notification with a “Go to Data > Trash” action.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
resources/i18n/en.json	Adds new English strings for delete-folder option and trash navigation notification.
react/src/pages/AdminModelCardListPage.tsx	Adds delete-folder checkbox, new vfolder delete mutation, and a post-delete trash notification with navigation.

ironAiken2

test을(를) 입력하여 확인하세요. The translated text seems a little awkward. 'Please enter <folder name> to delete.' How does that sound?

github-actions · 2026-04-28T06:48:18Z

Coverage report for `./packages/backend.ai-ui`

St.^❔	Category	Percentage	Covered / Total
🟢	Statements	81.87%	429/524
🟡	Branches	70.43%	362/514
🟡	Functions	76.86%	93/121
🟢	Lines	83.19%	391/470

Test suite run success

319 tests passing in 13 suites.

Report generated by 🧪jest coverage report action from 19752e5

ironAiken2

test을(를) 입력하여 확인하세요. The translated text seems a little awkward. 'Please enter <folder name> to delete.' How does that sound?

Please check this comment

ironAiken2

LGTM

github-actions · 2026-05-04T23:59:40Z

Coverage Report for backend-ai-ui-coverage (./packages/backend.ai-ui)

Status	Category	Percentage	Covered / Total
🔵	Lines	8.58%	361 / 4203
🔵	Statements	7.72%	410 / 5305
🔵	Functions	9.19%	94 / 1022
🔵	Branches	6.94%	362 / 5215

File Coverage

File	Stmts	Branches	Functions	Lines	Uncovered Lines
Changed Files
packages/backend.ai-ui/src/components/BAIDeleteConfirmModal.tsx	0%	0%	0%	0%	9-134

Generated in workflow #421 for commit 6eaccdb by the Vitest Coverage Report Action

github-actions · 2026-05-05T00:00:42Z

Coverage Report for react-coverage (./react)

Status	Category	Percentage	Covered / Total
🔵	Lines	6.52%	1783 / 27337
🔵	Statements	5.39%	1978 / 36668
🔵	Functions	5.2%	296 / 5691
🔵	Branches	3.78%	1293 / 34141

File Coverage

File	Stmts	Branches	Functions	Lines	Uncovered Lines
Changed Files
react/src/components/AdminModelCardSettingModal.tsx	0%	0%	0%	0%	72-210
react/src/components/ModelCardDrawer.tsx	0%	0%	0%	0%	43-348
react/src/pages/AdminModelCardListPage.tsx	0%	0%	0%	0%	59-571

Generated in workflow #421 for commit 6eaccdb by the Vitest Coverage Report Action

graphite-app · 2026-05-07T07:19:44Z

Merge activity

May 7, 7:19 AM UTC: ironAiken2 added this pull request to the Graphite merge queue.
May 7, 7:23 AM UTC: Merged by the Graphite merge queue.

…l card deletion (#6881) Resolves #6841 (FR-2622) ## Summary - Added `vfolderId` field to `AdminModelCardListPageQuery` so the model card's associated vfolder ID is available in the deletion flow. - Added `AdminModelCardListPageDeleteVFolderMutation` (`deleteVfolderV2`) to soft-delete (move to trash) the associated model folder. - Added `alsoDeleteFolder` state (default `false`) that resets when the deletion dialog closes. - Added an opt-in `Checkbox` (wrapped in `Tooltip`) to the single-delete `BAIDeleteConfirmModal` via the `extraContent` prop, allowing admins to also trash the associated model folder in the same action. - Updated the `onOk` handler: when checked, `deleteVfolderV2` is called after card deletion; both paths show a `notification.success` with a "Go to Data > Trash" action button navigating to `/data?statusCategory=deleted` (with folder filter when folder is also deleted). - Added new i18n keys under `adminModelCard` in `en.json`: `AlsoDeleteModelFolder`, `AlsoDeleteModelFolderTooltip`, `ModelCardAndFolderDeleted`, `ModelCardDeletedFolderKept`, `GoToTrash`.

Resolves #6841 (FR-2622) Stacked on top of #6881. ## Summary Translates the 5 new `adminModelCard.*` i18n keys introduced in #6881 into all 20 target languages (de, el, es, fi, fr, id, it, ja, ko, mn, ms, pl, pt-BR, pt, ru, th, tr, vi, zh-CN, zh-TW). Keys translated: - `AlsoDeleteModelFolder` - `AlsoDeleteModelFolderTooltip` - `GoToTrash` - `ModelCardAndFolderDeleted` - `ModelCardDeletedFolderKept` Terminology kept consistent with existing translations (e.g., "trash" → 휴지통 / ゴミ箱 / Papierkorb / Poubelle, "model card" → 모델 카드 / モデルカード). Keys inserted at the alphabetically correct position within `adminModelCard`.

Copilot AI review requested due to automatic review settings April 22, 2026 05:42

github-actions Bot added area:ux UI / UX issue. area:i18n Localization labels Apr 22, 2026

github-actions Bot assigned agatha197 Apr 22, 2026

github-actions Bot added the size:L 100~500 LoC label Apr 22, 2026

Copilot started reviewing on behalf of agatha197 April 22, 2026 05:43 View session

agatha197 mentioned this pull request Apr 22, 2026

chore(FR-2622): add i18n translations for delete-folder option #6883

Merged

Copilot AI reviewed Apr 22, 2026

View reviewed changes

Comment thread resources/i18n/en.json

Comment thread react/src/pages/AdminModelCardListPage.tsx Outdated

Comment thread react/src/pages/AdminModelCardListPage.tsx Outdated

agatha197 force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch 6 times, most recently from 5a33b0a to 196618a Compare April 22, 2026 07:21

This was referenced Apr 22, 2026

test(FR-2652): add E2E tests for admin model card deletion with folder option #6887

Closed

fix(FR-2651): list all model-store folders when creating a model card #6888

Merged

agatha197 requested review from ironAiken2 and nowgnuesLee April 22, 2026 10:16

ironAiken2 requested changes Apr 23, 2026

View reviewed changes

Comment thread resources/i18n/en.json

Comment thread react/src/pages/AdminModelCardListPage.tsx

agatha197 force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch from 196618a to 2763cba Compare April 23, 2026 11:49

agatha197 requested a review from ironAiken2 April 23, 2026 11:49

agatha197 force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch 2 times, most recently from 893816e to b61a8af Compare April 24, 2026 07:53

agatha197 mentioned this pull request Apr 24, 2026

chore(FR-2652): update E2E coverage report — admin model store 26/26 (100%) #6969

Closed

5 tasks

agatha197 requested a review from yomybaby April 24, 2026 08:07

ironAiken2 requested changes Apr 27, 2026

View reviewed changes

agatha197 force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch from b61a8af to cbde122 Compare April 28, 2026 06:45

agatha197 requested a review from ironAiken2 April 28, 2026 06:46

ironAiken2 requested changes Apr 28, 2026

View reviewed changes

agatha197 force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch from cbde122 to 4fe4032 Compare April 28, 2026 08:00

agatha197 requested a review from ironAiken2 April 28, 2026 08:02

ironAiken2 approved these changes Apr 28, 2026

View reviewed changes

This was referenced Apr 28, 2026

feat(FR-2622): add bulk vfolder trash support on model card bulk deletion #7135

Merged

test(FR-2622): add E2E tests for bulk vfolder trash on model card bulk deletion #7137

Closed

agatha197 force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch from 4fe4032 to 6d6f443 Compare April 30, 2026 04:34

agatha197 mentioned this pull request Apr 30, 2026

test(FR-2652): fix parallel interference and folder-dialog flakiness in delete spec #7162

Closed

2 tasks

agatha197 force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch 2 times, most recently from 19752e5 to 6f30d94 Compare May 4, 2026 23:57

agatha197 force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch 2 times, most recently from aa20978 to 2e8fa5b Compare May 6, 2026 02:28

agatha197 mentioned this pull request May 6, 2026

test(FR-2622): add E2E tests for admin model card management #7250

Open

agatha197 force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch 3 times, most recently from 6f6bf8c to 34262a5 Compare May 7, 2026 05:13

graphite-app Bot force-pushed the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch from 34262a5 to 6eaccdb Compare May 7, 2026 07:20

graphite-app Bot merged commit 6eaccdb into main May 7, 2026
12 checks passed

graphite-app Bot deleted the 04-22-fix_fr-2622_add_delete-folder_option_and_trash_notification_on_model_card_deletion branch May 7, 2026 07:23

Conversation

agatha197 commented Apr 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

agatha197 commented Apr 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

How to use the Graphite Merge Queue

Uh oh!

github-actions Bot commented Apr 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report for ./react

Test suite run success

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ironAiken2 left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Apr 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report for ./packages/backend.ai-ui

Test suite run success

Uh oh!

ironAiken2 left a comment

Choose a reason for hiding this comment

Uh oh!

ironAiken2 left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented May 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Report for backend-ai-ui-coverage (./packages/backend.ai-ui)

Uh oh!

github-actions Bot commented May 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Report for react-coverage (./react)

Uh oh!

graphite-app Bot commented May 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

agatha197 commented Apr 22, 2026 •

edited

Loading

agatha197 commented Apr 22, 2026 •

edited

Loading

github-actions Bot commented Apr 22, 2026 •

edited

Loading

Coverage report for `./react`

github-actions Bot commented Apr 28, 2026 •

edited

Loading

Coverage report for `./packages/backend.ai-ui`

github-actions Bot commented May 4, 2026 •

edited

Loading

github-actions Bot commented May 5, 2026 •

edited

Loading

graphite-app Bot commented May 7, 2026 •

edited

Loading