Closed minrk closed 1 year ago
Thanks @minrk for this.
Looks like CI is failing due to a lint issue.
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.
Right this seems to be an issue with JupyterLab 3.6 (https://github.com/jupyterlab/jupyterlab/issues/14029).
Sorry for dropping off. Fixed lint.
There's a weird CSS jump as the dialog changes sizes when the message changes. I'm not sure how best to fix that.
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 Copy Link
:
https://user-images.githubusercontent.com/591645/225566553-915abc12-6b43-4127-9671-c70ba5cdfdf4.mp4
Although it shows up correctly in the dialog, probably just an issue when copying to the clipboard.
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:
https://user-images.githubusercontent.com/151929/218059111-9a9ef7c3-df78-47e5-9425-dc2cd0ffe25f.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.