fix(FR-1645): Mismatch in Selected Count After Partial Folder Deletion

[agatha197]

Resolves #4578 (FR-1645)

Improve folder selection handling and deletion callback

This PR enhances two key aspects of folder management:

1. Adds a preserveOtherPageSelections parameter to handleRowSelectionChange function, allowing selections to persist across different pages
2. Updates the deletion callback in VFolderNodes to pass the deleted folder ID to the parent component

Changes:

• Enhanced handleRowSelectionChange to properly maintain selections when navigating between pages
• Implemented the improved selection handling in VFolderNodeListPage with cross-page selection preservation
• Modified the deletion success callback to pass the deleted folder ID to the parent component

Checklist:

• 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

[agatha197]

• fix(FR-1645): Mismatch in Selected Count After Partial Folder Deletion #4580 👈 (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 hot fixes, skip the queue and merge this PR next

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]

Coverage report for ./react

St.❔	Category	Percentage	Covered / Total
🔴	Statements	4.64% (-0% 🔻)	531/11434
🔴	Branches	3.77% (-0% 🔻)	302/8011
🔴	Functions	2.89% (-0% 🔻)	102/3535
🔴	Lines	4.59% (-0% 🔻)	513/11176

Show files with reduced coverage 🔻

St.❔	File	Statements	Branches	Functions	Lines
🔴	helper/index.tsx	41.26% (-1.67% 🔻)	39.41% (-0.71% 🔻)	24.53% (-2% 🔻)	41.85% (-1.65% 🔻)

Test suite run success

121 tests passing in 14 suites.

Report generated by 🧪jest coverage report action from fa489f7

[Copilot]

Pull Request Overview

This PR enhances the row selection behavior in the VFolder nodes table by introducing a new preserveOtherPageSelections parameter to the handleRowSelectionChange helper function. This allows cross-page selections to be properly maintained when pagination is used.

• Refactored handleRowSelectionChange to support two selection modes via a new optional parameter
• Updated VFolderNodeListPage to enable the new cross-page selection preservation mode
• Modified the onRequestChange callback in VFolderNodes to pass the deleted folder ID for proper cleanup

Reviewed Changes

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

File	Description
react/src/helper/index.tsx	Added preserveOtherPageSelections parameter to handleRowSelectionChange with conditional logic for handling cross-page vs. current-page-only selection behavior
react/src/pages/VFolderNodeListPage.tsx	Enabled cross-page selection preservation by passing 'id' and true as additional arguments to handleRowSelectionChange
react/src/components/VFolderNodes.tsx	Updated mutation success callback to pass folder ID to onRequestChange instead of calling it without arguments

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[nowgnuesLee]

LGTM

[yomybaby]

I closed this PR without merging. This FR-1645 is resolved by #4613 . Please review it. Thank you. @agatha197 @nowgnuesLee