-
Notifications
You must be signed in to change notification settings - Fork 3
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
Too many limitions by react-docgen #1
Comments
Hi, My first approach was to load the components (using require) but react proptypes are functions and we can't extrapolate information in runtime, so I fell back to using react docgen. I have now dedicated an hour to check if I can somehow return to my first approach. I will be overriding the react proptypes with a runtime readable version so any component can be loaded and the information can be extracted. I will comment in here again when I have a first working version. |
Hi, I tried using react-bootstrap. The parsing part is now working, but it seems that most components have "oneOf" properties, which are still unsupported by this project. |
I use your reactReflectiveProptypes.js and I must say it's amazing, almost exactly what I needed. A few remark though, required is always set to true, even if isRequired is not specified in propTypes. Also any reason to make an isRequired object with an empty function and have type and required inside ? Also line 9 and 30 are never used. Thanks awesome work :) |
Hi @moimael , thanks for the comment. I am going to check the required issue. |
Hi, I've also checked the lines 9 and 30, I'll be removing them. Thank you for the feedback. If you have other questions or would like any new feature please tell me |
Too many limitions by react-docgen ,I tried many components ,such as http://react-bootstrap.github.io/, it throws error.May have better way to parse react components?
The text was updated successfully, but these errors were encountered: