Viktor Petersson 8c6cfaf26a feat(server): offer HandBrake GUI steps for rejected video uploads (#3040) ...

* feat(server): offer HandBrake GUI steps for rejected video uploads

- Add _handbrake_steps mirroring the ffmpeg recipe's codec/1080p choices
- Persist the steps to metadata.error_handbrake on rejection
- Render them as a numbered list with a handbrake.fr link in the modal

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* fix(server): make HandBrake steps a real preset-based walkthrough

- Lead with the stock "Fast 1080p30" preset (H.264 MP4, 1080p cap)
- HEVC boards just flip Video Encoder to "H.265 (x265)"
- Spell out source pick, Save As/Browse, and Start Encode
- Drop the cap arg: the 1080p preset is the low-RAM fix too

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* test: assert on HANDBRAKE_URL constant, not the literal

Reference processing.HANDBRAKE_URL instead of a duplicated URL literal
so CodeQL stops flagging the assertion as incomplete-URL-substring
sanitization (false positive in a test containment check).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* docs: address Copilot review on HandBrake steps

- Reword final step: upload as a new asset (the Edit modal has no
  upload control)
- Rename stale-clear test to mention error_handbrake too

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

2026-06-09 08:55:57 +02:00

80 KiB

Raw Permalink Blame History

View Raw