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

Chats directory config #91

Merged
merged 9 commits into from
Oct 29, 2024
Merged

Conversation

brichet
Copy link
Collaborator

@brichet brichet commented Oct 23, 2024

Fixes #61

Allow the user to configure the default directory for chat creation and list (in side panel).

Note

This is a suggestion if we wish to continue listing chat files in a single directory in the side panel.
We could also think of a way to manage chat files in the whole tree file:

  • look for chat files in large tree file without overload
  • display chats coming from several locations (and which can have the same name)
  • create chats in different locations

Copy link
Contributor

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

@brichet brichet added the enhancement New feature or request label Oct 23, 2024
@brichet brichet marked this pull request as ready for review October 24, 2024 14:23
"defaultDirectory": {
"description": "Default directory where to create and look for chat.",
"type": "string",
"default": "",
Copy link
Member

Choose a reason for hiding this comment

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

Should the setting provide more information (for example via the description) on where files are saved by default? Since an empty string can be ambiguous?

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!

@jtpio
Copy link
Member

jtpio commented Oct 28, 2024

btw, maybe we could add a note to the README and / or the changelog that jupyter-chat may break its API often, especially now with the Jupyter AI integration?

Thinking about it since this PR changes a few parameters, function signatures, typings.

@brichet brichet merged commit 528a770 into jupyterlab:main Oct 29, 2024
11 checks passed
@brichet brichet deleted the chats_directory_config branch October 29, 2024 06:23
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.

Path configuration for .chat files
2 participants