-
Notifications
You must be signed in to change notification settings - Fork 14
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
add dialog checkbox to include token in URL #55
Conversation
something about AsyncIterator
show different warnings under JupyterHub and not
Thanks @minrk for this. Looks like CI is failing due to a lint issue. |
Right this seems to be an issue with JupyterLab 3.6 (jupyterlab/jupyterlab#14029). |
f40e881
to
7c39b3f
Compare
Sorry for dropping off. Fixed lint. |
It's probably ok for now. We can always track this in a separate issue to iterate on it. |
Just tried with the Binder for this PR: https://mybinder.org/v2/gh/minrk/jupyterlab-link-share/include-token-checkbox And it looks like the token is not included after enabling the checkbox and clicking on include-token-url.mp4Although it shows up correctly in the dialog, probably just an issue when copying to the clipboard. |
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.
Thanks!
New release with this change: https://github.com/jupyterlab-contrib/jupyterlab-link-share/releases/tag/v0.3.0 |
This could be considered a solution to #54, though it's the simplest possible one (no settings or server-side override).
There are lots of other ways to address #54 in terms of UX - where/when/who picks which link(s) to share. This is just one option.
In action:
Screen.Recording.2023-02-10.at.10.45.12.mov
Extra bit: I had to bump tsconfig target to es2018 to get it to build for some error with AsyncIterator. I'm not sure why since I didn't change any dependencies.