Skip to content

Conversation

@Syxless
Copy link
Contributor

@Syxless Syxless commented Dec 2, 2025

Petit résumé de la PR:

Correction du systeme d'application automatique des perks : le systeme ne selectionne plus des perks non debloques par la ville.

É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

  • Ajout du filtrage isPerkUnlockedForCity dans toutes les fonctions de generation de perks (basic, event, all).
  • Mise a jour des appels dans MayorManager pour utiliser les nouvelles signatures (city en parametre).
  • Ajout d'un check qui stoppe la creation du maire si aucune combinaison valide de perks n'est disponible, avec un log warn pour l'indiquer.
  • Je n'ai pas pu tester la fonctionnalite car je ne dispose pas du plugin ItemsAdder.

Copy link
Contributor

@gab4000 gab4000 left a comment

Choose a reason for hiding this comment

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

Teste, vérifie que ça n'entrave pas les autres features liées, et mets des loggers en anglais stp 😊

@Syxless
Copy link
Contributor Author

Syxless commented Dec 3, 2025

Tout a été testé, selon mes tests, ça fonctionne !

@gab4000
Copy link
Contributor

gab4000 commented Dec 4, 2025

Pour moi tout est bon, j'ai pas la perm d'accepter mais ça me paraît bien, reste plus qu'une review d'un manager

@iambibi iambibi added this to the 2.2.0 milestone Dec 4, 2025
@iambibi iambibi added the ✨ Fixes Quelque chose a été patch label Dec 4, 2025
@iambibi iambibi assigned iambibi and Syxless and unassigned iambibi Dec 4, 2025
iambibi
iambibi previously approved these changes Dec 4, 2025
Copy link
Member

@iambibi iambibi left a comment

Choose a reason for hiding this comment

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

et ben ça me parait vrm clean pour une premiere PR, surtout en ayant pas énormément connaissance de notre structure des villes, maires, perk, ect

@iambibi iambibi merged commit c47dee0 into ServerOpenMC:master Dec 6, 2025
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Fixes Quelque chose a été patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Application des perks automatiques

3 participants