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

Dont blank out info for removed content #1813

Closed
Nutomic opened this issue Oct 5, 2021 · 5 comments
Closed

Dont blank out info for removed content #1813

Nutomic opened this issue Oct 5, 2021 · 5 comments
Labels
area: moderation type: discussion General talk without any clear action

Comments

@Nutomic
Copy link
Member

Nutomic commented Oct 5, 2021

I think that #1680 goes too far and hides too much information of removed content, making the mod log useless. For example, here is a removed post:

Screenshot_20211005_111430

This used to have the title, text and link still shown, which allowed users to see for themselves what was removed, and to decide themselves if it made sense. Its also the reason we are adding a separate purge option.

So I suggest that we blank out info only if the creator themselves deleted an item, and probably for removed comments as well (there, text is hidden from the website, so it makes sense to hide it from the api too). But removed posts and communities should still show all information.

@Nutomic Nutomic added enhancement New feature or request area: moderation labels Oct 5, 2021
@dessalines
Copy link
Member

dessalines commented Oct 6, 2021

At first it was like that, then someone requested that we blank out posts and comments instead of just set the deleted flag. So we should pry vote on this, bc I don't want to have to keep going back and forth. It doesn't matter to me either way.

#1679

if the creator themselves deleted an item

Only a creator can delete it. Mods can remove it, and in that situation we might want to blank things out.

@Nutomic Nutomic added type: discussion General talk without any clear action and removed enhancement New feature or request labels Oct 8, 2021
@Nutomic
Copy link
Member Author

Nutomic commented Oct 8, 2021

I think if the text is completely hidden like it is now, that makes the mod log almost useless. Sure you can see what was removed, but you cant judge for yourself if that seems correct. There are only few cases where a post/comment is so bad that it should be hidden completely (and thats what purge is for).

One option could be to show the text of removed posts/comments only to users who are logged in. Or only to admins and mods of that community (could be configurable).

@dessalines
Copy link
Member

The modlog will show the original post / comment text (it isn't deleted after all), but when you go to them in the post, they'll be blanked out. So the modlog is transparent for removes, but they won't show in the post.

@Nutomic
Copy link
Member Author

Nutomic commented Oct 8, 2021

Looks like it currently shows the post title or comment text. But not the post link/body, which are often more relevant. The layout could also be improved tbh.

@dessalines
Copy link
Member

I'd be fine with not blanking out post titles or bodies, only comment content, but I don't wanna have to go back and forth either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: moderation type: discussion General talk without any clear action
Projects
None yet
Development

No branches or pull requests

2 participants