Actually hide NSFW content #1748

Merged
neb-b merged 8 commits from hide-nsfw into master 2018-07-13 20:31:15 +02:00
neb-b commented 2018-07-10 18:14:48 +02:00 (Migrated from github.com)

Dependent upon https://github.com/lbryio/lbry-redux/pull/53

Currently causes issues with the community top bids section. That will be fixed in a future PR.

Dependent upon https://github.com/lbryio/lbry-redux/pull/53 Currently causes issues with the community top bids section. That will be fixed in a future PR.
skhameneh (Migrated from github.com) approved these changes 2018-07-11 19:02:55 +02:00
tzarebczan commented 2018-07-12 19:22:28 +02:00 (Migrated from github.com)

This LGTM on the channel and search pages. Gave @seanyesmunt a heads up about subs page - if you are subbed to a channel that has NSFW (and have NSFW off), there's no indication on the sub page that they are hidden.

This LGTM on the channel and search pages. Gave @seanyesmunt a heads up about subs page - if you are subbed to a channel that has NSFW (and have NSFW off), there's no indication on the sub page that they are hidden.
tzarebczan commented 2018-07-12 19:42:15 +02:00 (Migrated from github.com)

Actually, found an edge case when my own content is marked as NSFW and going to the channel page - says result is hidden, but it's not. We should not hide it if it's our own content, right? So just need to suppress the message.
https://www.screencast.com/t/vmUvZLTkuPv

Actually, found an edge case when my own content is marked as NSFW and going to the channel page - says result is hidden, but it's not. We should not hide it if it's our own content, right? So just need to suppress the message. https://www.screencast.com/t/vmUvZLTkuPv
Sign in to join this conversation.
No reviewers
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#1748
No description provided.