Skip to content

Conversation

@tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Dec 18, 2025

What is this pull request for?

The archive overlay is handled by a turbo-frame. We need to reload that frame instead of using the old jQuery based weird script loading.

When passing only from the ingredient that opens the uploader, we scope the files to that accepted formats only.

Fix the Attachment replace button so it respects the allowed_filetypes configuration.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

The archive overlay is handled by a turbo-frame.
We need to reload that frame instead of using the
old jQuery based weird script loading.

Signed-off-by: Thomas von Deyen <[email protected]>
When passing `only` from the ingredient that opens
the uploader, we scope the files to that accepted
formats only.

Signed-off-by: Thomas von Deyen <[email protected]>
@tvdeyen tvdeyen added the bug label Dec 18, 2025
@tvdeyen tvdeyen requested a review from a team as a code owner December 18, 2025 17:27
@tvdeyen tvdeyen added the backport-to-8.0-stable Needs to be backported to 8.0-stable label Dec 18, 2025
@codecov
Copy link

codecov bot commented Dec 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.20%. Comparing base (50bba1c) to head (d6b9c32).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3526   +/-   ##
=======================================
  Coverage   97.20%   97.20%           
=======================================
  Files         287      287           
  Lines        7586     7586           
=======================================
  Hits         7374     7374           
  Misses        212      212           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tvdeyen tvdeyen merged commit e47d0e0 into main Dec 19, 2025
29 of 30 checks passed
@tvdeyen tvdeyen deleted the fix-overlay-uploader branch December 19, 2025 15:31
@alchemycms-bot
Copy link

💔 All backports failed

Status Branch Result
8.0-stable Backport failed because of merge conflicts

You might need to backport the following PRs to 8.0-stable:
- Add upload button test for allowed file types
- Fix CollectionOption comparison in uploader button template

Manual backport

To create the backport manually run:

backport --pr 3526

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@alchemycms-ci-bot
Copy link
Collaborator

💚 All backports created successfully

Status Branch Result
8.0-stable

Questions ?

Please refer to the Backport tool documentation

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

Labels

backport-to-8.0-stable Needs to be backported to 8.0-stable bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants