[GitHub] [superset] codyml commented on pull request #20684: fix(dashboard): Fix missing metadata on draggable dashboard edit chart cards

[GitBox <gi...@apache.org>]

codyml commented on PR #20684:
URL: https://github.com/apache/superset/pull/20684#issuecomment-1185653640

   > I did some digging and this is the [PR](https://github.com/apache/superset/pull/11336) that introduced that filter. Maybe @nytai has some context on it?
   
   @michael-s-molina Looks like that PR transitioned from another endpoint that had a similar restriction – do you know if there's any spec for what the permissions are supposed to be around adding charts to a dashboard?  My thought would be that if you have view permissions on a chart, you should be able to add it to a dashboard that you have edit permissions for, but that's just off the top of my head, I don't have a great understanding of the permissions model.


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org