-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
feat: add required
prop to Slider component
#2297
Conversation
Features
Documentation
ContributorsCommit-Lint commandsYou can trigger Commit-Lint actions by commenting on this PR:
|
expect(component.find('input').prop('required')).toBe(true); | ||
}); | ||
|
||
it('should render required asterisk when required prop is true', () => { |
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.
Similar blocks of code found in 3 locations. Consider refactoring.
Visit the preview URL for this PR (updated for commit 1ecf873): https://react-rainbow--pr2297-slider-add-required-1xpt0fmo.web.app (expires Thu, 14 Oct 2021 14:10:58 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 |
required
prop to Slider component
Code Climate has analyzed commit 1ecf873 and detected 1 issue on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
fix: #2295
Changes proposed in this PR:
Add
required
prop to Slider componentImprove Slider components docs
I have followed (at least) the PR section of the contributing guide.