Skip to content

Add a form in the side panel to export documents to the Cloud #622

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

Merged
merged 9 commits into from
Jul 2, 2025

Conversation

Kerollmops
Copy link
Member

This PR adds a new form for uploading documents to the Cloud (or elsewhere). You can abort the upload if it takes too long or the URL or API key is wrong. The interface shows the progress with a simple percentage on the upload button.

It uses the new export route to perform this operation.

Capture d’écran 2025-06-27 à 14 58 44 Capture d’écran 2025-06-27 à 15 01 22 Capture d’écran 2025-06-27 à 15 06 08 Capture d’écran 2025-06-27 à 15 01 23

@Kerollmops Kerollmops requested review from Strift and mdubus June 27, 2025 13:08
@Kerollmops Kerollmops added the enhancement New feature or request label Jun 30, 2025
@mdubus
Copy link
Member

mdubus commented Jul 2, 2025

@Kerollmops I've fixed the CI and removed the duplicated code.
Can you please check that this is still working as expected? Then we can merge it and release 🙏

I've tried to use this feature but I'm getting a 404 error, not sure why BTW 🤔
Capture d’écran 2025-07-02 à 09 58 12

@Kerollmops
Copy link
Member Author

I've tried to use this feature but I'm getting a 404 error, not sure why BTW 🤔

You are not using the right version of Meilisearch, and the version you have doesn't expose the /export route. Try with Meilisearch on the main branch. You'll also need another instance running on another port. I'll try the feature now 😃

@mdubus
Copy link
Member

mdubus commented Jul 2, 2025

I've tried to use this feature but I'm getting a 404 error, not sure why BTW 🤔

You are not using the right version of Meilisearch, and the version you have doesn't expose the /export route. Try with Meilisearch on the main branch. You'll also need another instance running on another port. I'll try the feature now 😃

Ah yeah that makes sense now that you mention it 🤣
I'll let you test it then, it seems a bit complicated to test it on my side 🙏

@Kerollmops
Copy link
Member Author

Hey @mdubus 👋 After thorough testing, the PR looks good and works perfectly with and without embeddings. The progress works excellently and is helpful. Nice work to both of us ✅ 🙏

@mdubus mdubus merged commit a8d218c into main Jul 2, 2025
4 checks passed
@mdubus mdubus deleted the cloud-export-dataset-form branch July 2, 2025 08:46
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.

2 participants