Skip to content

(WIP) Allow excluding non public only videos per source (member only)#768

Draft
phyzical wants to merge 7 commits intokieraneglin:masterfrom
phyzical:feature/allow-excluding-member-only-videos
Draft

(WIP) Allow excluding non public only videos per source (member only)#768
phyzical wants to merge 7 commits intokieraneglin:masterfrom
phyzical:feature/allow-excluding-member-only-videos

Conversation

@phyzical
Copy link
Copy Markdown

@phyzical phyzical commented Aug 11, 2025

closes #588
closes #648

What's new?

  • Adds logic to extract a videos "public" status via availaiblty = "public" this for the most part reflects member only videos.
    • will need to review this as it might also filter other videos some may want like unlisted (can pivot the naming if thats the case)
  • adds a new option per source member_content_behviour was replicated via the short_form_behaviour and live_behaviour logic for profile

Any other comments?

Still need to look into if this is good enough or it will remove videos some may want, (personally i wouldnt mind skipping unavailable videos too) so can rename this to like "public_videos" if we prefer

Also need to update the index so need to mind the syntax to alter at

  • I am the original author of this code and I am giving it freely to the community and Pinchflat project maintainers

i know you said you were pondering how to solve this. so figured i would start in a direction to get the ball rolling

image image image

bwittgen added a commit to bwittgen/pinchflat that referenced this pull request Mar 29, 2026
bwittgen added a commit to bwittgen/pinchflat that referenced this pull request Mar 29, 2026
… public field

PR kieraneglin#768 added members_content_behaviour to sources and a query filter
on media_items.public, but never added the public field to the
MediaItem schema or a migration for the column.

- Add 'public' boolean field to MediaItem schema (default: true)
- Add migration to add 'public' column to media_items table
- Fix format_matching_source_profile_preference to handle 'include' case

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

[FR] Ignore Members only Videos [FR] Disable member only content downloads

1 participant