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

Compatibility with notebook #112

Merged
merged 8 commits into from
Dec 9, 2024
Merged

Compatibility with notebook #112

merged 8 commits into from
Dec 9, 2024

Conversation

brichet
Copy link
Collaborator

@brichet brichet commented Dec 6, 2024

This PR enable jupyterlab_chat in Notebook.

Fixes #105

  • Add integration test to ensure the chat is opening in side panel and as single document widget

@brichet brichet added the enhancement New feature or request label Dec 6, 2024
Copy link
Contributor

github-actions bot commented Dec 6, 2024

Binder 👈 Launch a Binder on branch brichet/jupyter-chat/notebook

@brichet
Copy link
Collaborator Author

brichet commented Dec 6, 2024

Bump CI

@brichet brichet closed this Dec 6, 2024
@brichet brichet reopened this Dec 6, 2024
@brichet
Copy link
Collaborator Author

brichet commented Dec 6, 2024

Bump CI again

@brichet brichet closed this Dec 6, 2024
@brichet brichet reopened this Dec 6, 2024
@brichet brichet marked this pull request as ready for review December 7, 2024 14:46
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.

Thanks!

*
* NOTES: In Notebook application, the panel is collapsed when using the
* `activateById` when it is already opened.
* See https://github.com/jupyter/notebook/issues/7534
Copy link
Member

Choose a reason for hiding this comment

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

It would indeed be nice to fix this upstream, so we could then remove that logic here and the dependency on @jupyter-notebook/application.

@brichet brichet merged commit e8aa316 into jupyterlab:main Dec 9, 2024
11 checks passed
@brichet brichet deleted the notebook branch December 9, 2024 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compatibility with Notebook
2 participants