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

Tell players if any mod actions have been used on them. #624

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

Conversation

forkiesassds
Copy link
Contributor

Not sure if this requires any improvements.

Fixes #616.

@UnknownShadow200
Copy link
Collaborator

  1. Doesn't check for who not being null, so if you try to ban/freeze/mute an offline player, it'll error
  2. In my opinion, this doesn't really address the underlying issue of Warning messages are not visible in level only local chat #616
    It'd be better to change LogAction to something along the lines of PlayerOperations.MessageAction, which changes who gets messaged if the target player is in a localchat level (although the way that is implemented is also still somewhat wrong and needs to be redone)

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.

Warning messages are not visible in level only local chat
2 participants