Senither / hypixel-discord-chat-bridge

A two-way chat bridge between Hypixel guild chat and Discord.
MIT License
29 stars 118 forks source link

Add broadcasts for most guild events #23

Closed itsneyoa closed 3 years ago

itsneyoa commented 3 years ago

Added broadcasts for all the following guild events:

omg leaking guild chat D: oh noe more top secret leaks

Thanks to PR #22 for reminding me I'd started this :)

SaegusaMayumi1234 commented 3 years ago

well should I close my pr? I think yours is better so #22 It has different purpose but idk

itsneyoa commented 3 years ago

It has different purpose but idk

As far as I can see the main differences are:

As the Discord commands will only work in the command specified in the configuration, in my opinion it makes more sense to log all events there - running the mute command only to have to switch channels to ensure it worked seems pointless. Additionally some events are broadcasted to all guild members anyway, however events like mutes aren't so it may be a good idea to put them in a different channel. @SergeantSar is working on officer chat support at the moment, so that may tie nicely into this.

Being togglable in the configuration could be easily added, however I'm not sure if it's really necessary.

The formatting of the message wasn't something I gave much thought about, I didn't see any benefit from adding the executing staff member to the messages so decided to leave it out, however again if we were to add the events to an officer chat channel it may be better to show the staff member in order to identify and prevent staff abuse.

Again, this is all what I thought made the most sense so if you disagree or have any other suggestions please leave them here :)

SaegusaMayumi1234 commented 3 years ago

Well I think so.

Anyways this is not really necessary but I just spotted missing error message like:

you don't need to add this if you think this is not necessary to add

itsneyoa commented 3 years ago

I've added all those errors too now, it seems since I added most of the event replies something's changed so the ---------------------------------------------------- before and after the actual message messes up .startsWith() and .endsWith(), that's also been fixed.