-
Notifications
You must be signed in to change notification settings - Fork 391
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document how to customize page #1923
base: main
Are you sure you want to change the base?
Conversation
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
|
||
BinderHub uses `Jinja <https://jinja.palletsprojects.com/en/stable/>`_ as template engine | ||
to process the page template `binderhub/templates/page.html <https://github.com/jupyterhub/binderhub/blob/main/binderhub/templates/page.html>`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's something wrong with the link syntax- it's not linked:
https://binderhub--1923.org.readthedocs.build/en/1923/customizing.html#header-and-footer-customization
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it'd be helpful to also highlight the important parts of the template- is <div id="root"></div>
the critical bit?
release: {{ .Release.Name }} | ||
data: | ||
{{- (.Files.Glob "files/*").AsConfig | nindent 2 }} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is additional Helm configuration needed to mount the file in the ConfigMap into the BinderHub pod?
I will address the comments in a couple of weeks. |
Closes #1907
We might want to make add a new
traitlets
for users to provide their own Jinja template that will be used inbinderhub/binderhub/main.py
Line 42 in 4f0a91a
binderhub/binderhub/main.py
Lines 41 to 46 in 4f0a91a