Skip to content

feat(ai): spreadsheet conversion input filters#4843

Merged
juliusknorr merged 1 commit into
mainfrom
feat/ai-csv-conversion
Jun 12, 2025
Merged

feat(ai): spreadsheet conversion input filters#4843
juliusknorr merged 1 commit into
mainfrom
feat/ai-csv-conversion

Conversation

@elzody
Copy link
Copy Markdown
Collaborator

@elzody elzody commented Jun 11, 2025

Summary

Pass the required input filters to correctly parse the CSV output that the LLM generates. We assume that the LLM generates an output that always uses a set of standards, like , for the field separator and " as the text delimiter.

The RemoteService::convertTo() method can now accept conversion options which are mixed in with the multipart form data, as required to pass input filters. But they are optional.

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Documentation (manuals or wiki) has been updated or is not required

@elzody elzody requested a review from juliusknorr as a code owner June 11, 2025 20:18
@elzody elzody requested a review from julien-nc June 11, 2025 20:19
@elzody elzody self-assigned this Jun 11, 2025
@elzody elzody added the 3. to review Ready to be reviewed label Jun 11, 2025
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Jun 11, 2025
@elzody elzody moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity team Jun 11, 2025
Signed-off-by: Elizabeth Danzberger <lizzy7128@tutanota.de>
@juliusknorr
Copy link
Copy Markdown
Member

Cypress failure seems to originate from server somehow

@juliusknorr juliusknorr merged commit b490cd3 into main Jun 12, 2025
65 of 68 checks passed
@juliusknorr juliusknorr deleted the feat/ai-csv-conversion branch June 12, 2025 05:46
@github-project-automation github-project-automation Bot moved this from 👀 In review to ☑️ Done in 📝 Productivity team Jun 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Ready to be reviewed

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants