conda-store-ui 2024.10.1 + memory router

[gabalafou]

Companion to conda-incubator/conda-store-ui#429.

Pull request checklist

• Did you test this change locally?
• n/a Did you update the documentation (if required)?
• n/a Did you add/update relevant tests for this change (if required)?

How to test

Load the extension locally and navigate. Make sure you can open an environment, click the edit button, click the cancel button, go to another environment, etc.

[gabalafou]

This a draft PR because it depends on conda-incubator/conda-store-ui#429 getting merged and a new release of conda-store-ui getting cut.

This PR will have to eventually update package.json with the new, fixed version of conda-store-ui.

[gabalafou]

Now that conda-incubator/conda-store-ui#429 has been merged, we can go ahead with reviewing and merging this PR, so I'm taking it out of draft mode.

[peytondmurray]

Just tested this out locally. To be clear about installation, I'm running docker compose up --build inside the latest commit of conda-store-ui (dc1a1988b5e6212c16439ef38cc0c9be92dc8740, which includes your routing fix) to bring up conda-store at localhost:8080/conda-store, which I see is the default for jupyterlab-conda-store. Is there anything else I need to do to authenticate or anything like that?

I'm getting 404 when I try to use this:

[gabalafou]

@peytondmurray sorry about that! I took this PR out of draft mode without re-reading the PR description.

In order to test the Jupyter frontend extension locally, you have to use an as-of-yet-not-released version of Conda Store UI that incorporates conda-incubator/conda-store-ui#429.

But I thought we could get ahead of the next Conda Store UI release because this change is additive. It doesn't break anything, although I forgot about TypeScript. The TypeScript error should go away once the conda-store-ui dependency is bumped to include a future version that includes the memory router PR.

I guess I should put this back in draft mode until the next release of Conda Store UI, then come back to this PR and update the UI version in package.json. Then it should be possible to follow the test instruction in the PR description and the TypeScript error should go away.

[trallard]

Since we have now a 2024.10.1. release for conda-store-ui can this be taken out of Draft @gabalafou ?

[gabalafou]

I updated the conda-store-ui version in this PR, so it should be possible to follow the manual testing instructions in the PR description.

I went through them myself and everything worked.

Once @peytondmurray does the same, I think this should be good to merge.

[trallard]

I just tested locally, and everything worked as expected.
I only noticed that the version string went missing, but that is not a big deal.

I will leave it to @peytondmurray to review and merge.

[trallard]

LGTM so will go ahead and approve, thanks @gabalafou

[trallard]

CI failure is unrelated so will go ahead an merge (fix for CI is in #66 )