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

Update some views to opt out of "Hide Videos From Channels" #3748

Conversation

PikachuEXE
Copy link
Collaborator

Pull Request Type

  • Bugfix
  • Feature Implementation
  • Documentation
  • Other

Related issue

Addresses #3744, for a feature introduced in #2849

Description

Update following views to opt out of "Hide Videos From Channels", e.g. ignore channels to hide preference

  • Channel
    Only reachable when user wants to view it explicitly, no point viewing the channel with all content hidden
  • History
    Avoid unexpected false sense of absence when searching
  • Subscription
    User subscribe for channels explicitly
  • User playlist
    User add videos into user playlist(s) explicitly

Screenshots

Lazy

Testing

Mentioned views should still show content for channels added to "Hide Videos From Channels"
Other views (search, trending, popular, etc.) should still have stuff from channels in "Hide Videos From Channels" hidden

Desktop

  • OS:
  • OS Version:
  • FreeTube version:

Additional context

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) July 10, 2023 06:13
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jul 10, 2023
Copy link
Member

Choose a reason for hiding this comment

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

LGTM!

@FreeTubeBot FreeTubeBot merged commit 35868b4 into FreeTubeApp:development Jul 13, 2023
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Jul 13, 2023
@PikachuEXE PikachuEXE deleted the fix/hide-channels-but-not-in-some-views branch July 13, 2023 00:50
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.

5 participants