Skip to content

[2.x] Allow file uploads through file fields - #17

Merged
royduin merged 7 commits into
2.xfrom
feature/upload-files-2x
Sep 29, 2026
Merged

royduin merged 7 commits into
2.xfrom
feature/upload-files-2x

Conversation

@Jade-GG

@Jade-GG Jade-GG commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

ref: BORDEX-1459 & BORDEX-1448

This seemed relatively simple at first, but then ended up being a somewhat complicated problem. When you request a quote of a product that has a file field, it won't add the file to the quote data.

I've had to consider 2 different possibilities:

  • You can request a quote from your cart. This means the file is uploaded to the Magento server, and thus can be easily retrieved from there. (Note that this currently requires Magento to be on the same container/server as Rapidez)
  • You can request a quote directly from a product page. This means the file isn't yet uploaded anywhere, and as such we save it to an IndexedDB keyval store.

After this, the file gets uploaded to Statamic by using the container's flysystem functionality, making sure to restrict file size and file extensions.

@Jade-GG
Jade-GG requested review from indykoning and royduin August 21, 2026 12:15
@Jade-GG

Jade-GG commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator Author

Note that I will add a 5.x version once this one has been merged. I started on this version because that's the version that the customer that requested this feature is on.

@royduin

royduin commented Sep 8, 2026

Copy link
Copy Markdown
Member

Follow-up: BORDEX-1478

@Jade-GG

Jade-GG commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator Author

@indykoning @royduin is this PR on your radar still? 🙂

@Jade-GG
Jade-GG requested a review from BobWez98 September 22, 2026 11:35
@royduin
royduin merged commit 54b0ca8 into 2.x Sep 29, 2026
1 check passed
@royduin
royduin deleted the feature/upload-files-2x branch September 29, 2026 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants