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 Vega downloads and download URLs in general #7022
Conversation
Thanks for making a pull request to JupyterLab! To try out this branch on binder, follow this link: |
Before, we would get URLs like http://example.com/files/file.txt/?_xsrf=token, and now the slash after file.txt is eliminated.
…rver. Perhaps we should also use the services package to actually send the request to the server. That would complicate these plugins beyond simple rendermime plugins, though. Fixes jupyterlab#7017
180ce43
to
70ac6fa
Compare
CC @domoritz for a sanity check on how we are overriding the vega sanitizer. |
This should be backported to 1.0.x once it is merged. |
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.
You clicked on the "launch binder" link above? |
Yes |
I'm seeing the same thing. Someone on another issue said that Binder is having issues right now. |
LGTM, thanks! |
@meeseeksdev backport to 1.0.x |
…2-on-1.0.x Backport PR #7022 on branch 1.0.x (Fix Vega downloads and download URLs in general)
References
Fixes #7017
See also vega/altair#1651
Code changes
Two main things:
User-facing changes
Vega plots with json transforms now work.
Backwards-incompatible changes
None.