-
Notifications
You must be signed in to change notification settings - Fork 373
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 redux variant to standalone variant migration guide #1653
Add redux variant to standalone variant migration guide #1653
Conversation
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.
@TheZoker We can also provide the previously available jsonformsReducers
via the redux part of the react
package. This eases the migration a lot. Can you please add this to this PR?
MIGRATION.md
Outdated
|
||
The standalone JSON Forms variant is the new default and the main focus for new features and bug fixes. | ||
We definitely recommend migrating to this version as soon as possible. | ||
All current redux functionally can also be achieved with the standalone version. |
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.
All current redux functionally can also be achieved with the standalone version. | |
All current Redux functionally can also be achieved with the standalone version. |
f80bfa4
to
b8370c7
Compare
Co-authored-by: Stefan Dirix <sdirix@eclipsesource.com>
321b2dd
to
e27305c
Compare
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.
LGTM
189a816
to
7c1326b
Compare
Since redux will be moved from a peerDependency to a optionalPeerDependency and some things need to be changed in order to keep an JSON Form app working, we add a new section to the MIGRATION.md file.
Related issue: #1537