react-jsonschema-form
react-jsonschema-form copied to clipboard
fix(@rjsf/core): Add formElement to Form TS definitions
Reasons for making this change
Adds the formElement definition to the Form TS interface
The Form object internally manages a reference called formElement, initially null that can be set via the ref property.
However, the TS definitions do not reflect this, therefore, consuming the library from a pure TS project will throw an error when attempting to access it.
I think this is the only change needed, but let me know if I'm missing anything
Checklist
- [ ] I'm updating documentation
- [ ] I've checked the rendering of the Markdown text I've added
- [x] I'm adding or updating code
- [ ] I'm adding a new feature
- [ ] I've updated the playground with an example use of the feature
@rtalvarez So, in the upcoming v5 beta, Form has been converted to Typescript and it's types are properly being exported. Take a look at the code in the rjsf-v5 branch and let me know whether this seems appropriate for your needs...
@rtalvarez So, in the upcoming v5 beta,
Formhas been converted to Typescript and it's types are properly being exported. Take a look at the code in therjsf-v5branch and let me know whether this seems appropriate for your needs...
Hey @heath-freenome , sorry for the late reply, I was out a couple of days. Unfortunate about that needing to be any, but other than that, yes that should address the issue. Do we know when v5 is expected to come out of beta?
@rtalvarez So, in the upcoming v5 beta,
Formhas been converted to Typescript and it's types are properly being exported. Take a look at the code in therjsf-v5branch and let me know whether this seems appropriate for your needs...Hey @heath-freenome , sorry for the late reply, I was out a couple of days. Unfortunate about that needing to be
any, but other than that, yes that should address the issue. Do we know when v5 is expected to come out of beta?
Hoping to have it out before the end of August...
@rtalvarez Can you verify that the v5 beta has the proper typing that you need?
@rtalvarez Can you verify that the v5 beta has the proper typing that you need?
Sure, I'll try it and get back to you
Since this file no longer exists I am closing the PR...