-
Notifications
You must be signed in to change notification settings - Fork 8
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
Migrate to prebuilt extension to support JupyterLab v3.x #50
Conversation
ec24c41
to
8d0c85b
Compare
Yes, if there is no way to support both at once, 2 branches sounds good (with a v1.x branch?)... and the sooner we can drop the older branch the simpler. |
21fc93c
to
53ea29e
Compare
29a0980
to
13e9d92
Compare
f8f8781
to
f16e1f4
Compare
If we break compatiblity with JLab 2, we might as well cross the bridge entirely and migrate to a prebuilt extension. Therefore:
|
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.
Awesome stuff!! 🎉
@@ -110,6 +89,9 @@ See https://github.com/silx-kit/jupyterlab-h5web/releases. | |||
|
|||
## Troubleshoot | |||
|
|||
If the following instructions do not solve your issue, you can open an | |||
[issue on GitHub](https://github.com/silx-kit/jupyterlab-h5web/issues). |
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 could also point to h5web@esrf.fr
for people without a github account
Co-authored-by: Max Klein <telamonian@users.noreply.github.com>
Fix #49
Unfortunately, this breaks compatibility with JupyterLab 2.x. We might need to work on separate branches/releases.