- Notifications
You must be signed in to change notification settings - Fork5.5k
Update Jupytext Notebook url path#6961
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
parmentelat commentedJul 6, 2023
cool, thanks@RRosio to haven care of this, please let me know if/when this gets released so I can give it a try |
jtpio commentedJul 17, 2023
@parmentelat were you able to try this change with the built assets for this PR? |
| (widgetName==='default'&&ext==='.ipynb')|| | ||
| widgetName==='Notebook' | ||
| widgetName==='Notebook'|| | ||
| widgetName==='Jupytext Notebook' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Or maybe we can check ifwidgetName contains the stringNotebook?
jupytext is a third-party extension and not officially supported, so not sure we should hardcode its special case here.
parmentelat commentedJul 17, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Hi; unfortunately no, I am mostly away from keyboard these days |
jtpio commentedJul 17, 2023
Normally the built artifact (wheel) can be installed just like a released version with Likely the most important part would be to make sure it is tested in an isolated virtual environment. |
parmentelat commentedJul 17, 2023
I made quick test
however with this setup, on matter how I open a jupytext notebook, I still see the |
RRosio commentedJul 17, 2023
With the PR I observed that right-clicking |
jtpio commentedJul 18, 2023
Normally double clicking should open the document with its default factory. Without configuration this seems to still be But testing with this PR and using the user settings provided by@parmentelat in#6914 (comment) to override the default factory, double clicking seems to be opening the file with jupytext-default-factory-settings.mp4 |
RRosio commentedJul 18, 2023
Ah okay, I was missing the user settings. Once using those settings, I too witness that same behavior. |
jtpio left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thanks@RRosio for looking into this issue!
The change looks good, and should already improve the case of Jupytext 👍

Fixes#6942
I believe this addresses the last point in the issue above, as when a Jupytext Notebook is opened it no longer adds the
/editroute but rather/notebooks.