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

Allow other plugins to hook into VentureChat #92

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PhoenixOrigin
Copy link

I'm making a plugin and as far as I could see, the only chat message sending possiblity was for DiscordSRV so I just made an extra simple new function for other plugins. Please tell me if I need to change anything / there is a reason this doesn't already exist

@Aust1n46
Copy link
Owner

What is the purpose of this PR?

Not trying to sound like a jerk, but this new method isn't going to do anything when it's invoked. There's no listener for that type of message packet.

What exactly are you trying to do? Send messages over VentureChat's plugin messaging channel? Or just send a message to listeners of a particular channel? Because you can do the latter by iterating over the online players and checking if they're listening to a channel.

@PhoenixOrigin
Copy link
Author

Give other plugins the ability to act like DiscordSRV (send a message to all online players from discord)

@Aust1n46
Copy link
Owner

Aust1n46 commented Dec 4, 2023

The DiscordSRV hook only works because I created it and added routing support on both the proxy side of the plugin and the server sided plugin message listener.

If you want a generic way to send pub/sub messages using plugin message channels, you can implement that in your own plugin and hook into VentureChat's API to access channel data.

Or if you're doing a one off implementation just for yourself, you could fork this repo and implement the listeners inside of VentureChat for your plugin. But that's not something I'd accept as a PR contribution.

Sorry for the late reply, I don't always get around to checking this alongside the Spigot forums that often.

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.

None yet

2 participants