etr2460 commented on pull request #13992: URL: https://github.com/apache/superset/pull/13992#issuecomment-815048469
Is this the default behavior we want to support? I wonder if there's value in creating alternate SecurityManagerPlugins for different use cases, and then if you wanted `dashboard_rbac` for your deployment, you'd use the specific SecurityManager for that use case. Mainly asking as I haven't been following this work too closely, and this looks like a fairly specific change (for example it doesn't apply to Druid datasources and assumes that a BaseDatasource is always a SqlaTable) -- 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]
