Skip to content

Conversation

@MORIMOTO520212
Copy link
Contributor

@MORIMOTO520212 MORIMOTO520212 commented Jun 7, 2025

Important

Adds support for $in filter in delete_documents_and_chunks_by_filter in management_service.py to allow deletion of multiple document IDs.

  • Behavior:
    • Adds support for $in filter in delete_documents_and_chunks_by_filter in management_service.py to allow deletion of multiple document IDs.
    • Updates document_ids set with UUIDs from $in filter values.

This description was created by Ellipsis for c095ff8. You can customize this summary. It will automatically update as commits are pushed.

Example of specifying in the body:

{
    "document_id": {
        "$in": ["id1", "id2"]
    }
}

@MORIMOTO520212 MORIMOTO520212 changed the title update: add $in filter to document deletion update: add $in filter to delete_documents_and_chunks_by_filter Jun 7, 2025
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Caution

Changes requested ❌

Reviewed everything up to c095ff8 in 1 minute and 31 seconds. Click for details.
  • Reviewed 15 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

Workflow ID: wflow_nXhqdKODf0yqYtpQ

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
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.

1 participant