-
Notifications
You must be signed in to change notification settings - Fork 2
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
Variable edition state optimization #164
Labels
enhancement
New feature or request
Comments
This was referenced Feb 10, 2023
We may want to use React Hook Form for this one. |
Unintentionally closed |
sergkudinov
pushed a commit
to sergkudinov/tdp-ui
that referenced
this issue
Apr 13, 2023
Add react-hook-form to optimize state. Sync code editor in read mode (import values from view). Persit user settings accross pages (raw mode, show unused tabs). Fixes TOSIT-IO#96 Fixes TOSIT-IO#164 TODO: sync code editor in write mode.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The user input of the form are stored in Redux. It would be better to store them in a local context (as it was in the first place, but keeping the reducers introduced by Redux), and then send it to Redux once validated.
Also, we must ensure that the variables that have been modified and then the modification canceled, are not send to the server. (as described in #157 (comment)).
The text was updated successfully, but these errors were encountered: