simcha90 commented on a change in pull request #13437:
URL: https://github.com/apache/superset/pull/13437#discussion_r588980235
##########
File path: superset-frontend/src/dataMask/actions.ts
##########
@@ -0,0 +1,62 @@
+/**
Review comment:
I think it's the same reason why we put `chart` in the root, it's
parallel to chart I think... Also this feature have enough functionality to
have independent place in root :) Also I think in future we will need to add
here also `ui filtering` like ui changes that will applied to other charts, may
be also we will need to use here to show how it's applied to itself
----------------------------------------------------------------
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]
