https://kotlinlang.org logo
#http4k
Title
# http4k
v

Vojtěch Knyttl

02/16/2021, 2:24 PM
I am not sure whether it is a bug or feature - consider having lens
Copy code
<http://Query.int|Query.int>().required("minValue", "Required minimal value")
This will nicely generate OpenApi description with
integer
definition. This seems to be the result of
IntegerParam
in
mapWithNewMeta(StringBiDiMappings.long(), IntegerParam)
. However, when using
.multi
, this doesn't change, the OpenApi definition stays at
integer
. I can se ParamMeta contains also
ArrayParam
, so I guess this should ideally result in somehting like
integer array
? If I understand correctly, the
ArrayParam
is not used for .
multi
and I am not sure how this should be done properly?
@dave @s4nchez I can create an issue or I can also try to fix this, just I would like to know the expected behaviour here
d

dave

02/17/2021, 8:05 PM
I've fixed this in master in OA2 and OA3, so it will go out with the next release. 🙂
v

Vojtěch Knyttl

02/17/2021, 8:21 PM
👍 thanks!
okay that seems to be a bigger change than I expected 🙂
thanks for the quick fix!
d

dave

02/17/2021, 8:24 PM
Yeah- I realised where the bodies are buried and wondered how hard it would be..
v

Vojtěch Knyttl

02/17/2021, 8:25 PM
can't wait for the release!
superb
4 Views