[GitHub] [superset] dpgaspar commented on issue #13856: Allow enable/disable Alerts and Reports frontend by user

[GitBox <gi...@apache.org>]

dpgaspar commented on issue #13856:
URL: https://github.com/apache/superset/issues/13856#issuecomment-811076967


   Like @nytai said, at the app bootstrap, the routes get registered on Flask. Routes that get registered depending on a feature flag won't get unregistered if the feature flag mutates during a request, for example depending on the user that made the request.
   
   The menu REST endpoint result depend on the registered menus and the user's permission to them, this depends dynamically on the user that made the request and his permissions.
   
   A possible way, don't know if it feasible on Airbnb:
   By default Alerts & Reports permissions are added to all roles, further roles can be created like AlphaNoAlertReports, GammaNoAlertReports and be given to those users. This can be done programatically, has well has the user registration to these roles
   
   
   
    
   
   


-- 
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org