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

   ### SUMMARY
   This PR introduced a new permission to enable view and drilling actions on 
the Dashboard independently of the `can_explore` permission #26798. However, 
from recent feedback it is clear that the need is to have more granularity for 
these permissions.
   
   This PR introduces the following permissions:
   
   - `can_view_table` to enable the "View as table" option in the chart actions 
in a Dashboard
   - `can_view_query` to enable the "View query" option in the chart actions in 
a Dashboard
   
   This PR also implements the following changes:
   - It removes the permission `can_view_and_drill` introduced in #26798.
   - It ties Drill to detail to both `can_explore` and `can_samples` as 
`can_samples` is a required backend permission for Drill to detail to work
   - It ties Drill by  to both `can_explore` and 
`can_write_ExploreFormDataRestAPI` as the latter is a required backend 
permissions for the submission of form data when interacting with Drill by
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   <!--- Skip this if not applicable -->
   
   ### TESTING INSTRUCTIONS
   <!--- Required! What steps can be taken to manually verify the changes? -->
   
   ### 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:
   - [ ] 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
   - [ ] 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]

Reply via email to