amitNielsen commented on issue #10408:
URL: 
https://github.com/apache/incubator-superset/issues/10408#issuecomment-688648112


   @mistercrunch 
   
   * when giving access to a dashboard, so dashboard itself is accessible but 
then each one of the chart are doing additional calls to get the underlying 
data from the **datasource**, so those calls will also need to be validated 
that although there is no direct access to the **datasource** the access to 
that dashboard is allowing to pull data for the need of viewing it inside that 
specific dashboard
   
   Giving access to a specific dashboard can be sufficient enough for the user 
to view the dashboard without the complexicity of adding all underlying 
datasources 👍 🚀 
   
   * for users who have access to dashboard but not the dataset, the explore 
chart should be invisible to simplify **dashboard viewers** to my opinion same 
as edit dashboard should be invisible


----------------------------------------------------------------
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.

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