Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Add System to Nickname Blacklist #1437

Merged
merged 1 commit into from
Jul 12, 2018
Merged

Conversation

DSchalla
Copy link
Member

Summary

This PR is related to the Server PR linked below and introduces system as a blacklisted username.

Ticket Link

N/A -> mattermost/mattermost#9060

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

@jasonblais
Copy link
Contributor

jasonblais commented Jul 11, 2018

How about first/last name?

@DSchalla
Copy link
Member Author

Out of scope for this PR, it's certainly a valid concern @jasonblais. Currently all blacklists just apply to the username according to my understanding.

@jasonblais jasonblais added the 2: Dev Review Requires review by a core commiter label Jul 11, 2018
@jasonblais jasonblais requested review from crspeller, hmhealey and grundleborg and removed request for hmhealey July 11, 2018 22:11
@crspeller crspeller merged commit 19c0c7d into mattermost:master Jul 12, 2018
@lindalumitchell lindalumitchell added the Tests/Done Release tests have been written label Jul 12, 2018
@amyblais amyblais added Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation labels Jul 21, 2018
fincha pushed a commit to fincha/mattermost-webapp that referenced this pull request Oct 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2: Dev Review Requires review by a core commiter Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation Tests/Done Release tests have been written
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants