Ajout d'un événement journalier (Moisson Dorée) - Partie finale#1348
Ajout d'un événement journalier (Moisson Dorée) - Partie finale#1348iambibi wants to merge 23 commits into
Conversation
# Conflicts: # src/main/java/fr/openmc/core/features/events/contents/dailyevents/contents/miraculousfishing/MiraculousFishingEvent.java # src/main/java/fr/openmc/core/registry/items/CustomItemRegistry.java
# Conflicts: # src/main/java/fr/openmc/core/OMCPlugin.java # src/main/java/fr/openmc/core/registry/items/CustomItemRegistry.java
|
Pret pour etre review (je viens de finir l'événement), je m'attaque a la suite, mine de rien la v2.5 est vrm bientot prete xD |
|
ptdr je me suis enflammé trop tot, j'ai encore les interfaces à faire |
|
pour le coup j'ai fini, pret à review. |
| /** | ||
| * Interface permettant aux classes d'enregistrer une sous feature | ||
| */ | ||
| public interface HasFeature { |
There was a problem hiding this comment.
Si c'est une sous feature alors le nom plus adapter serai HasSubFeature
There was a problem hiding this comment.
Ben oui mais je veux dire que mon événement à une feature lié. Pas forcément une sub feature
There was a problem hiding this comment.
Ben c'est littéralement ton commentaire
sous feature
Voiciiiii :) J'ai check un peu se qu'il avais écris je t'ai mis que se que je trouve pertinant et je lui ai indiqué deux trois truc que j'ai pu voir moi aussi, hésite pas a me dire si tu veux que j'en solve qlq un 7$ de coût 🥀 |
3 participants
Petit résumé de la PR:
Étape nécessaire afin que la PR soit fini (si PR en draft)
Decrivez vos changements