-
Notifications
You must be signed in to change notification settings - Fork 172
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
Add JSON Form Widget readme #4344
Conversation
… fix some language
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.
A couple suggestions. Also wonder if we should more explicitly state that we have as a goal to make this module work independently of DKAN.
…paragraph to reference RJSF
Ty both! We'll chat a bit about where to add the diagrams and if/how to add the decoupling goal on Fridays huddle. Then one more review after and this should be good. |
45c932c
to
492fb18
Compare
492fb18
to
84a34fa
Compare
…nto their own file for now
Co-authored-by: Dan Feder <dan.feder@civicactions.com>
Describe your changes
This adds a readme file to the JSON Form Widget module and updates related documentation accordingly.
QA Steps
Checklist before requesting review
If any of these are left unchecked, please provide an explanation