Skip to content

fix(scorm): upload package zip as private file#2412

Closed
raizasafeel wants to merge 1 commit into
frappe:developfrom
raizasafeel:file-uploader
Closed

fix(scorm): upload package zip as private file#2412
raizasafeel wants to merge 1 commit into
frappe:developfrom
raizasafeel:file-uploader

Conversation

@raizasafeel
Copy link
Copy Markdown
Contributor

Summary

  • SCORM package zips were uploaded as public files, so the raw archive URL was accessible to anyone with the link
  • Now SCORM zips upload with private: true. Other uploads (cover images, settings images, etc) explicitly stay public

@cypress
Copy link
Copy Markdown

cypress Bot commented May 19, 2026

lms    Run #3685

Run Properties:  status check passed Passed #3685  •  git commit 201947ac1b ℹ️: Merge b1dac4033d3ae9f0fe2d6127f01e6ed27cb17838 into 4860cf340b567bb1e1bf592ff795...
Project lms
Branch Review file-uploader
Run status status check passed Passed #3685
Run duration 00m 33s
Commit git commit 201947ac1b ℹ️: Merge b1dac4033d3ae9f0fe2d6127f01e6ed27cb17838 into 4860cf340b567bb1e1bf592ff795...
Committer Raizaaa
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 1
View all changes introduced in this branch ↗︎

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.76%. Comparing base (41fb007) to head (b1dac40).
⚠️ Report is 9 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2412      +/-   ##
===========================================
+ Coverage    47.74%   47.76%   +0.02%     
===========================================
  Files          150      150              
  Lines         6658     6659       +1     
===========================================
+ Hits          3179     3181       +2     
+ Misses        3479     3478       -1     

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

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.

2 participants