Skip to content

Fix deleted_tasks field type in TaskDeletion struct #666

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 1 commit into from
Jul 11, 2025

Conversation

coinmoles
Copy link
Contributor

Pull Request

Related issue

Fixes #665

What does this PR do?

  • Change the type of deleted_tasks field in TaskDeletion from usize to Option<usize>
    • This should be Option<usize> since it can be null according to the API documentation

PR checklist

Please check if your PR fulfills the following requirements:

  • Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
  • Have you read the contributing guidelines?
  • Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

@Mubelotix Mubelotix changed the base branch from main to dev July 11, 2025 14:54
Copy link
Collaborator

@Mubelotix Mubelotix left a comment

Choose a reason for hiding this comment

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

Nice catch! Thank you

@Mubelotix Mubelotix merged commit b175a09 into meilisearch:dev Jul 11, 2025
@Mubelotix Mubelotix added the breaking-change The related changes are breaking for the users label Jul 15, 2025
@Mubelotix Mubelotix mentioned this pull request Jul 15, 2025
meili-bors bot added a commit that referenced this pull request Jul 15, 2025
682: Merge dev into main r=curquiza a=Mubelotix

Full changelog after this PR: (includes changes merged before this PR)

## What's Changed

### Breaking changes

* Fix deleted_tasks field type in TaskDeletion struct by `@coinmoles` in #666 (`TaskDeletion.deleted_tasks` and `TaskCancelation.canceled_tasks` have been made `Option`s)

### Enhancements

* Support all experimental features by `@ellnix` in #656
* Add `usedDatabaseSize` to stats by `@Alirexaa` in #653
* Add facet_search API functionality by `@hmacr` in #512
* Allow users to customize facet value sort behaviour by `@hmacr` in #514
* Support `embedders` setting and other vector/hybrid search related configuration by `@CommanderStorm` in #554
* Add documents and embbedings database metrics to stats response by `@Alirexaa` in #652
* Support facet search and prefix search settings by `@ellnix` in #634
* Serialize for MultiSearchResponse by `@milesgranger` in #668
* Implement federated multi search by `@LukasKalbertodt` in #625
* Add support for similar docs query by `@JiaYingZhang` in #674

### Maintainance

* Remove _md code samples for Mintlify migration by `@curquiza` in #661
* Add missing `exist` to `*_update` functions' docs by `@funlennysub` in #617
* Add Code Coverage GH action by `@Alirexaa` in #654








## New Contributors
* `@Alirexaa` made their first contribution in #653
* `@funlennysub` made their first contribution in #617
* `@milesgranger` made their first contribution in #668
* `@coinmoles` made their first contribution in #666
* `@JiaYingZhang` made their first contribution in #674

**Full Changelog**: v0.28.0...v0.29.0


Co-authored-by: hmacr <[email protected]>
Co-authored-by: Clémentine U. - curqui <[email protected]>
Co-authored-by: Frank Elsinga <[email protected]>
Co-authored-by: Clémentine <[email protected]>
Co-authored-by: Vladimir Donich <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change The related changes are breaking for the users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TaskDeletion struct has incorrect type for field deleted_tasks
2 participants