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

Basic support for renaming the RTC session #276

Closed
wants to merge 2 commits into from

Conversation

jtpio
Copy link
Member

@jtpio jtpio commented Jul 19, 2021

References

Fixes jupyterlite/jupyterlab-webrtc-docprovider#20

Code changes

Implement simple logic to rename a file, and continue the RTC session seamlessly.

User-facing changes

Users should be able to rename a file. However since the files are distributed (everyone has a copy in indexeddb), users must explicitly save the notebook to get a copy.

Backwards-incompatible changes

@jtpio jtpio added the enhancement New feature or request label Jul 19, 2021
@jtpio
Copy link
Member Author

jtpio commented Jul 19, 2021

Not really working for now.

users must explicitly save the notebook to get a copy.

Also we'll probably want to rename the existing files for the other peers, instead of creating new ones with new names.

@jtpio
Copy link
Member Author

jtpio commented Oct 2, 2021

Posting this PR here as reference as it might be relevant when we resume work here: jupyterlab/jupyterlab#11197

@jtpio
Copy link
Member Author

jtpio commented Dec 21, 2022

Closing as this would likely have to be implemented in https://github.com/jupyterlite/jupyterlab-webrtc-docprovider now (or take a different approach).

@jtpio jtpio closed this Dec 21, 2022
@jtpio jtpio added this to the Reference milestone Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RTC: implement setPath
1 participant