Skip to content

Commit fb5168a

Browse files
committed
Set template config correctly
Turns out just setting it on NotebookApp doesn't set it on ServerApp now, and so all of our templates were actually just no-ops for a while. Discovered while testing jupyterhub#3178
1 parent 5380d1f commit fb5168a

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

mybinder/files/etc/jupyter/jupyter_notebook_config.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
import notebook
55

6-
c.NotebookApp.extra_template_paths.append("/etc/jupyter/templates")
6+
c.ServerApp.extra_template_paths.append("/etc/jupyter/templates")
77

88

99
# For old notebook versions we have to explicitly enable the translation
@@ -21,7 +21,7 @@
2121
# Disable JITSI integration for now
2222
jitsi_url = ""
2323

24-
c.NotebookApp.jinja_template_vars.update(
24+
c.ServerApp.jinja_template_vars.update(
2525
{
2626
"binder_url": binder_launch_host + binder_request,
2727
"persistent_binder_url": binder_launch_host + binder_persistent_request,

mybinder/values.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ tags: {}
4242

4343
etcJupyter:
4444
jupyter_notebook_config.json:
45-
NotebookApp:
45+
ServerApp:
4646
allow_origin: "*"
4747
tornado_settings:
4848
trust_xheaders: true

0 commit comments

Comments
 (0)