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]: Problèmes de temps #1589

Closed
Nargacaura opened this issue Nov 9, 2022 · 7 comments · Fixed by #1592 or #1619
Closed

[BUG]: Problèmes de temps #1589

Nargacaura opened this issue Nov 9, 2022 · 7 comments · Fixed by #1592 or #1619
Labels
bug Something isn't working
Milestone

Comments

@Nargacaura
Copy link
Contributor

Nargacaura commented Nov 9, 2022

Description du bug.

  • Le trajet entre 2 mini-events dure 9 minutes au lieu de 10 8min45 environ au lieu de 9min45 sur la version stable et l'alpha depuis la tentative de fix, et le tuto ne dure que 2 minutes au lieu de 3. [Corrigé pour le tuto]
  • Les altérations d'état durent une minute de plus. [Merging depuis [BUG]: une minute supplémentaire sur les altés  #1607]

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

😨 Grave

Reproduction du bug.

  • Faire /report afin d'avoir un mini-event, et un autre afin d'avoir l'état du voyage.
  • Subir une altération d'état (peu importe sa durée) et regarder son profil via /profile ou faire /report.

Captures d'écran

Cas du tuto: [Corrigé]
Capture d’écran 2022-11-09 à 19 14 31

Cas des altérations d'état: voir l'issue mentionnée dans la description du bug.

Sur quel shard avez vous rencontré le bug ?

Tous

@Nargacaura Nargacaura added the bug Something isn't working label Nov 9, 2022
@BastLast BastLast added this to the 4.0.3.a milestone Nov 9, 2022
@Pietagorh
Copy link
Member

Screen pour le temps entre 2 minis ?

@Nargacaura
Copy link
Contributor Author

Nargacaura commented Nov 9, 2022

Les deux reports ont été fait immédiatement l'un après l'autre.
Capture d’écran 2022-11-09 à 20 25 17

@Pietagorh
Copy link
Member

Par rapport au tuto, j'suis quasi sûr que c'est ça
await Maps.startTravel(player, newLink, interaction.createdAt.valueOf(), NumberChangeReason.BIG_EVENT, interaction.createdAt); où faudrait mettre des Date.now(), parce qu'il a pu se passer 2 minutes entre les deux en soit

Par contre pour le minievent aucune idée

@BastLast BastLast modified the milestones: 4.0.3.a, 4.0.3.b Nov 11, 2022
@BastLast BastLast mentioned this issue Nov 11, 2022
@BastLast BastLast linked a pull request Nov 11, 2022 that will close this issue
@Nargacaura Nargacaura changed the title [BUG]: Raccourcissement d'un trajet d'une minute par mini-event [BUG]: Problèmes de temps Nov 13, 2022
@BastLast BastLast reopened this Nov 14, 2022
@BastLast
Copy link
Member

le bug n'existe pas sur les bot bêta mais touche le bot alpha et le bot en prod @niqore j'ai besoin de toi, je ne comprend pas pourquoi la gestion des dates serait différente dans un environnement dockerisé, on a probablement un truc qui donne une valeur avec 1 minute d'écart j'ai essayer de comprendre mais j'ai pas réussi.

@BastLast
Copy link
Member

je pense que cela vient des changements que @romain22222 a fait sur l'utilisation de la date now au lieu de la date du message ou je sais plus quoi

@BastLast BastLast modified the milestones: 4.0.4, 4.1.0 Nov 15, 2022
@BastLast
Copy link
Member

@Pietagorh tu peux t'occuper de ça ce soir ?

@Pietagorh
Copy link
Member

J'vais voir

Pietagorh pushed a commit that referenced this issue Nov 17, 2022
@Pietagorh Pietagorh linked a pull request Nov 17, 2022 that will close this issue
BastLast pushed a commit that referenced this issue Nov 17, 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.

3 participants