meilisearch / open-api

MIT License
2 stars 1 forks source link

OpenAPI — Verify and fix Content-Type header description #2

Open gmourier opened 1 year ago

gmourier commented 1 year ago

The header content-type seems to have been introduced in places where it isn't required in the open-api spec. Moreover, we don't describe the other accepted content types (ndjson, csv) for the document addition/update routes.

gmourier commented 1 year ago

v1.2 and v1.3 open-API changelog mention a breaking change while it's not true. Wondering if that openAPI warning will solve that situation if managed. (it's a common warning for both versions)

Capture d’écran 2023-07-31 à 16 46 46

https://bump.sh/meilisearch/docs/0772e3a7-2923-4d62-95e3-c4339c525968/versions?filter%5Bstatus%5D=released

We have detected some duplicate 'operationId' attributes on 2 different operations within your file. Please make sure the following operations have a unique identifier: "POST /indexes/{indexUid}/documents/delete", "DELETE /indexes/{indexUid}/documents/{documentId}".