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

Fix tree URLs to work when there is a workspace in the URL. #7174

Merged
merged 1 commit into from Sep 9, 2019
Merged

Fix tree URLs to work when there is a workspace in the URL. #7174

merged 1 commit into from Sep 9, 2019

Conversation

afshin
Copy link
Member

@afshin afshin commented Sep 9, 2019

This is a bug fix of a regression that prevented /tree/ URLs from working when there was a workspace in the URL.

References

Fixes #7156

Code changes

Fixes a regular expression.

User-facing changes

/tree/ URLs with workspaces work now.

Backwards-incompatible changes

N/A

@afshin afshin added the bug label Sep 9, 2019
@afshin afshin self-assigned this Sep 9, 2019
@jupyterlab-dev-mode
Copy link

Thanks for making a pull request to JupyterLab!

To try out this branch on binder, follow this link: Binder

Copy link
Member

@ian-r-rose ian-r-rose left a comment

Choose a reason for hiding this comment

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

I've confirmed that this fixes the issue. Thanks @afshin!

@ian-r-rose ian-r-rose added this to the 1.1.x milestone Sep 9, 2019
@afshin afshin merged commit b80c738 into jupyterlab:master Sep 9, 2019
@jasongrout
Copy link
Contributor

Should this be backported to 1.1.x?

@jasongrout jasongrout modified the milestones: 1.1.x, 1.2 Sep 10, 2019
@afshin
Copy link
Member Author

afshin commented Sep 12, 2019

@jasongrout Yes, I think so.

@jasongrout
Copy link
Contributor

@meeseeksdev backport to 1.1.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Sep 12, 2019
blink1073 added a commit that referenced this pull request Sep 16, 2019
…4-on-1.1.x

Backport PR #7174 on branch 1.1.x (Fix tree URLs to work when there is a workspace in the URL.)
@lock lock bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Oct 12, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. Tag:Workspaces
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Combined URL (workspaces + tree) does not work as intended
3 participants