Skip to content

Recommendations policy#7633

Open
immae wants to merge 2 commits into
Chocobozzz:developfrom
immae:recommendations
Open

Recommendations policy#7633
immae wants to merge 2 commits into
Chocobozzz:developfrom
immae:recommendations

Conversation

@immae

@immae immae commented May 28, 2026

Copy link
Copy Markdown
Contributor

Description

This commit adds a new option attached to a video, that allows to chose which kind of recommendation will appear below it in the viewer. It currently allows four setups:

  • Any video (same as currently)
  • Local videos only
  • Owner videos only
  • Same channel videos only

This allows the user who owns the video to restrict the recommendations associated to his videos.

AI disclaimer: Tests were written in big part with AI help. It also sketched me a draft of the feature implementation but no line of this draft remain in this PR, it helped mostly with providing indications and relevant code locations since I’m not familiar with the project.

Related issues

#1600

Has this been tested?

  • 👍 yes, I added tests to the test suite
  • 💭 no, because this PR is a draft and still needs work
  • 🙅 no, because this PR does not update server code
  • 🙋 no, because I need help

Screenshots

2026-05-28_15_52_42

@immae immae changed the title Recommendations Recommendations policy May 28, 2026
@immae immae force-pushed the recommendations branch from 3c511a7 to 8570ff0 Compare May 28, 2026 14:43
immae added 2 commits May 29, 2026 10:48
This commit adds a new option attached to a video, that allows to chose
which kind of recommendation will appear below it in the viewer.
It currently allows four setups:
- Any video (same as currently)
- Local videos only
- Owner videos only
- Same channel videos only

This allows the user who owns the video to restrict the recommendations
associated to his videos.
Tests for the recommendation feature
@immae immae force-pushed the recommendations branch from 8570ff0 to deb970c Compare May 29, 2026 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants