-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Extract @jupyterlab/shared-models to @jupyter-notebook/ydoc #13389
Extract @jupyterlab/shared-models to @jupyter-notebook/ydoc #13389
Conversation
Thanks for making a pull request to jupyterlab! |
@meeseeksdev please backport to 3.6.x |
Oops, something went wrong applying the patch ... Please have a look at my logs. |
@meeseeksdev please backport to 3.6.x |
Oops, something went wrong applying the patch ... Please have a look at my logs. |
Should the extension migration be updated to indicate that The current version still indicates the following: https://jupyterlab.readthedocs.io/en/latest/extension/extension_migration.html#api-breaking-changes |
…ab#13389) * Switch to @jupyter-notebook/ydoc * Move `fromJSON` in `@jupyter-notebook/ydoc` * Bump to @jupyter-notebook/ydoc 0.2.0-alpha.1 * Don't modify the staging folder * Remove shared model from CI * Fix example * Switch to final 0.2.0 ydoc release
References
Follow-up of jupyterlab/frontends-team-compass#152 (comment)
Related PR: jupyter-server/jupyter_ydoc#86
Code changes
Extract the Y document schemas in a mono repo along side the Python version of the schemas.
The
fromJSON
has been extracted too has it requires to know internal details of the document schemas; hence is better handle directly over there.User-facing changes
N/A
Backwards-incompatible changes
The API did not changed but is provided by another package.