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

docs: move default props to Dropzone component for react-docgen #1015

Merged

Conversation

samuelfullerthomas
Copy link
Contributor

What kind of change does this PR introduce?

  • bugfix
  • feature
  • refactoring / style
  • build / chore
  • documentation

Did you add tests for your changes?

  • Not relevant

If relevant, did you update the documentation?

  • Yes, I've updated the documentation

Summary

Hello! This PR simply moves the default props from the hook to the defaultProps property on the Dropzone component, so that the defaults can be documented on the documentation site.

Current

current

With default props

with-pr

Does this PR introduce a breaking change?

No!
Other information

@coveralls
Copy link

coveralls commented Oct 5, 2020

Pull Request Test Coverage Report for Build 5fc4ffcf4ca71db905778fc559c3a38e0c08b839-PR-1015

  • 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.01%) to 99.449%

Totals Coverage Status
Change from base Build aaa9710334a5f28e221cef84c11cf4a013c8c37c: -0.01%
Covered Lines: 201
Relevant Lines: 201

💛 - Coveralls

@samuelfullerthomas
Copy link
Contributor Author

Any thoughts on this @rolandjitsu ?

src/index.js Outdated Show resolved Hide resolved
samuelfullerthomas and others added 2 commits October 14, 2020 10:24
Co-authored-by: Roland Groza <rolandjitsu@users.noreply.github.com>
@rolandjitsu rolandjitsu merged commit 1b1177d into react-dropzone:master Oct 14, 2020
@samuelfullerthomas samuelfullerthomas deleted the move-default-props branch October 14, 2020 13:58
@github-actions
Copy link

🎉 This PR is included in version 11.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants