Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filter out unsupported watch page only mixes from search results #6244

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

absidue
Copy link
Member

@absidue absidue commented Dec 1, 2024

Filter out unsupported watch page only mixes from search results

Pull Request Type

  • Bugfix

Related issue

Description

Watch page only mixes are not supported in FreeTube as they only exist on the YouTube watch page, that means they don't have their own playlist pages. Those unsupported mix playlists used to be filtered out automatically as they had their own Mix node which we didn't parse, but now YouTube has switched them to the same LockupView with content_type = 'PLAYLIST' that they use for normal playlists.

Some mixes such as the ones on https://youtube.com/music have actual playlist pages, they can be identified by the fact that their playlist IDs start with RDCL, which this pull request explicitly checks for to make sure that those types don't get filtered out.

Testing

Search for music and check that the mix playlist doesn't show up in the local API search results.

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: bf06696

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Dec 1, 2024
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) December 1, 2024 10:56
@PikachuEXE
Copy link
Collaborator

What is mixes... (I never use Youtube unless leaving comments which is rare)

@FreeTubeBot FreeTubeBot merged commit c448b3c into FreeTubeApp:development Dec 5, 2024
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Dec 5, 2024
@absidue absidue deleted the search-mixes branch December 5, 2024 23:02
jlvivero pushed a commit to jlvivero/FreeTube that referenced this pull request Dec 7, 2024
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.

6 participants