Antonio-RiveroMartnez opened a new pull request, #32718: URL: https://github.com/apache/superset/pull/32718
### SUMMARY We recently introduced the `Groups` [with this PR](https://github.com/apache/superset/pull/32121), these changes are adding support for them on the `RLS` rules. This way it become even more friendly and usable setting rules for users. ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF #### Creating a RLS rule ##### Regular rule <img width="481" alt="Screenshot 2025-03-18 at 11 47 14 AM" src="https://github.com/user-attachments/assets/44245af3-62b2-419f-93f9-590c1378b003"; /> ##### Base rule <img width="481" alt="Screenshot 2025-03-18 at 11 47 26 AM" src="https://github.com/user-attachments/assets/0e046584-510b-4e10-9f1f-29cda90d57b6"; /> #### Rules being applied based on groups from direct user <-> group association and rule being linked to the group <img width="1648" alt="Screenshot 2025-03-18 at 11 54 34 AM" src="https://github.com/user-attachments/assets/6139354f-66a9-4854-a6ec-4d3efd07e8a8"; /> #### Rules being applied based on groups from role association (i.e, no direct user <-> group relation, but the group linked to a role and the user linked to that same role) ##### RLS rule details <img width="480" alt="Screenshot 2025-03-18 at 11 58 38 AM" src="https://github.com/user-attachments/assets/6728d29a-b194-4334-89a9-775064f68fff"; /> ##### Group details <img width="480" alt="Screenshot 2025-03-18 at 11 58 53 AM" src="https://github.com/user-attachments/assets/c6b2a151-5943-457e-9b90-13879084f697"; /> ##### Rule applied <img width="1660" alt="Screenshot 2025-03-18 at 11 59 52 AM" src="https://github.com/user-attachments/assets/e445c110-62a4-4f1a-b27f-5527421475f6"; /> ### TESTING INSTRUCTIONS 1. Create group(s) using the existing functionality 2. Access the Row Level Security menu and associate the group(s) you created 3. Verify the clause gets into the generated query when for example creating a chart using a restricted table ### ADDITIONAL INFORMATION <!--- Check any relevant boxes with "x" --> <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue --> - [ ] Has associated issue: - [ ] Required feature flags: - [X] Changes UI - [X] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [X] Migration is atomic, supports rollback & is backwards-compatible - [ ] Confirm DB migration upgrade and downgrade tested - [ ] Runtime estimates and downtime expectations provided - [ ] Introduces new feature or API - [ ] 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]
