Skip to content

Remove unwatch reason from the !nominate output#1354

Merged
Xithrius merged 1 commit into
masterfrom
remove-unnomiation-reason
Jan 17, 2021
Merged

Remove unwatch reason from the !nominate output#1354
Xithrius merged 1 commit into
masterfrom
remove-unnomiation-reason

Conversation

@Akarys42
Copy link
Copy Markdown
Contributor

As the watch reason can contain private information, we shouldn't share it with the whole staff. This commit removes the previous unwatch reason from the !nominate output.

As the watch reason can contain private information, we shouldn't share it with the whole staff.
@Akarys42 Akarys42 added a: moderation Related to community moderation functionality: (moderation, defcon, verification) p: 1 - high High Priority labels Jan 16, 2021
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.004%) to 57.355% when pulling dd0d14c on remove-unnomiation-reason into 8b1276a on master.

Copy link
Copy Markdown
Contributor

@Xithrius Xithrius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Xithrius Xithrius merged commit 844a1b6 into master Jan 17, 2021
@Xithrius Xithrius deleted the remove-unnomiation-reason branch January 17, 2021 00:28
@Xithrius Xithrius added the t: feature New feature or request label Jan 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: moderation Related to community moderation functionality: (moderation, defcon, verification) p: 1 - high High Priority t: feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants