Evan Rusackas
Evan Rusackas
Hi @mcdogg17 👋 This PR has been inactive for about 6 months and has failing CI tests (mysql, postgres, sqlite, unit-tests). Converting to draft until the test failures are resolved....
This has been silent for quite a while. I just tried on current Superset/Preset, and it seems to work as expected. Closing as not able to reproduce, but happy to...
^ Does that solve it? Maybe we just need some documentation updates? CC @villebro
@dependabot rebase
Seems about right to me, but should we also remove [this](https://github.com/apache/superset/blob/d11b6d557e06579e735f58628d6cbdc106b4009e/superset/security/manager.py#L278) while we're at it?
I went ahead and removed `RowLevelSecurityFiltersModelView` from the security manager file as well, since it appears obsolete.
Maybe @geido or @msyavuz can make sure i'm not being dumb in removing that... pretty sure it's no longer used, but would love a second pair of eyes ;)
Hi @xavier-GitHub76 👋 This PR has merge conflicts and is 1688 commits behind master. Converting to draft until resolved. When you have time, please rebase on `master` to resolve the...
Would you be willing to open a quick PR to patch this? I'm not sure if it'll make any practical difference in this case though :/
Thanks for opening the PR. It seems to need a little touch-up, but hopefully we can close all this out soon :D