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

feat: Display OBJKTs that were flagged by the Teia content moderation team as NSFW or photosensitive. #378

Merged
merged 6 commits into from
Feb 25, 2024

Conversation

Zir0h
Copy link
Contributor

@Zir0h Zir0h commented Feb 2, 2024

Very ugly fix for #373

Copy link

github-actions bot commented Feb 2, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-02-25 19:17 UTC

@melMass
Copy link
Member

melMass commented Feb 2, 2024

This is d59703f
image

@melMass
Copy link
Member

melMass commented Feb 2, 2024

only show it for yourself in cbf47b8

moderation corner (added in c00872e)
image

@melMass
Copy link
Member

melMass commented Feb 2, 2024

If we remove the BG style:
image

@Zir0h
Copy link
Contributor Author

Zir0h commented Feb 2, 2024

Nice! Looks way cleaner without the background I think!
Thanks for the help!

@Zir0h
Copy link
Contributor Author

Zir0h commented Feb 3, 2024

Maybe we also need to check when the objkt was minted, because the nsfw/photosensitive attributes didn't exist at the start

@melMass
Copy link
Member

melMass commented Feb 3, 2024

What do you mean? I mean how is this PR affecting that? I think we normalize it all and old mints can still be moderated

@Zir0h
Copy link
Contributor Author

Zir0h commented Feb 3, 2024

If I understood #373 correctly, it's meant nudge repeat offenders of untagged nsfw/photo-sensitivity in the right direction.
It feels a bit unfair if you minted work really early and now you're seeing "you forgot to tag this, so we had to do it for you" all over your profile 😅

@melMass
Copy link
Member

melMass commented Feb 4, 2024

I see what you mean now!
Not sure we should complexify it just for that tbh, but I'm open if it's a real issue

@Zir0h
Copy link
Contributor Author

Zir0h commented Feb 4, 2024

I see what you mean now! Not sure we should complexify it just for that tbh, but I'm open if it's a real issue

Indeed, keep it simple 😄

@melMass
Copy link
Member

melMass commented Feb 4, 2024

I think now that's it's a simple corner with text it's ok, with the stripped red borders maybe not 🤣

@melMass melMass merged commit 1dbff3e into main Feb 25, 2024
3 checks passed
@melMass melMass deleted the feat-display-moderated-objkt-status branch February 25, 2024 19:16
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.

2 participants