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

Conversation

@ManyTheFish
Copy link
Member

Summary

Update payload_size_too_large error related to meilisearch/meilisearch#3739


Changes

{
-    "message": "The provided payload reached the size limit.",
+    "message": "The provided payload reached the size limit. The maximum accepted payload size is :playloadSizeLimit.",
    "code": "payload_too_large",
    "type": "invalid_request",
    "link": "https://docs.meilisearch.com/errors#payload_too_large"
}

@ManyTheFish ManyTheFish requested a review from macraig May 25, 2023 10:22
@ManyTheFish ManyTheFish marked this pull request as ready for review May 25, 2023 10:22
@gmourier gmourier changed the title Update 0061-error-format-and-definitions.md Update payload_too_large error message May 29, 2023
@gmourier gmourier added Ready For Review Feature specification must be reviewed. Implemented Feature specification has been implemented. Q2:2023 v1.2.0 labels May 29, 2023
@gmourier gmourier mentioned this pull request May 29, 2023
1 task
@gmourier gmourier self-requested a review May 29, 2023 06:29
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.

Thank you @ManyTheFish 🚂

@gmourier gmourier removed the Ready For Review Feature specification must be reviewed. label May 29, 2023
Copy link
Contributor

@macraig macraig left a comment

Choose a reason for hiding this comment

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

Thanks for taking this up Many!

@macraig macraig merged commit 0ac6709 into release-v1.2.0 Jun 5, 2023
@macraig macraig deleted the update-payload-size-too-large-error-message branch June 5, 2023 15:45
macraig pushed a commit that referenced this pull request Jun 5, 2023
* Bump OpenAPI version

* CSV Content-Type — Add support for the boolean type in CSV (#228)

* add support for the boolean type in csv

* fix a typo

* Update text/0028-indexing-csv.md

---------

Co-authored-by: Guillaume Mourier <[email protected]>

* Filter operators - `IS NULL` and `IS EMPTY` filter operators (#232)

* Introduce the spec of the IS EMPTY filter

* Introduce the spec of the IS NULL filter

* Fix suggestions

* Uppercase the IN operator

Co-authored-by: Guillaume Mourier <[email protected]>

* Fix a title number issue

Co-authored-by: Guillaume Mourier <[email protected]>

---------

Co-authored-by: Guillaume Mourier <[email protected]>

* Search Algorithms Refactor: Relevancy Changes (#233)

* Remove old exactness-criterion specification

* Remove old asc-desc-criterion specification

* Get and delete documents by filter (#236)

* specify the new fetch documents route

* update open-api

* try to be nice with bumpsh

* specify the new analytics of the get/fetch documents route

* group the analytics behind already used groups

* fix a placeholder text

* group the get and fetch route under a common section

* rename the section to contain the name of both routes

* clarify how to send the parameter to both routes

* fix the type of the parameters

* add the new fetch route to the api keys actions

* capitalize json

Co-authored-by: cvermand <[email protected]>

* update open-api

* Specify the new route to delete documents by filter

* add the new analytics event

* improve a comment

* add the post - delete route to the api key actions

* Fix the json payload

Co-authored-by: cvermand <[email protected]>

* Apply suggestions from code review

Co-authored-by: Guillaume Mourier <[email protected]>

* Update the details

Co-authored-by: Guillaume Mourier <[email protected]>

* delete the invalid_document_delete_filter error code and introduce the missing_document_filter

* fix the open-api thingy

---------

Co-authored-by: cvermand <[email protected]>
Co-authored-by: Guillaume Mourier <[email protected]>

* Auto deletion of tasks (#238)

* auto deletion of tasks

* Update text/0060-tasks-api.md

Co-authored-by: gui machiavelli <[email protected]>

* Update text/0060-tasks-api.md

Co-authored-by: gui machiavelli <[email protected]>

---------

Co-authored-by: gui machiavelli <[email protected]>

* Experimental reduce indexing memory usage (#239)

* Add the experimental-reduce-indexing-memory-usage option in the telemetry policies

* Add the experimental-reduce-indexing-memory-usage option in the instance options

* Update 0055-sort.md (#240)

related to meilisearch/meilisearch#3749

* Update `payload_too_large` error `message` (#241)

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

Update error message in the specification

* Update open-api.yaml

---------

Co-authored-by: Tamo <[email protected]>
Co-authored-by: Clément Renault <[email protected]>
Co-authored-by: cvermand <[email protected]>
Co-authored-by: gui machiavelli <[email protected]>
Co-authored-by: Many the fish <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Implemented Feature specification has been implemented. Q2:2023 v1.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants