Skip to content

Conversation

adamantike
Copy link
Contributor

@adamantike adamantike commented Aug 12, 2025

Description
Use the right filesystem handler for the manual to be uploaded to the resources path instead of the rom path.

Fixes #2253.

Checklist

  • I've tested the changes locally
  • I've updated relevant comments
  • I've assigned reviewers for this PR
  • I've added unit tests that cover the changes

Use the right filesystem handler for the manual to be uploaded to the
`resources` path instead of the `rom` path.
@adamantike adamantike requested review from gantoine and zurdi15 August 12, 2025 19:19
Copy link

trunk-io bot commented Aug 12, 2025

Running Code Quality on PRs by uploading data to Trunk will soon be removed. You can still run checks on your PRs using trunk-action - see the migration guide for more information.

Copy link

Test Results

548 tests  ±0   547 ✅ ±0   57s ⏱️ -1s
  1 suites ±0     1 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 1d2403d. ± Comparison against base commit 6aca53f.

Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
8755 6175 71% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
backend/endpoints/rom.py 39% 🟢
TOTAL 39% 🟢

updated for commit: 1d2403d by action🐍

@adamantike adamantike merged commit 83b8937 into master Aug 12, 2025
9 checks passed
@adamantike adamantike deleted the fix/upload-manual-path branch August 12, 2025 19:27
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.

[Bug] Manual upload of .pdf manual file does not work
2 participants