-
Notifications
You must be signed in to change notification settings - Fork 9
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
ERM-3183 Create agreement line form does not clear on "Save and creat… #1331
base: master
Are you sure you want to change the base?
Conversation
…e another" * add minimal delay between pushing the view page and opening the create page again
License CLA Stuck? (Developer should make sure that it is really stuck before clicking) |
Jest Unit Test Statistics 1 files ±0 154 suites ±0 8m 7s ⏱️ ±0s For more details on these failures, see this check. Results for commit a11d3f4. ± Comparison against base commit 7cab206. ♻️ This comment has been updated with latest results. |
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.
I think I approve this as a fix, but I reckon we should probably look into another way to ensure the form reinitialises...
Depends on if this needs backporting or not as to whether we merge here or close and reexamine
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.
Can we change this to use the form.reset mechanism we do elsewhere please?
From Create we need to reset the form, from edit we should redirect to "create" (which should clear the form for us)
…e another" * try resetting createAnother checkbox
|
…e another"