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

Adds full support for socialspy in EssentialsDiscord #5620

Open
wants to merge 5 commits into
base: 2.x
Choose a base branch
from

Conversation

mrbubbles06
Copy link

@mrbubbles06 mrbubbles06 commented Jan 23, 2024

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

  • Most recent Paper version (1.20.4, git-Paper-393)
  • CraftBukkit/Spigot/Paper 1.12.2
  • CraftBukkit 1.8.8

Demonstration:

image image

@mrbubbles06 mrbubbles06 marked this pull request as ready for review January 23, 2024 08:24
@nelind3
Copy link

nelind3 commented Feb 20, 2024

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 think the message formatting should mirror the socialspy formatting. That is instead of {sender-username} -> {receiver-username}: {message} i think [SS][{sender-displayname} -> {receiver-displayname}]: {message} or similar is more appropriate.
  • I think the message type name in the config should be something like socialspy or private-message in the same style as the other message types.

@Bobcat00
Copy link
Contributor

I think the message formatting should mirror the socialspy formatting. That is instead of {sender-username} -> {receiver-username}: {message} i think [SS][{sender-displayname} -> {receiver-displayname}]: {message} or similar is more appropriate.

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).

@nelind3
Copy link

nelind3 commented Feb 21, 2024

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.

@Bobcat00
Copy link
Contributor

I agree. Also, my PR does not change the default being display names.

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.

Allow EssentialsDiscord to "have socialspy"
4 participants