Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG]: double message level up #1515

Closed
KirIcare opened this issue Oct 12, 2022 · 12 comments · Fixed by #1660
Closed

[BUG]: double message level up #1515

KirIcare opened this issue Oct 12, 2022 · 12 comments · Fixed by #1660
Assignees
Labels
bug Something isn't working
Milestone

Comments

@KirIcare
Copy link

KirIcare commented Oct 12, 2022

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

@KirIcare KirIcare added the bug Something isn't working label Oct 12, 2022
@BastLast BastLast modified the milestones: 4.0.1, 4.0.0.e, 4.0.0.f Oct 12, 2022
@Procion06
Copy link

Double message mission complétée

Screenshot_12

@BastLast BastLast modified the milestones: This week, next week Oct 21, 2022
@BastLast
Copy link
Member

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
Copy link
Member

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

@BastLast BastLast reopened this Oct 27, 2022
@BastLast BastLast modified the milestones: 4.0.1.c, 4.0.2.a Oct 27, 2022
@KirIcare
Copy link
Author

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
Copy link
Author

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
Copy link
Member

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

@KirIcare
Copy link
Author

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

@KirIcare
Copy link
Author

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
Copy link
Member

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

@KirIcare
Copy link
Author

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

@BastLast BastLast modified the milestones: 4.0.2.a, 4.0.2.b Oct 30, 2022
@KirIcare
Copy link
Author

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
Copy link
Contributor

Nargacaura commented Nov 1, 2022

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.

@BastLast BastLast modified the milestones: 4.0.2.b, 4.1.0 Nov 6, 2022
@voltou91 voltou91 self-assigned this Dec 4, 2022
voltou91 added a commit that referenced this issue Dec 4, 2022
@voltou91 voltou91 linked a pull request Dec 4, 2022 that will close this issue
voltou91 added a commit that referenced this issue Dec 4, 2022
BastLast pushed a commit that referenced this issue Dec 5, 2022
BastLast pushed a commit that referenced this issue Dec 5, 2022
@BastLast BastLast closed this as completed Dec 5, 2022
@BastLast BastLast modified the milestones: 4.2.0, 4.1.1 Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants