Skip to content
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

De-/Serialization to allow full bidirectional notebook editing #73

Merged
merged 4 commits into from
Dec 8, 2022

Conversation

sourishkrout
Copy link
Member

@sourishkrout sourishkrout commented Dec 2, 2022

Working branch to test and work out integration.

Note: dev wasm lib needs to be manually copied for now. Not expecting tests to pass until PR is finalized.

@sourishkrout sourishkrout marked this pull request as draft December 2, 2022 16:38
@sourishkrout sourishkrout changed the base branch from main to cb-editability December 8, 2022 14:52
@sourishkrout sourishkrout marked this pull request as ready for review December 8, 2022 14:52
@sourishkrout sourishkrout merged commit 17e954a into cb-editability Dec 8, 2022
@sourishkrout sourishkrout deleted the wasm-interface branch December 8, 2022 14:53
sourishkrout added a commit that referenced this pull request Dec 8, 2022
sourishkrout added a commit that referenced this pull request Dec 12, 2022
sourishkrout added a commit that referenced this pull request Dec 13, 2022
sourishkrout added a commit that referenced this pull request Dec 13, 2022
sourishkrout added a commit that referenced this pull request Dec 13, 2022
…ace (#81)

* re-attempt to implement editing capabilities
* block editing for files that aren't checked in
* De-/Serialization to allow full bidirectional notebook editing (#73)
* First revision of WASM serializer interface
* No more Yarn
* Keep testability of wasmReady
* Use metadata as is instead of attributes top key
* Sunset read-only serializer
* Fix git tracked verification
* Deal with prompts
* Yarn -> NPM

Co-authored-by: Christian Bromann <git@bromann.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants