Skip to content

Commit 76c5b40

Browse files
eschuthoclaude
andcommitted
fix(embedded): fix template context for embedded views
- Use render_app_template() in EmbeddedView to provide proper template context - Move tokens variable definition before its usage in spa.html template - Fixes 'tokens is undefined' error in embedded dashboard views 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 9617b8b commit 76c5b40

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

superset/embedded/view.py

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
from superset import event_logger, is_feature_enabled
2525
from superset.daos.dashboard import EmbeddedDashboardDAO
2626
from superset.superset_typing import FlaskResponse
27-
from superset.utils import json
2827
from superset.views.base import BaseSupersetView, common_bootstrap_payload
2928

3029

@@ -76,7 +75,7 @@ def embedded(
7675
dashboard_version="v2",
7776
)
7877

79-
bootstrap_data = {
78+
extra_bootstrap_data = {
8079
"config": {
8180
"GUEST_TOKEN_HEADER_NAME": current_app.config["GUEST_TOKEN_HEADER_NAME"]
8281
},
@@ -86,10 +85,7 @@ def embedded(
8685
},
8786
}
8887

89-
return self.render_template(
90-
"superset/spa.html",
88+
return self.render_app_template(
89+
extra_bootstrap_data=extra_bootstrap_data,
9190
entry="embedded",
92-
bootstrap_data=json.dumps(
93-
bootstrap_data, default=json.pessimistic_json_iso_dttm_ser
94-
),
9591
)

superset/templates/superset/spa.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,11 @@
6969
{% endblock %}
7070
</head>
7171

72+
{% set tokens = theme_tokens | default({}) %}
7273
<body {% if standalone_mode %}class="standalone"{% endif %} style="margin: 0; padding: 0; background-color: {{ tokens.get('colorBgBase', '#ffffff') }};">
7374

7475
{% block body %}
7576
<div id="app" data-bootstrap="{{ bootstrap_data }}">
76-
{% set tokens = theme_tokens | default({}) %}
7777
{% set spinner_style = "width: 70px; height: auto; position: absolute; top: 50%; left: 50%; transform: translate(-50%, -50%);" %}
7878

7979
{% if spinner_svg %}

0 commit comments

Comments
 (0)