-
Notifications
You must be signed in to change notification settings - Fork 93
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
Deduplicate conda-store in JupyterLab main menu #2347
Deduplicate conda-store in JupyterLab main menu #2347
Conversation
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 @krassowski
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.
LGTM as well, though just to make sure, this is used when the user opts for installing the extension what happens when the extension is not available?
"args": { | ||
"url": "/conda-store", | ||
"text": "Environment Management", | ||
"newBrowserTab": true |
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.
Without this, does the extension still opens as a separate tab?
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.
That code was opening the /conda-store
in a new browser tab.
"text": "Environment Management", | ||
"newBrowserTab": true | ||
} | ||
"command": "condastore:open", |
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.
Hi @krassowski thanks for this, could you explain a bit how this works? does this call the extension, or the endpoint for the conda-store webpage as well?
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.
The extension is included in the default docker image, so there is no opting-in. When extension is not available because We could have it both ways: when extension is available, open it as a JupyterLab tab; when extension is not available open it in a new browser tab. I can make this work straightaway by adding some logic in |
A dedicated logic (new command) in |
Yes, I agree. I was also trying to lean towards this, as we would be able to keep the names "non-dependent" on the extension itself. Would you be able to include these changes as part of this PR, or do you prefer if this is a separate step? |
I will include the changes here. It will require bumping the pin to |
to opening `/conda-store` in new tab otherwise; use the same title for both
Done. I updated top-level issue. This now depends on nebari-dev/nebari-docker-images#130. Tested locally with the new image. Recording: |
Is this and nebari-dev/nebari-docker-images#130 good to merge? |
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.
LGTM!!
Reference Issues or PRs
Uses
jupyterlab-conda-store
if available, fallbacks to opening/conda-store
in new browser tab otherwise. The label in each case is set to "Environment Manager" (but we can adjust it).Before
jupyterlab-conda-store
:After
jupyterlab-conda-store
if available or/conda-store
in new tab otherwise.What does this implement/fix?
Put a
x
in the boxes that applyTesting