search

oltodo / worldcup-for-slack

2018 Worldcup Slack bot
0 stars 1 forks source link

Add secondary events #14

Closed wulfy closed 6 years ago

wulfy commented 6 years ago

c'est un début, reste à trouver les events et à, peut etre, améliorer le code

oltodo commented 6 years ago

La PR est bien mais je pense que ce que tu appelles ici "évènements secondaires" devraient en fait faire parti des évènements "primaires". Donc rajoute ces évènements comme des évènements normaux et garde ton code du lastEventEmit pour des évènements vraiment secondaires du type faute, changement ou hors-jeu.

wulfy commented 6 years ago

ok j'ajoute les quelques évènements qui sont détectés correctement en évènements primaires et je trouverai d'autres évènements secondaires (effectivement la faute)

wulfy commented 6 years ago

@oltodo ca m'a l'air bon j'ai fait quelques tests. J'ai ajouté les coups francs pour que les event "faute" ne restent pas sans rien (ça fait bizarre une faute et rien derrière)

J'ai laissé remise en jeu comme évènement secondaire.

wulfy commented 6 years ago

@oltodo rebasé et ready to merge (si tu veux que je le fasse ;) )