Skip to content
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

Launch into /lab by default by changing TLJH config's default value #775

Merged
merged 4 commits into from
Aug 9, 2023

Conversation

raybellwaves
Copy link
Contributor

@raybellwaves raybellwaves commented Dec 17, 2021

This PR changes the default value for the TLJH config user_environment.default_app to jupyterlab, which makes c.Spawner.default_url be set to /lab by default in TLJH.

To revert to the behavior before this, a TLJH admin can do:

sudo tljh-config set user_environment.default_app classic

  • Add / update documentation
  • Add tests

Closes #773

@raybellwaves raybellwaves mentioned this pull request Dec 17, 2021
2 tasks
@raybellwaves raybellwaves changed the title WIP: try default_app launch on lab by default Dec 17, 2021
Copy link
Member

@GeorgianaElena GeorgianaElena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for opening this @raybellwaves!

I don't feel comfortable enough to merge this, because I believe this needs some discussion before. For example, there is a conversation about this in z2jh which I believe concluded that migrating to JupyterHub 2.0 which defaults to JupyterLab is the way to go.

Maybe we can do the same for TLJH and focus on migrating to JupyterHub 2.0, and then figure what needs to change in order to accommodate the hub's default interface.

@manics
Copy link
Member

manics commented Dec 17, 2021

If we resolve #724 this should be easier to deal with

@consideRatio
Copy link
Member

I rebased this PR and pushed a commit updating the docs about this change, and adding a breaking change changelog entry about it.

@consideRatio consideRatio changed the title launch on lab by default Launch into /lab by default by changing TLJH config's default value Aug 9, 2023
@consideRatio consideRatio merged commit 55db9c5 into jupyterhub:main Aug 9, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

launch on /lab by default
5 participants