mistercrunch commented on a change in pull request #4463: New Landing Page v1.0
URL: 
https://github.com/apache/incubator-superset/pull/4463#discussion_r170333861
 
 

 ##########
 File path: superset/views/utils.py
 ##########
 @@ -0,0 +1,53 @@
+from __future__ import absolute_import
+from __future__ import division
+from __future__ import print_function
+from __future__ import unicode_literals
+
+from collections import defaultdict
+
+from flask import g
+from flask_appbuilder.security.sqla import models as ab_models
+
+from superset import db
+
+
+def bootstrap_user_data(given_username=None):
+    if given_username:
+        username = given_username
+    else:
+        username = g.user.username
+
+    user = (
+        db.session.query(ab_models.User)
+        .filter_by(username=username)
+        .one()
+    )
+    roles = {}
+    permissions = defaultdict(set)
+    for role in user.roles:
 
 Review comment:
   For the welcome page you don't need all that in the payload, maybe this 
utility function should have a `include_perms=False` arg. Man I wish we had 
GraphQL instead of having to change the backend logic...

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to