Skip to content

Conversation

@silverkszlo
Copy link
Collaborator

📝 Summary

This PR is related to this PR in Viewer and in combination they resolve this issue in customer-feature-requests.

Before:

When editing a markdown document online in files and download it from the editor/viewer without saving it manually, the last saved version is downloaded. The last saved version may not include what was just added/edited after the previous save operation. So the downloaded file may differ from the file one sees online.

Now:

Clicking on Download now triggers an automatic save operation of the document. If the saving for some reason fails, the last saved version is still being downloaded.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 45.96%. Comparing base (2c561dd) to head (3a1cd92).
⚠️ Report is 74 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7889   +/-   ##
=======================================
  Coverage   45.96%   45.96%           
=======================================
  Files         141      141           
  Lines        3964     3964           
  Branches      747      747           
=======================================
  Hits         1822     1822           
  Misses       1838     1838           
  Partials      304      304           

☔ 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.

@silverkszlo silverkszlo force-pushed the downloadcallback-from-viewer branch from 841e594 to 3a1cd92 Compare November 5, 2025 10:01
@juliusknorr juliusknorr merged commit 00e5cfe into main Nov 13, 2025
76 of 78 checks passed
@juliusknorr juliusknorr deleted the downloadcallback-from-viewer branch November 13, 2025 08:14
@silverkszlo
Copy link
Collaborator Author

/backport to stable32 please

@silverkszlo
Copy link
Collaborator Author

/backport to stable31 please

@silverkszlo
Copy link
Collaborator Author

/backport to stable30 please

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants