-
-
Notifications
You must be signed in to change notification settings - Fork 99
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
Changes needed for improved single document mode #101
Conversation
@@ -24,6 +24,8 @@ | |||
# ----------------------------------------------------------------------------- | |||
|
|||
|
|||
MASTER_URL_PATTERN = '/(?P<mode>lab|doc)(?P<workspace>/workspaces/[a-zA-Z0-9\-\_]+)?(?P<tree>/tree/.*)?' |
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.
We had intentionally made lab
in the url configurable in case folks have a different name for their application.
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 we should be okay with deployers/admins owning the server, port, and base_url
, and deciding that all the paths that get routed by the application server after base_url
belong to us, but I did wonder if we should make this be a major version bump because even if the programmatic API is backward-compatible, URL decisions are also API from my point of view.
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.
With this change the /lab
portion formally becomes part of our API, and this is backwards incompatible. We definitely need a major release for this. Agree with what @afshin is saying here that the server, port and base_url are the configurable portion.
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.
👍
Onward! |
I'll cut a 2.x alpha for this |
The PR contains the changes needed for about to the PR'd work on JupyterLab to improve the single document mode. Here is a summary of the changes to
jupyter_server
:'/(?P<mode>lab|doc)(?P<workspace>/workspaces/[a-zA-Z0-9\-\_]+)?(?P<tree>/tree/.*)?'
workspaces_url
andtree_url
config and page config no longer make sense and have been removed.workspace
,mode
, andtreePath
page config values that are populated by matching the regular expression. Thus, different frontend extensions no longer need to manage their own complex regexp matching to find this information.