Use filtered endpoint to suppress trending/top/new views #2613

Closed
opened 2019-07-08 19:55:42 +02:00 by tzarebczan · 4 comments
tzarebczan commented 2019-07-08 19:55:42 +02:00 (Migrated from github.com)

The Issue

Spoke to @seanyesmunt about this - currently we have a way to hide inappropriate content on the trending page, but this changes with the new version since it's not longer produced by the API. The API team added a new endpoint, similar to the blocked_list, to help us do this: https://api.lbry.com/file/list_filtered

Most common use case will be improperly tagged content, non marked NSFW, or content that's inappropriate but not illegal.

This should hide content from any of the trending/top/new views. It should not block the URL from being accessed directly. We may want to block from search as well.

I think this will be removed from the desktop app eventually, but we'll need to continue doing it on lbry.tv.

System Configuration

  • LBRY Daemon version:
  • LBRY App version:
  • LBRY Installation ID:
  • Operating system:

Anything Else

Screenshots

<!-- Thanks for reporting an issue to LBRY and helping us improve! To make it possible for us to help you, please fill out below information carefully. Before reporting any issues, please make sure that you're using the latest version. - App releases: https://github.com/lbryio/lbry-desktop/releases - Standalone daemon: https://github.com/lbryio/lbry/releases We are also available on live chat at https://chat.lbry.com --> ## The Issue Spoke to @seanyesmunt about this - currently we have a way to hide inappropriate content on the trending page, but this changes with the new version since it's not longer produced by the API. The API team added a new endpoint, similar to the blocked_list, to help us do this: https://api.lbry.com/file/list_filtered Most common use case will be improperly tagged content, non marked NSFW, or content that's inappropriate but not illegal. This should hide content from any of the trending/top/new views. It should not block the URL from being accessed directly. We may want to block from search as well. I think this will be removed from the desktop app eventually, but we'll need to continue doing it on lbry.tv. ## System Configuration <!-- For the app, this info is in the About section at the bottom of the Help page. You can include a screenshot instead of typing it out --> <!-- For the daemon, run: curl 'http://localhost:5279' --data '{"method":"version"}' and include the full output --> - LBRY Daemon version: - LBRY App version: - LBRY Installation ID: - Operating system: ## Anything Else <!-- Include anything else that does not fit into the above sections --> ## Screenshots <!-- If a screenshot would help explain the bug, please include one or two here -->
neb-b commented 2019-07-08 20:08:45 +02:00 (Migrated from github.com)

Should this only happen on lbry.tv?

Should this only happen on lbry.tv?
tzarebczan commented 2019-07-08 20:09:46 +02:00 (Migrated from github.com)

If someone posts unmarked porn on with a 500K LBC bid, you want all app users to see that?

If someone posts unmarked porn on with a 500K LBC bid, you want all app users to see that?
tzarebczan commented 2019-07-08 20:11:41 +02:00 (Migrated from github.com)

So I think it needs to stay in the app until we have a solution for community tagging / blocking specific claims (even then, just seeing it will be a big deal if the viewers are underage/etc).

So I think it needs to stay in the app until we have a solution for community tagging / blocking specific claims (even then, just seeing it will be a big deal if the viewers are underage/etc).
tzarebczan commented 2019-07-18 19:08:59 +02:00 (Migrated from github.com)

Thanks @jessopb !!! Completed in https://github.com/lbryio/lbry-desktop/pull/2621

Thanks @jessopb !!! Completed in https://github.com/lbryio/lbry-desktop/pull/2621
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#2613
No description provided.