DraftBot-A-Discord-Adventure / DraftBot

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

[BUG]: double message level up #1515

Closed KirIcare closed 2 years ago

KirIcare commented 2 years ago

Description du bug.

Comme le nom l'indique, 2 messages pour un qu'un seul level up

Selon vous, quelle est l'importance de ce bug ?

🙂 Faible

Reproduction du bug.

Je sais c'est pas moi mais level up je suppose ?

Captures d'écran

IMG_20221012_102330

Sur quel shard avez vous rencontré le bug ?

Shard 3

Procion06 commented 2 years ago

Double message mission complétée

Screenshot_12

BastLast commented 2 years ago

Possible d'avoir plus d'infos ? Comment l'xp qui a fait le montage de level a été gagnée ? Quelles ressources ont été perçues, quelle est l'id discord du joueur affecté ?

BastLast commented 2 years ago

je rouvre suite au problème relevé par @KirIcare , si tu as plus d'infos d'ailleurs , hésite pas à envoyer tout ce que tu sais à l'occasion

KirIcare commented 2 years ago

Alors tout ce que je sais c'est que: -je me suis mis lv50 manuellement -au moment où je suis passé level 51, j'ai pris 525xp, 25 pour l'évent et 500 grâce aux deux missions de campagne (58 et 59) complété d'un coup -j'ai également était mis en alté suite à l'évent (aucun problème de ce côté juste je précise tout ce qu'il s'est passé)

KirIcare commented 2 years ago

J'ai réussi à le reproduire en test et le point commun à l'air d'être le fait de compléter de mission de campagne en même temps. Premier screen j'ai level up en complétant une mission de campagne et j'ai eu qu'un message. Deuxième screen j'ai encore level up en complétant 2 missions de campagne cette fois et le bug s'est reproduit On peut voir que lorsque j'ai ré ouvert l'issue, j'ai eu le double message en complétant 2 missions de campagne également IMG_20221028_005222 IMG_20221028_005244

BastLast commented 2 years ago

Ça ça aide énormément ! Merci !!!

KirIcare commented 2 years ago

Je pense que y'a plus besoin de chercher IMG_20221028_010515 Screenshot_20221028-010459

KirIcare commented 2 years ago

Le seul truc qu'il me resterais à test c'est de savoir si ça s'applique qu'aux missions de campagne ou également aux missions secondaires

BastLast commented 2 years ago

Pas besoin de tester ça c'est sur ça s'applique aux 2

KirIcare commented 2 years ago

Alors c'est bon mon rôle s'arrête ici Je suis content pour une fois que j'arrive à trouver un truc de ma propre initiative, ça fait plaisir et ça donne envi de test encore plus mdr

KirIcare commented 2 years ago

Juste pour info, ça parais logique mais au cas où, le bug s'applique peut importe le nombre de missions complétés tant qu'il y en a au moins 2 Screenshot_20221031-201504

Nargacaura commented 2 years ago

Pour apporter un peu plus de précisions sur ce bug, il y a autant de messages de level up que de missions complétées. Quelque chose laisse donc à penser que le message de level up est vérifié à chaque mission complétée. Par exemple, j'ai reproduit le bug sur le bot alpha avec une mission de campagne et 3 missions secondaires... 4 messages de level up. Les screens précédents le montrent aussi (4 messages de level up pour 4 missions, ou 2 pour 2 missions).

Capture d’écran 2022-11-01 à 15 58 50

Si le check de level up est effectué sur la mission, il faudrait le sortir de la boucle de check de complétion, ou bien exécuter chaque action pour chaque mission au lieu de tout balancer à la fin.