Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add limit max feature plus matching tests #60

Closed
wants to merge 3 commits into from

Conversation

shirilad
Copy link

With max property given, the code will now verify that the given value is indeed smaller than max.
If not, it won't allow the value, and return the value without its most right digit.

shirilad added 3 commits May 14, 2017 15:16
With max property given, the code will now verify that the given value
is indeed smaller than max. If not, it won't allow the value, and return
the value without its most right digit
@shirilad
Copy link
Author

The generated files merge might be a bit off. Hope I didn't mess something up there.
I'm guessing a clean bundle run will solve it anyways, but if there's something there needs fixing lmk and I'll try to solve accordingly.

@shirilad shirilad closed this May 15, 2017
@shirilad shirilad reopened this May 15, 2017
@s-yadav
Copy link
Owner

s-yadav commented Jul 16, 2017

Instead of max prop added isAllowed prop to reject a value

@s-yadav s-yadav closed this Jul 16, 2017
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