-
-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
Add ability to mark statuses as sensitive from reports in admin UI #17668
Conversation
82306f3
to
2b9ad43
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I would have preferred something more lightweight, without generating a strike or an email, but it works.
One change I would like, though, is allow that feature not only for toots with media attachments but also for toots with preview cards.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, although I think I'd prefer “Relevant posts:” rather than “Posts cited:” which sounds odd to me.
Also I would have preferred a more lightweight way that did not involve strikes, and maybe improved discoverability (previously, this was right in the statuses list page, now this is in the report page, and it's not clear beforehand that reporting gives the option to mark as sensitive), but that can come later.
Otherwise, this looks good.
sensitive: From now on, all your uploaded media files will be marked as sensitive and hidden behind a click-through warning. | ||
silence: You can still use your account but only people who are already following you will see your posts on this server, and you may be excluded from various discovery features. However, others may still manually follow you. | ||
suspend: You can no longer use your account, and your profile and other data are no longer accessible. You can still login to request a backup of your data until the data is fully removed in about 30 days, but we will retain some basic data to prevent you from evading the suspension. | ||
get_in_touch: If you believe this is an error, you can reply to this e-mail to get in touch with the staff of %{instance}. | ||
reason: 'Reason:' | ||
statuses: 'Posts that have been found in violation:' | ||
statuses: 'Posts cited:' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
“Relevant posts”?
…astodon#17668) * Add ability to mark statuses as sensitive from reports in admin UI * Allow mark as sensitive action on statuses with preview cards
Using status editing from #16697, make it work across federation.
Change "Posts that have been found in violation:" to "Posts cited:" to make notifications less ominous.