Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Specify the effect of the * value when used in the task filters #218

Merged
merged 1 commit into from
Feb 6, 2023

Conversation

irevoire
Copy link
Member

@irevoire irevoire commented Jan 12, 2023

Summary

meilisearch/meilisearch#3342

Specify the effect of the * value when used in the task filters.

Changes

Specify the effect of the * value when used in the task filters.

Attention To Reviewers

I specified the effect of the * value when used in the task filters.

@irevoire irevoire requested a review from gmourier January 12, 2023 16:57
@gmourier gmourier added Catch-Up The PR describes already implemented behaviors. Ready For Review Feature specification must be reviewed. Patch This PR patch a feature specification and removed Patch This PR patch a feature specification labels Jan 16, 2023
Copy link
Member

@gmourier gmourier left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you for the precision and the catch-up @irevoire! Can you link the spec to the related issue on the Meilisearch repository, please?

@gmourier gmourier mentioned this pull request Jan 19, 2023
1 task
@gmourier gmourier merged commit 74221ce into release-v1.0.0 Feb 6, 2023
@gmourier gmourier deleted the irevoire-patch-2 branch February 6, 2023 10:47
@gmourier gmourier removed the Ready For Review Feature specification must be reviewed. label Feb 6, 2023
gmourier added a commit that referenced this pull request Feb 6, 2023
* Bump open-api spec

* Simplify dump version section since v1.0 can import all dumps (#203)

* Simplify dump version section since v1.0 can import all dumps

* Relax target import version

Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com>

---------

Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com>

* Rename dump command from `--dumps-dir` to `--dump-dir` (#204)

* Propagate the change to --dumps-dir to the specification

* Rename section header in openAPI documentation

* Dump destination -> Dump directory

* Finish destination -> directory renaming

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

---------

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* Update the specification of soft-deleted documents (#206)

* Grammarly'd soft-deletion page

* Soft-deleted: update the functional specification

* Remove `--max-index-size` and `--max-task-db-size` (#207)

* Robustify Primary Key Inference - Update related errors (#208)

* update inference errors

* Add `index_` prefix to newly-introduced error codes

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* Update error messages following meilisearch/meilisearch#3301

---------

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* Reject master keys shorter than 16 bytes in `production` (#209)

* Specify what happens when too short a master key is provided

* Change error message

* Characters -> bytes for master key length

* Specify Error and Warning messages regarding the master-key

* Update text/0119-instance-options.md

* Update text/0119-instance-options.md

* Specify warning messages when a master key is missing or lower than 16 bytes in env development

* Apply suggestions from code review

Co-authored-by: Maryam <90181761+maryamsulemani97@users.noreply.github.com>

* Replace export MEILI_MASTER_KEY by --master-key

* Apply suggestions from code review

Co-authored-by: Louis Dureuil <louis.dureuil@gmail.com>

---------

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>
Co-authored-by: Maryam <90181761+maryamsulemani97@users.noreply.github.com>

* Add `OFF` log level to spec (#211)

* Errors handling enhancement (#212)

* Replace invalid_duplicate_index_found error code to invalid_swap_duplicate_index_found

* Remove bad_request for settings properties error; replaced with dedicated error codes

* cleaning old bad usage of the spec and replaces it with relevant information

* Split search bad_request error code to dedicated error code

* Catch-up: remove unused dump_not_found error code

* Do an update pass on old specs format

* Details context, variant cases

* Add system type;Split invalid_task_date_filter to dedicated error codes

* Precise index API bad_request error code to dedicated one

* Precise bad_request to a dedicated code for the swap indexes API

* Use invalid_index_primary_key when ?primaryKey is invalid instead of bad_request; + add TODO bad_request we forgot

* marking missed bad_request on /keys

* Add missed bad_request codes for GET /keys

* Add missed bad_request codes for GET /documents

* Removes _filter suffix from query parameter error_code for tasks API; Add missing bad_request error_code on GET /tasks

* Add missing bad_request error codes on GET /indexes

* remove future possibilities

* Replace search_Parameter by search into the /search error codes naming

* Replace bad_request code when the mandatory uid is missing when posting an index resource; split the generic missing_parameter to dedicated error codes

* fix sentence sense

* Split immutable_field to dedicated api key fields

* Removes missing todo marker; Add missing part

* Add suggested review comments

* Catch-up/Add  when an invalid index uid is gen for the :indexUid path parameter

* Add immutable_index_uid error code on PATCH /indexes/:indexUid when uid is specified in the request payload

* invalid_settings_ranking_rules can only be synchronous

* Details async/sync case for error related to minWordsSizeForTypos object

* Add immutable_index_created_at and immutable_index_updated_at for the Index API Resource

* Add missing_swap_indexes on the POST swap-indexes resource

* Update text/0061-error-format-and-definitions.md

Co-authored-by: Bruno Casali <brunoocasali@gmail.com>

* merge the :deserr_helper and the :deserr_location

* Rephrase and catch-up miss

* get rids of the index_not_accessible error code since no one remember about it

---------

Co-authored-by: Bruno Casali <brunoocasali@gmail.com>
Co-authored-by: Tamo <tamo@meilisearch.com>

* Add error message when doing a migration (#213)

* Add the migration CLI error

* Update text/0105-dumps-api.md

---------

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* Merge snapshot options (#214)

* Merge `--schedule-snapshot` and `--snapshot-interval-sec` options

* Update text/0119-instance-options.md

---------

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* Remove any reference to the disable-auto-batching argument (#215)

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* Specify the effect of the `*` value when used in the task filters (#218)

---------

Co-authored-by: Louis Dureuil <louis.dureuil@gmail.com>
Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com>
Co-authored-by: Maryam <90181761+maryamsulemani97@users.noreply.github.com>
Co-authored-by: Bruno Casali <brunoocasali@gmail.com>
Co-authored-by: Tamo <tamo@meilisearch.com>
Co-authored-by: Clément Renault <clement@meilisearch.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Catch-Up The PR describes already implemented behaviors. Q1:2023 v1.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants