rusackas commented on PR #37773: URL: https://github.com/apache/superset/pull/37773#issuecomment-4699150562
Thanks for sticking with this @jayvenn21. It's gone conflicting with master, so it'll need a rebase. One open question from @dpgaspar still stands: he floated doing this as a FAB config (like FAB_ADD_SECURITY_VIEWS) so the views just never register, which is cleaner than gating here. Worth deciding which direction we want before we run CI again? -- 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]
