Array of entities in request body
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Start in lib/grape-swagger/doc_methods/move_params.rb at MoveParams#build_definition, then read the related issue and the request-body examples. Compare the generated specification with Grape's requirement for an object containing the array under a top-level key. Done means the specification represents the intended JSON object without requiring a dummy optional parameter.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Related issue: https://github.com/ruby-grape/grape-entity/issues/252
As for example above, we want request body to be:
{
"accounts": [
{
"cma": 0,
"name": "string",
"environment": "string",
"sites": 0,
"username": "string",
"password": "string"
}
]
}
But actually:
[
{
"cma": 0,
"name": "string",
"environment": "string",
"sites": 0,
"username": "string",
"password": "string"
}
]
Grape also wants the array to be in the key of top level JSON object.
( Top level array data cannot be handled by Grape out of the box ( cf. https://github.com/ruby-grape/grape/issues/1730 ). )
So it seems the generated specification is broken.
A workaround is to adding dummy optional parameter.
It forces the request body to be a JSON object.
I've found that making MoveParams#build_definition to always use object_type fixes the issue.
Is this reasonable fix?
- Lingua principale
- Ruby
- Stelle
- 1.1k
- Fork
- 477
- Merge medio
- 5h 25m
- PR unite (30g)
- 3
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di ruby-grape/grape-swagger
-
Support default_response and default in 2.3; stop reading route.descForse già presa @numbata l’ha presa 57 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
ruby-grape/grape-swagger#989 ·
I maintainer di solito rispondono entro 1 giorno
-
Length limits (minLength/maxLength, minItems/maxItems) are dropped from body params moved into a definitionForse già presa @numbata l’ha presa 8 giorni fa. Aperta
ruby-grape/grape-swagger#1010 · 1 commento · 1 reazione · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
ruby-grape/grape-swagger#987 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Bug report: omitted array parameters with `array_use_braces: true`Forse già presa @olivier-thatch l’ha presa 531 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
ruby-grape/grape-swagger#952 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Bug Report: `default` for array params (`type: [String]`) is incorrectly placed inside `items`Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
ruby-grape/grape-swagger#950 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di ruby-grape/grape-swagger
Issue simili
-
JWT::EncodedToken raises TypeError for a non-object headerForse già presa @anakinj l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
jwt/ruby-jwt#780 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 70/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
zold-io/zold-ruby-sdk#85 ·