-
-
Notifications
You must be signed in to change notification settings - Fork 980
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
Adds full support for socialspy in EssentialsDiscord #5620
base: 2.x
Are you sure you want to change the base?
Conversation
Looks great (I already ended up making a custom extension to do this because it was so needed for our moderation ability. It would be amazing to have it as a built in). Personally though I'd prefer it if the naming and formatting was more consistent with the rest of EssentialsX and EssentialsXDiscord namely:
|
I have a PR to make the Social Spy messages configurable to use either the player name or the display name (because player names are so much better for SS messages). |
If that does go through (which it really should display names versus player names are very much a thing that should be configurable) i think the socialspy formatting here should still mirror the in game socialspy formatting and it's defaults. If the default changes to socialspy using player names then this should default to player names too in my opinion. It's essentially the same message they should have the same default formatting. |
I agree. Also, my PR does not change the default being display names. |
Information
This PR closes #5586.
Details
Proposed feature:
Adds support for socialspy (viewing private messages) in EssentialsDiscord.
Environments tested:
OS: MacOS 14.1
Java version: Temurin-17.0.10+7
Demonstration: