rusackas commented on PR #22325: URL: https://github.com/apache/superset/pull/22325#issuecomment-1359968820
@mayurnewase I was just giving the React UI a test, and it's SO MUCH NICER!!! I like the addition of the Name field... gives the user a chance to add something descriptive... as does the description field as well. Now people can better reason about what roles they're discussing within a deployment, and give some historical context as to why they exist or who created/owns them. Heads up that changing `RLS_FORM_QUERY_REL_FIELDS` to `RLS_FILTER_RELATED_FIELDS` may affect deployments that customize this value in various ways - are we OK with this change or would this constitute a breaking change (even if it's easy to correct)? CC @eschutho @john-bodley @dpgaspar -- 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: notifications-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
