Skip to content

fix: support negative range values#452

Open
Jesse0Michael wants to merge 1 commit intoasaskevich:masterfrom
Jesse0Michael:negative-range
Open

fix: support negative range values#452
Jesse0Michael wants to merge 1 commit intoasaskevich:masterfrom
Jesse0Michael:negative-range

Conversation

@Jesse0Michael
Copy link
Copy Markdown

@Jesse0Michael Jesse0Michael commented Oct 6, 2021

Q A
Documentation no
Bugfix no
BC Break no
New Feature yes
RFC no
QA yes

Description

allow the range validator to support negative param values by adjusting the regex. ie: range(-10|10)

Currently you would get the error: Range3: The following validator is invalid or can't be applied to the field: "range(-10|10)"


This change is Reviewable

allow the range validator to support negative param values. ie: `range(-180|180)`
@Jesse0Michael
Copy link
Copy Markdown
Author

Sorry, When i initially went looking for this feature i didn't find:

#441
#442

but found those when looking into another issue

@sergeyglazyrindev
Copy link
Copy Markdown

Hello guys!
I forked this package cause owner disappeared. Hope, he will be back, but it would be easier to merge these changes back if he is back
Link to my repo: create issue there and we'll discuss it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants