Skip to content

Ajout d'un roulement dans le ScoreBoard - #1512

Merged
iambibi merged 5 commits into
ServerOpenMC:masterfrom
Nocolm:SB
Oct 25, 2026
Merged

iambibi merged 5 commits into
ServerOpenMC:masterfrom
Nocolm:SB

Conversation

@Nocolm

@Nocolm Nocolm commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Petit résumé de la PR:

Permet d'affiché des informations tour par tour comme la boss bar mais dans le scoreboard

Étape nécessaire afin que la PR soit fini (si PR en draft)

  • Suivre le Code de Conduite
  • Enlever tous les imports non utilisés
  • Bien documenter la feature
  • Fournir un profileur (si besoin/demandé par un admin)
  • Avoir une milestone associée à la PR
  • Valider tout les checks
  • Tester et valider la feature/changement
  • Les Issues corrigée(s) en commun :

Decrivez vos changements

J'ai ajouté un roulement dans le scoreboard, c'est à dire que si le contest est en coure et qu'un autre event aussi, au lieu d'avoir un immense scoreboard, ils vont s'afficher a tour de rôle tous les X temps. Il y a un setting pour passer en mode Rapide, Normal, Lent qui change le temps entre chaque roulement, et un mode Tous qui affichera toutes les informations sans roulement

@Nocolm

Nocolm commented Oct 25, 2026

Copy link
Copy Markdown
Contributor Author

heu ok sa a merdé j'adore intelij T_T

Removed unused HalloweenManager import.

@iambibi iambibi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Juste le petit oubli du mot clé et c'est níquel


private int currentSwappingTime = 200; // Pour éviter d'attendre XXs au début
private int currentSwappingInfoIndex = 0;
InfoSwapper currentInfo = null;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prívate ? public ? Généralement vaut mieux préciser et au moins on se pose pas la question 👍

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a oui bien vu

@iambibi
iambibi merged commit 34deeaa into ServerOpenMC:master Oct 25, 2026
2 of 3 checks passed
@Nocolm

Nocolm commented Oct 25, 2026

Copy link
Copy Markdown
Contributor Author

NAN ATEND

@Nocolm

Nocolm commented Oct 25, 2026

Copy link
Copy Markdown
Contributor Author

il y a une erreur a un endroit a cause des différences entre mes branches

@Nocolm
Nocolm deleted the SB branch October 25, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants