Skip to content

Conversation

@arjxn-py
Copy link
Member

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

@arjxn-py arjxn-py force-pushed the python-api-new-file branch from c33176f to 580e480 Compare February 6, 2025 12:06
@arjxn-py arjxn-py added the bug PR that fixes a bug label Feb 6, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2025

Preview PR at appsharing.space

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2025

Integration tests report: appsharing.space

@arjxn-py arjxn-py marked this pull request as draft February 6, 2025 13:17
@martinRenou
Copy link
Member

As discussed in private, we should hold before merging this. The choice to create the file from Python is not correct since only the front-end will know the correct drive (in the sense of JupyterLab drive) where to create the file.

@martinRenou
Copy link
Member

Creation of empty files is handled in JS by #680

@arjxn-py arjxn-py deleted the python-api-new-file branch April 8, 2025 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug PR that fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants