bmaquet opened a new pull request, #32996:
URL: https://github.com/apache/superset/pull/32996

   ### SUMMARY
   This PR introduces "Select All" and "Deselect All" buttons to the Table & 
Roles elements of the Row Level Security (RLS) configuration interface. This 
enhancement aims to improve user experience, particularly when managing a large 
number of tables or roles, by allowing bulk selection and deselection instead 
of requiring individual clicks for each item. The implementation adds two 
distinct buttons to trigger these actions, updating the selection state 
accordingly.
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   #### BEFORE
   <img width="430" alt="Screenshot 2025-04-03 at 12 59 20" 
src="https://github.com/user-attachments/assets/bb87f34d-646e-4f8f-9ea3-e9c637067490";
 />
   
   #### AFTER
   <img width="427" alt="Screenshot 2025-04-03 at 13 02 44" 
src="https://github.com/user-attachments/assets/d761da32-03c9-425a-8d2c-707a7bda9e69";
 />
   
   ### TESTING INSTRUCTIONS
   1. Navigate to the Row Level Security configuration section.
   2. Click on Datasets or Roles boxes
   3. Observe the presence of the new "Select All" and "Deselect All" buttons.
   4. If starting with no items selected, click "Select All". Verify that all 
relevant items (e.g., tables, roles) are now selected.
   5. Click "Deselect All". Verify that all items are now deselected.
   6. Manually select a subset of items.
   7. Click "Deselect All". Verify all are deselected.
   8. Manually select a subset of items.
   9. Click "Select All". Verify all are selected.
   
   ### ADDITIONAL INFORMATION
   - [ ] Has associated issue: *(Fill in if applicable, e.g., `Fixes #123`)*
   - [ ] Required feature flags: *(Fill in if applicable)*
   - [x] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [x] Introduces new feature or API *(Implicitly, via UI interaction)*
   - [ ] Removes existing feature or API


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to