feat: allow to embed multiple superset dasboards

platform_plugin_aspects/extensions/filters.py

@@ -35,7 +35,7 @@ def run_filter(
             _ (str): instructor dashboard template name.
         """
         course = context["course"]
-        dashboard_uuid = settings.ASPECTS_INSTRUCTOR_DASHBOARD_UUID
+        dashboards = settings.ASPECTS_INSTRUCTOR_DASHBOARDS
         extra_filters_format = settings.SUPERSET_EXTRA_FILTERS_FORMAT
 
         filters = ASPECTS_SECURITY_FILTERS_FORMAT + extra_filters_format
@@ -45,7 +45,7 @@ def run_filter(
         context = generate_superset_context(
             context,
             user,
-            dashboard_uuid=dashboard_uuid,
+            dashboards=dashboards,
             filters=filters,
         )
 

platform_plugin_aspects/settings/common.py

@@ -20,7 +20,12 @@ def plugin_settings(settings):
         "username": "superset",
         "password": "superset",
     }
-    settings.ASPECTS_INSTRUCTOR_DASHBOARD_UUID = "1d6bf904-f53f-47fd-b1c9-6cd7e284d286"
+    settings.ASPECTS_INSTRUCTOR_DASHBOARDS = [
+        {
+            "name": "Instructor Dashboard",
+            "uuid": "1d6bf904-f53f-47fd-b1c9-6cd7e284d286",
+        },
+    ]
     settings.SUPERSET_EXTRA_FILTERS_FORMAT = []
     settings.EVENT_SINK_CLICKHOUSE_BACKEND_CONFIG = {
         # URL to a running ClickHouse server's HTTP interface. ex: https://foo.openedx.org:8443/ or

platform_plugin_aspects/settings/production.py

@@ -11,10 +11,8 @@ def plugin_settings(settings):
     settings.SUPERSET_CONFIG = getattr(settings, "ENV_TOKENS", {}).get(
         "SUPERSET_CONFIG", settings.SUPERSET_CONFIG
     )
-    settings.ASPECTS_INSTRUCTOR_DASHBOARD_UUID = getattr(
-        settings, "ENV_TOKENS", {}
-    ).get(
-        "ASPECTS_INSTRUCTOR_DASHBOARD_UUID", settings.ASPECTS_INSTRUCTOR_DASHBOARD_UUID
+    settings.ASPECTS_INSTRUCTOR_DASHBOARDS = getattr(settings, "ENV_TOKENS", {}).get(
+        "ASPECTS_INSTRUCTOR_DASHBOARDS", settings.ASPECTS_INSTRUCTOR_DASHBOARDS
     )
     settings.SUPERSET_EXTRA_FILTERS_FORMAT = getattr(settings, "ENV_TOKENS", {}).get(
         "SUPERSET_EXTRA_FILTERS_FORMAT", settings.SUPERSET_EXTRA_FILTERS_FORMAT

platform_plugin_aspects/static/html/superset.html

@@ -2,22 +2,31 @@
 
 <script src="https://unpkg.com/@superset-ui/embedded-sdk"></script>
 
-<div class="email-notifier-instructor-wrapper" width="parent">
+<div class="aspects-wrapper" width="parent">
     <h2>{{display_name}}</h2>
 
     {% if exception %}
     <p>{% trans 'Superset is not configured properly. Please contact your system administrator.'%}</p>
     <p>
         {{exception}}
     </p>
-    {% elif not dashboard_uuid %}
+    {% elif not dashboard_uuid and not superset_dashboards %}
     <p>
         Dashboard UUID is not set. Please set the dashboard UUID in the Studio. {{dashboard_uuid}}
     </p>
     {% elif superset_url and superset_token %}
-    <div class="superset-embedded-container" id="superset-embedded-container-{{xblock_id}}"></div>
+        {% if xblock_id %}
+        <div class="superset-embedded-container" id="superset-embedded-container-{{xblock_id}}"></div>
+        {% else %}
+            {% for dashboard in superset_dashboards %}
+                <div>
+                    <h3>{{dashboard.name}}</h3>
+                </div>
+                <div class="superset-embedded-container" id="superset-embedded-container-{{dashboard.uuid}}"></div>
+            {% endfor %}
+        {% endif %}
     <script type="text/javascript">
-        window.dashboard_uuid ="{{dashboard_uuid}}";
+        window.superset_dashboards = {{superset_dashboards | safe }};
         window.superset_url = "{{superset_url}}";
         window.superset_token = "{{superset_token}}";
     </script>

platform_plugin_aspects/static/js/embed_dashboard.js

@@ -24,7 +24,11 @@ function embedDashboard(dashboard_uuid, superset_url, superset_token, xblock_id)
       when the dashboard is loaded
       */
     });
-}
-if (window.dashboard_uuid !== undefined) {
-  embedDashboard(window.dashboard_uuid, window.superset_url, window.superset_token, window.xblock_id);
+};
+
+if (window.superset_dashboards !== undefined) {
+  window.superset_dashboards.forEach(function(dashboard) {
+    console.log(dashboard)
+    embedDashboard(dashboard.uuid, window.superset_url, window.superset_token, dashboard.uuid);
+  });
 }

platform_plugin_aspects/utils.py

@@ -28,7 +28,7 @@ def _(text):
 def generate_superset_context(  # pylint: disable=dangerous-default-value
     context,
     user,
-    dashboard_uuid=None,
+    dashboards=[],
     filters=[],
 ):
     """
@@ -38,17 +38,17 @@ def generate_superset_context(  # pylint: disable=dangerous-default-value
         context (dict): the context for the instructor dashboard. It must include a course object
         user (XBlockUser or User): the current user.
         superset_config (dict): superset config.
-        dashboard_uuid (str): superset dashboard uuid.
+        dashboards (list): list of superset dashboard uuid.
         filters (list): list of filters to apply to the dashboard.
     """
     course = context["course"]
     superset_config = settings.SUPERSET_CONFIG
 
-    superset_token, dashboard_uuid = _generate_guest_token(
+    superset_token, dashboards = _generate_guest_token(
         user=user,
         course=course,
         superset_config=superset_config,
-        dashboard_uuid=dashboard_uuid,
+        dashboards=dashboards,
         filters=filters,
     )
 
@@ -57,21 +57,21 @@ def generate_superset_context(  # pylint: disable=dangerous-default-value
         context.update(
             {
                 "superset_token": superset_token,
-                "dashboard_uuid": dashboard_uuid,
+                "superset_dashboards": dashboards,
                 "superset_url": superset_url,
             }
         )
     else:
         context.update(
             {
-                "exception": str(dashboard_uuid),
+                "exception": str(dashboards),
             }
         )
 
     return context
 
 
-def _generate_guest_token(user, course, superset_config, dashboard_uuid, filters):
+def _generate_guest_token(user, course, superset_config, dashboards, filters):
     """
     Generate a Superset guest token for the user.
 
@@ -102,16 +102,19 @@ def _generate_guest_token(user, course, superset_config, dashboard_uuid, filters
 
     formatted_filters = [filter.format(course=course, user=user) for filter in filters]
 
-    if not dashboard_uuid:
-        dashboard_uuid = settings.ASPECTS_INSTRUCTOR_DASHBOARD_UUID
+    # if not dashboard_uuid:
+    #    dashboard_uuid = settings.ASPECTS_INSTRUCTOR_DASHBOARD_UUID
 
     data = {
         "user": _superset_user_data(user),
-        "resources": [{"type": "dashboard", "id": dashboard_uuid}],
+        "resources": [
+            {"type": "dashboard", "id": dashboard["uuid"]} for dashboard in dashboards
+        ],
         "rls": [{"clause": filter} for filter in formatted_filters],
     }
 
     try:
+        logger.info(f"Requesting guest token from Superset, {data}")
         response = client.session.post(
             url=f"{superset_internal_host}api/v1/security/guest_token/",
             json=data,
@@ -120,7 +123,7 @@ def _generate_guest_token(user, course, superset_config, dashboard_uuid, filters
         response.raise_for_status()
         token = response.json()["token"]
 
-        return token, dashboard_uuid
+        return token, dashboards
     except Exception as exc:  # pylint: disable=broad-except
         logger.error(exc)
         return None, exc

platform_plugin_aspects/xblock.py

@@ -104,7 +104,7 @@ def student_view(self, context=None):
         context = generate_superset_context(
             context=context,
             user=user,
-            dashboard_uuid=self.dashboard_uuid,
+            dashboards=[{"name": self.display_name, "uuid": self.dashboard_uuid}],
             filters=self.filters,
         )
         context["xblock_id"] = str(self.scope_ids.usage_id.block_id)
@@ -125,7 +125,7 @@ def student_view(self, context=None):
         frag.initialize_js(
             "SupersetXBlock",
             json_args={
-                "dashboard_uuid": context.get("dashboard_uuid"),
+                "dashboard_uuid": self.dashboard_uuid,
                 "superset_url": context.get("superset_url"),
                 "superset_token": context.get("superset_token"),
                 "xblock_id": context.get("xblock_id"),