DraftBot-A-Discord-Adventure / DraftBot

A Discord adventure bot that speaks english and french
MIT License
228 stars 61 forks source link

inverser les arguments dans la signature de PacketListenerCallbackClient #2705

Open romain22222 opened 1 month ago

romain22222 commented 1 month ago

le contexte étant BIEN BIEN plus utilisé que le packet, il faudrait inverser les deux. Je fais une issue afin que je le fasse dans une PR à part, mais faudra le faire (y'a bien trop d'endroits où y'a un packet non utilisé pour l'handler)

romain22222 commented 1 month ago

image exemple typique dans les smallEvents : le packet est juste là pour indiquer le message, mais ne contient rien d'important, on n'a besoin que du contexte

Feiryn commented 1 week ago

Pour info, je viens de passer 40 min à essayer d'omettre le paquet dans les paramètres des handlers mais je n'ai pas réussi, il ne veut pas accepter mon type il est trop capricieux

Feiryn commented 8 hours ago

Bon je n'arrive pas à faire quelque chose pour ça et je n'ai pas envie de perdre trop de temps dessus. Je vais changer l'ordre et ensuite on verra si on arrive à rendre le paramètre optionel d'une façon ou d'une autre