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

Use the url resolver to its full capabilities for vega. #7098

Merged
merged 1 commit into from Aug 28, 2019

Conversation

jasongrout
Copy link
Contributor

References

Fixes #7047

Extends #7022

Code changes

The vega renderers were using the resolver a little bit, but needed to use it to its full extent.

User-facing changes

Backwards-incompatible changes

None

@jupyterlab-dev-mode
Copy link

Thanks for making a pull request to JupyterLab!

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

@jasongrout
Copy link
Contributor Author

CC @blink1073 or anyone else - I'd like to get this bugfix in 1.1.

Copy link
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@blink1073 blink1073 merged commit 105f7af into jupyterlab:master Aug 28, 2019
@jasongrout
Copy link
Contributor Author

@meeseeksdev backport to 1.0.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Aug 28, 2019
jasongrout added a commit that referenced this pull request Aug 28, 2019
…8-on-1.0.x

Backport PR #7098 on branch 1.0.x (Use the url resolver to its full capabilities for vega.)
@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 Sep 27, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Sep 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug pkg:vega status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vega extension not loading in subdirectory
2 participants