Skip to content

Fix shared document interface#677

Merged
martinRenou merged 5 commits intojupytercad:mainfrom
arjxn-py:fix-shared-document-interface
Feb 6, 2025
Merged

Fix shared document interface#677
martinRenou merged 5 commits intojupytercad:mainfrom
arjxn-py:fix-shared-document-interface

Conversation

@arjxn-py
Copy link
Member

@arjxn-py arjxn-py commented Feb 6, 2025

Copy link
Member

@martinRenou martinRenou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@martinRenou
Copy link
Member

It looks like https://github.com/jupytercad/JupyterCAD/blob/main/packages/schema/src/doc.ts#L69 should be using JSONObject

@arjxn-py
Copy link
Member Author

arjxn-py commented Feb 6, 2025

It looks like main/packages/schema/src/doc.ts#L69 should be using JSONObject

Thanks. Yes, i'm on it :)

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2025

Integration tests report: appsharing.space

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2025

Preview PR at appsharing.space

@arjxn-py arjxn-py requested a review from martinRenou February 6, 2025 11:16
@martinRenou
Copy link
Member

martinRenou commented Feb 6, 2025

Mmh, you should change setSource too. Not sure why TypeScript doesn't complain.

@arjxn-py arjxn-py force-pushed the fix-shared-document-interface branch from 048c7f9 to c33410f Compare February 6, 2025 11:48
Copy link
Member

@martinRenou martinRenou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@martinRenou martinRenou merged commit a777c33 into jupytercad:main Feb 6, 2025
10 checks passed
@arjxn-py arjxn-py deleted the fix-shared-document-interface branch April 8, 2025 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants