search

open-formulieren / open-forms

Smart and dynamic forms
https://open-forms.readthedocs.io
Other
37 stars 26 forks source link

Registration variable attachments/uploaded_attachment_urls missing in v2 #4745

Closed LaurensBurger closed 11 hours ago

LaurensBurger commented 1 month ago

Product versie / Product version

2.7.x - 2.8.0

Customer reference

Den Haag

Omschrijf het probleem / Describe the bug

It seems impossible to map the "uploaded_attachment_urls" since there is not entry for these in the variables > registration tab. So currently the submissions are missing the array "attachments" when using the v2 mapping.

(missed in #4475)

sergei-maertens commented 1 month ago

AFAIK this is deliberate since you're supposed to map the individual file components rather than throw all attachments onto a big pile and losing all the context.

joeribekker commented 1 month ago

Agree with Sergei, so, they should really find a businesscase for this. Ie. not a bug.

sergei-maertens commented 3 weeks ago

After plenty of internal discussion, we still feel that we shouldn't pick this up, but will do so anyway because the alternatives are even worse in the current situation.

We'll provide a separate registration variable that will contain all the upload URLs. Hopefully we can find a way to label this as "not recommended" and can document the intended usage, but that will also require some rethinking on the "productaanvraag/verzoek" side of things.

sergei-maertens commented 3 weeks ago

I will consider this for backport IF it's easy to do so, since it really is an enhancement.