Skip to content
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

Submitting a FormBuilder form submits all FormBuilder forms #1662

Closed
emteknetnz opened this issue Jan 25, 2024 · 3 comments
Closed

Submitting a FormBuilder form submits all FormBuilder forms #1662

emteknetnz opened this issue Jan 25, 2024 · 3 comments

Comments

@emteknetnz
Copy link
Member

emteknetnz commented Jan 25, 2024

If there are multiple FormBuilder forms at once, submitting one will submit all of them

This bug was worked around on the following PRs:

We should:

  • Work out a way to prevent this from happening
  • Remove the workaround code in the two PRs above

Acceptance criteria

  • When submitting a nested form builder form, only the form attached to the action that triggered the submission gets submitted.
  • Hack working around this bug are removed.

Related

INSTALLER CI RUN

This CI run shows the PRs working nicely together.

PRs

@maxime-rainville
Copy link
Contributor

Just to clarify, is the problem only for nested form builders. Or if you have two form builders next to each other without one containing the other, will one form submit the other?

@emteknetnz
Copy link
Member Author

I'm not sure if the asset-admin form builder that was getting submitted in the first example was even technically nested, it just an a human user level it appears to be nested.

I don't know if it makes a different if they're nested or not, it felt very much like there was some sort of 'submit event' being broadcast that was the was being subscribed to

@sabina-talipova
Copy link
Contributor

PRs have been merged. Task is completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants