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

Disallow empty names? #3

Open
jD91mZM2 opened this issue Dec 21, 2017 · 3 comments
Open

Disallow empty names? #3

jD91mZM2 opened this issue Dec 21, 2017 · 3 comments

Comments

@jD91mZM2
Copy link
Contributor

jD91mZM2 commented Dec 21, 2017

This might seem like an obvious thing, but I wonder if we should disallow empty names?
Problem with these things is that by limiting something, we're bound to have somebody complain about it.

Also, if we limit names... Should we also limit names with only whitespace in them? Maybe even trim all leading/trailing whitespace? Support unicode? Read a setting from a config file?

Seems easy, quickly grows complicated. Should we allow disallow empty messages too?
FYI: Neither discord nor reddit prevent you from sending messages with only a zero width space. This hasn't really been a problem so far.

@jD91mZM2
Copy link
Contributor Author

Another idea: We allow users to set a custom regex. Won't be nearly as fast, but should be more powerful.

@Mnpn
Copy link

Mnpn commented Dec 21, 2017

Honestly IMO names should always be readable.

@jD91mZM2
Copy link
Contributor Author

What does that mean we should do though?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants