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

Add option to include reported statuses in warning e-mail #11639

Merged
merged 1 commit into from
Aug 23, 2019

Conversation

Gargron
Copy link
Member

@Gargron Gargron commented Aug 21, 2019

image

The option only appears when acting on a report (and can be disabled, just like the e-mail itself can be omitted). When enabled, it takes the statuses from the report. I've investigated adding a UI for selecting specific statuses instead of taking the ones from the report, but it makes the UI very clunky and more confusing, and it seems to me like this is enough.

@Gargron Gargron added the moderation Administration and moderation tooling label Aug 21, 2019
@Gargron Gargron changed the title Add option to included reported statuses in warning e-mail Add option to include reported statuses in warning e-mail Aug 21, 2019
@Gargron Gargron force-pushed the feature-reported-statuses-in-warning branch 3 times, most recently from 8cd4610 to b3b7320 Compare August 22, 2019 02:21
@Gargron Gargron marked this pull request as ready for review August 22, 2019 02:21
@Gargron Gargron force-pushed the feature-reported-statuses-in-warning branch from b3b7320 to 7a8c38a Compare August 22, 2019 02:22
@Gargron Gargron merged commit 73ca0bb into master Aug 23, 2019
@ykzts ykzts deleted the feature-reported-statuses-in-warning branch September 5, 2019 03:05
hiyuki2578 pushed a commit to ProjectMyosotis/mastodon that referenced this pull request Oct 2, 2019
rtucker referenced this pull request in vulpineclub/mastodon Jan 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
moderation Administration and moderation tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants