@uppy/form crashes browser on v3.1.0 and higher

[joostdebruijn]

@uppy/form crashes browser on v3.1.0 and higher

[joostdebruijn]

The Firefox-profiler is working better, I discovered. Even when the tab is crashing, you can view the recording. It shows similar statistics as Chrome:

[Murderlon]

Hi, you've enabled both submitOnSuccess and triggerUploadOnSubmit, which doesn't make sense I think because one triggers the other. If I remove one of them, everything works as expected.

Is there a use case you have for both? Otherwise I'll make a PR to throw when both options are set.

[joostdebruijn]

@Murderlon Thanks for the quick response. I've both options set because I would like to trigger the upload on submit and I want the form to be submitted once the upload has been completed. I think I misunderstand the submitOnSuccess, because the form will be actually submitted after the upload has been complete after the upload is being triggered by the form submission.

[joostdebruijn]

@Murderlon Hm, I've submitOnSuccess disabled and triggerUploadOnSubmit enabled now. The files are uploaded once I submit the form, however - the form itself is not submitted after the upload completes...

[Murderlon]

You're right I overlooked something. In that case we just need a check whether the upload already happened.