amitmiran137 commented on pull request #13992: URL: https://github.com/apache/superset/pull/13992#issuecomment-815070794
> 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) the entire feature is under the DASHBOARD_RBAC ff and about the druid datasources, I'll change it to use the BaseDatasource -- 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]
