Skip to content

fix: implement confirmation window for multi-select actions - #1288

Merged
jekabs-karklins merged 8 commits into
developfrom
SWAP-5030-uo-show-confirmation-window-before-executing-acti
Dec 19, 2025
Merged

fix: implement confirmation window for multi-select actions#1288
jekabs-karklins merged 8 commits into
developfrom
SWAP-5030-uo-show-confirmation-window-before-executing-acti

Conversation

@jekabs-karklins

@jekabs-karklins jekabs-karklins commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

Description

This pull request implements a confirmation window for multi-select actions in the proposal table

Motivation and Context

This change is required to introduce a safeguard against accidental bulk actions on the proposal table. It has happened in past that user unintentionally selected all proposals on the page and clicked CLONE, which resulted in a large number of unintended cloned proposals being created, leading to significant cleanup effort.

Changes

  1. A Warning icon from @mui/icons-material is imported to visually indicate the caution required for bulk actions.
  2. A new function runWithMultiSelectConfirm is introduced. This function takes an action as an argument and checks if more than one proposal is selected. If so, it opens a confirmation window with a detailed warning message before executing the action.

How Has This Been Tested?

Fixes Jira Issue

https://jira.esss.lu.se/browse/SWAP-5030

Depends On

Tests included/Docs Updated?

  • I have added tests to cover my changes.
  • All relevant doc has been updated

@jekabs-karklins
jekabs-karklins requested a review from a team as a code owner December 9, 2025 14:27
@jekabs-karklins
jekabs-karklins requested review from Scott-James-Hurley and janosbabik and removed request for a team December 9, 2025 14:27
@jekabs-karklins jekabs-karklins changed the title fix: implement confirmation window for multi-select actions in Propos… fix: implement confirmation window for multi-select actions Dec 10, 2025
@jekabs-karklins
jekabs-karklins merged commit fb276ef into develop Dec 19, 2025
36 of 37 checks passed
@jekabs-karklins
jekabs-karklins deleted the SWAP-5030-uo-show-confirmation-window-before-executing-acti branch December 19, 2025 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants