Arunodoy18 opened a new pull request, #37592: URL: https://github.com/apache/superset/pull/37592
### SUMMARY Fixes a regression where granting permissions (e.g. "can read on Dashboard") to the Public role caused Admin users to lose access to dashboards. This happened due to incorrect permission resolution behavior where updates to the Public role impacted higher privilege roles. This change ensures permission aggregation is additive and does not allow lower privilege roles to override higher privilege role permissions. ### TESTING INSTRUCTIONS 1. Grant "can read on Dashboard" to Public role 2. Verify Admin can still access dashboards via API 3. Verify Public role behavior works as expected ### ADDITIONAL INFORMATION - [x] Has associated issue: Fixes #24837 - [ ] Required feature flags - [ ] Changes UI - [ ] DB Migration -- 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]
