-
Notifications
You must be signed in to change notification settings - Fork 171
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
Implement min / max tags and validation - Fixes #226 #229
Conversation
934060b
to
42e37da
Compare
42e37da
to
662928f
Compare
@i-like-robots the -0.6% comes from the private method. I'm not really sure if/how I can make the code coverage cover that. |
border: 1px solid green; | ||
} | ||
|
||
.react-tags.react-tags__invalid { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should name these classes is-valid
and is-invalid
validationState | ||
}) | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we need to store this in state, because the implementation of _getValidationState()
is so simple I think it's safe to call within render()
.
Are there any updates on this? also linking to #226 |
Also wondering if there are any updates on this? |
07a315c
to
dbd521f
Compare
Closing as validation is now supported in v7. |
This is based on the changes in PR #228