search

Cadiducho / 40ServidoresMC

Plugin de Minecraft para enlazar tu servidor a la web de 40ServidoresMC
Apache License 2.0
5 stars 15 forks source link

Pullrequest #41

Closed apatuka closed 2 years ago

apatuka commented 2 years ago

El tópico principa de este PR es sobre eventos.

He creado un evento llamado Vote40Event que permitirá a los desarrolladores implementar en sus plugins disparadores que se ejecuten al ser llamado el Vote40Event.

Adicional a eso he encontrado algunos inconvenientes realizando pruebas, los cuales ya he corregido, todo podrá visualizarse dentro del PR.

Nestu14 commented 2 years ago

No se si Cadiducho tendrá intención de añadir el evento o no, pero más allá de esos un par de cosas.

Teniendo en cuenta que el propio plugin permite añadir comandos que se ejecuten cuando alguien realiza el voto, ¿qué beneficio tiene añadir el evento que has creado? El único motivo por el que habría beneficio de usar un evento es poder modificar las recompensas o cancelarlas junto al broadcast, algo que no permite lo que has hecho.

A parte de eso, al hacer una PR deberías centrarla en algo especifico y no hacer cambios innecesarios para eso como los arreglos que mencionas o el workflow que has añadido.