codenamelxl commented on PR #32340: URL: https://github.com/apache/superset/pull/32340#issuecomment-2677451721
> I see that you're removing it from Alpha, but I don't see it being added to other roles. Should it be? Just making sure the work matches the PR description :) Hi, Thanks for the quick look. Originally I removed it from Alpha_Only config so that it will be granted to all Standard Roles, not just Alpha. This works, for Gamma but not for sql_lab. I have moved the TableSchemaView to only granting to sql_lab to be similar to TabStateView (which I think should be more desirable since the view is only being used by sql_lab). The MR title is changed accordingly. -- 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]
