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

"Links" ability to use colored OOC with NameColor flag #32379

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

AsnDen
Copy link
Contributor

@AsnDen AsnDen commented Sep 22, 2024

About the PR

Changes Admin flag check to NameColor flag check.

Why / Balance

Now:
If user has every single flag exept admin flag he won't get OOC color

Guys with NameColor flag can change their color but won't be able to see it (cause they don't have admin flag)

After:
If user can change his OOC, then he (and others) can see his colored OOC. If can't - he can't.

Technical details

Changes Admin flag check to NameColor flag check.

Requirements

Changelog

🆑
ADMIN:

  • tweak: Now only NameColor flag gives ability to use colored OOC

@AsnDen
Copy link
Contributor Author

AsnDen commented Sep 22, 2024

I've noticed that admins lost their standard red color. If it is crucial for them, I guess it will be possible to make check if user either has admin flag or NameColor flag

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

Successfully merging this pull request may close these issues.

1 participant