Script content reactive to form - #37
Conversation
Currently, if the user makes a change in the script content and then makes a change in the form, all the changes that the user made to the script content will be reset. This update does two things: 1. The script content is reactive to the form. That is, if you make a change to the script content, it will be updated in the form. 2. By doing this, the user does not need to reset their script content if they reset the form.
|
Thank you for the PR. I am currently away on a business trip, so it may take some time for me to make a decision. |
|
No problem. For a bit of context, this edit is based on user experience who wanted to be able to modify the script and change form options afterwards. Thanks |
|
Thank you again for the PR. I think this PR can be divided into two features:
Regarding feature 1, I ran into unexpected behavior with the following form.yml: each click on the checkbox duplicates the text. Regarding feature 2, I am reluctant to merge it. As you described, there are cases where the script fundamentally cannot be converted back into the widgets. I want Open Composer to behave exactly as users intend, and I do not want users to have to puzzle over why the reverse conversion sometimes does not work. For these reasons, I would like to keep this PR open for now and ask you to rework it to contain only feature 1, with the duplication issue above fixed. Since the PR is updated automatically when you push to your branch, there is no need to open a new one. Please note that we do not plan to make major changes to Open Composer. The two features you developed work correctly in most cases, so I believe they will be very useful at your site. If you would prefer to keep both features together, maintaining them in a fork on your side may be a better option — in that case, please let me know and I will close this PR. |
Currently, if the user makes a change in the script content and then makes a change in the form, all the changes that the user made to the script content will be reset.
This update does two things: