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

Remove the unused ISettingRegistry token from the file browser plugin #356

Merged
merged 2 commits into from
Sep 18, 2024

Conversation

trungleduc
Copy link
Member

@trungleduc trungleduc commented Sep 18, 2024

3.0.0b5 is broken due to this token.

Copy link
Contributor

Binder 👈 Launch a Binder on branch trungleduc/jupyter-collaboration/hox-fix

Copy link
Member

@jtpio jtpio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks!

@jtpio jtpio added the bug Something isn't working label Sep 18, 2024
@jtpio
Copy link
Member

jtpio commented Sep 18, 2024

check-links seems to be failing consistently and can likely be tracked separately.

@jtpio jtpio merged commit 7eac099 into jupyterlab:main Sep 18, 2024
19 of 21 checks passed
@trungleduc
Copy link
Member Author

Thanks @jtpio, could you trigger the release action? The current version is broken

@davidbrochart
Copy link
Collaborator

I'm doing it.

@davidbrochart
Copy link
Collaborator

Done.

@krassowski
Copy link
Member

In an unlikely circumstance if anyone else finds them bitten by this - the error on the server side would be:

tornado.web.HTTPError: HTTP 404: Not Found (Settings not found (jupyterlab))

because the .load('jupyter') method was called on settings registry, not on translator. In UI the consequence was that the file browser was not showing up.

Thanks for the quick fix!

Should we add a snapshot test for filebrowser showing up?

andrii-i pushed a commit to andrii-i/jupyter-collaboration that referenced this pull request Feb 11, 2025
…in (jupyterlab#356)

* Remove unused `ISettingRegistry` token

* Lint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants