Skip to content

bug: BYNT-1331 - Web Import downloads low quality#62

Merged
tarekio merged 4 commits intomainfrom
bynt-1331-low-quality-web-import
Jun 12, 2025
Merged

bug: BYNT-1331 - Web Import downloads low quality#62
tarekio merged 4 commits intomainfrom
bynt-1331-low-quality-web-import

Conversation

@cango91
Copy link
Contributor

@cango91 cango91 commented Jan 17, 2025

Jira Issue

  1. BYNT-1331

Description

With the existing format selector, we're asking for the highest available merged video-audio, which nowadays is usually at 360p. By requesting highest quality separate streams and merging them we can download highest available quality.

Checklist

  • Tests added/updated
  • Documentation updated (if needed)
  • New strings prepared for translations

API Changes (if applicable)

  • Permissions checked
  • Endpoint tests added

Additional Notes

[Any other relevant information]

@cango91 cango91 added the bug Something isn't working label Jan 17, 2025
@cango91 cango91 self-assigned this Jan 17, 2025
@cango91 cango91 requested a review from tarekio January 17, 2025 08:00
@tarekio
Copy link
Contributor

tarekio commented May 31, 2025

Let's remove all quality options and rely on yt-dlp as it does what we need by default: https://github.com/yt-dlp/yt-dlp?tab=readme-ov-file#format-selection

Copy link
Contributor

@tarekio tarekio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@tarekio tarekio requested a review from level09 June 10, 2025 19:08
Copy link
Collaborator

@level09 level09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to go 🚀

@tarekio tarekio merged commit 2ec3af3 into main Jun 12, 2025
7 checks passed
@tarekio tarekio deleted the bynt-1331-low-quality-web-import branch June 12, 2025 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants