extended-validation @Digits - make fraction part optional #3346
Replies: 1 comment 6 replies
-
|
Can you please expand your example please with actual examples. Are you talking about the AST grammar, the scalar inputs / outputs of something else ? |
Beta Was this translation helpful? Give feedback.
-
|
is it not a https://github.com/graphql-java/graphql-java-extended-validation#range |
Beta Was this translation helpful? Give feedback.
-
|
won't work, see here. My plan would be to make this possible
is something different. I also want to support numbers like 99999.99999, but not "100000". Range is inclusive |
Beta Was this translation helpful? Give feedback.
-
|
gotcha - yeah it seems fair one way might be to make it That way when its not specified you accept any number of fractional parts |
Beta Was this translation helpful? Give feedback.
-
|
yeah but that seems very not obvious and verbose :) Ok I try to create PR and let's see what owners think |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
We have a requirement, that we check only integer part of number, but not the fractional. Looks like DigitsConstraint does not support that.
I can raise a PR, but I want to check first whether it's acceptable change.
Beta Was this translation helpful? Give feedback.
All reactions