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

feat: add refer option to export conversation.json using API #271

Closed
wants to merge 1 commit into from

Conversation

moonmd
Copy link

@moonmd moonmd commented Dec 3, 2024

Rebased #268
Refers conversations.json to the API for export

@pionxzh
Copy link
Owner

pionxzh commented Dec 3, 2024

With the information from #272, I now understand the issue. To handle this problem, I want to do a bigger rework on the export dialog. But it means I cannot merge this PR. I'm sorry about that. 🙏 I feel "refer" would be difficult for users to understand what it is for.

@pionxzh
Copy link
Owner

pionxzh commented Dec 3, 2024

image
something like this

@moonmd
Copy link
Author

moonmd commented Dec 3, 2024

Good idea to break it out to chunks because selecting that many conversations is tedious.
I was trying to figure out a way to shift-select groups, but I couldn't get the shift key to be detected.
"Load All" will never finish if you don't make it "Load 1000". Maybe continue with that concept and give some options.
Load: 100 250 500 750 1000

Also, with a 1000 conversations, it takes a long time to build the zip files. You may want to add a status bar showing that it is still working on it.

@moonmd moonmd closed this Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants