Patch du bug du /settings et mailbox + cache #457
Conversation
|
Faire un cache |
|
(Pas de requests SQL h24) |
|
revérifiez mon code pour voir si ça marche bien SVP |
Tu toi test normalement |
|
de quoi ? |
|
bah faut pas |
|
Toujours pas, quand le joueur ce connecte tu load toujours la db |
|
non |
gab4000
changed the title
|
tu as bien test avec un second compte ? |
# Conflicts: # src/main/java/fr/communaywen/core/AywenCraftPlugin.java
src/main/java/fr/communaywen/core/mailboxes/menu/letter/SendingLetter.java
Show resolved
Hide resolved
|
|
||
| @EventHandler | ||
| public void onJoin(PlayerJoinEvent e) { | ||
| Bukkit.getScheduler().runTaskAsynchronously(this.plugin, () -> { |
There was a problem hiding this comment.
ta boucle est innutile elle va juste mettre Player dans un autre thread et le reste sur le principal thread
|
mais? pourquoi avoir rechanger le taux max d'item? |
|
bon laisse tomber je laisse la main au autres |
gab4000
changed the title
|
🫤 |
|
regarde les review et ce seras good |
src/main/java/fr/communaywen/core/settings/SettingsListener.java
Outdated
Show resolved
Hide resolved
src/main/java/fr/communaywen/core/mailboxes/menu/letter/SendingLetter.java
Outdated
Show resolved
Hide resolved
src/main/java/fr/communaywen/core/mailboxes/menu/letter/SendingLetter.java
Outdated
Show resolved
Hide resolved
|
c bon au cas ou |
gab4000
left a comment
gab4000
left a comment
There was a problem hiding this comment.
prêt a être merge
|
a merge, je pense que les 2 joueurs actifs sur le serveur seront content d'avoir leur mailbox |
|
Good @AxenoDev ? |
Xernas78
left a comment
Xernas78
left a comment
There was a problem hiding this comment.
Oké je fais confiance a axewoui
5 participants
Avez vous lu le Code de Conduite?: oui
Votre code se compile t-il en local ?: oui
Avez-vous supprimez les imports inutilisés ?: oui
Décrivez vos changements
Clairement et avec des screenshots si nécessaires
Ecriture des paramètres automatique lors de la connexion pour ne pas retourner null lors de l'ouverture des paramètres
Ajout d'un cache pour éviter les requêtes SQL permanentes
Réparation du /players
Réparation de la mailbox
Nouveau namespace pour le /settings :
Pensez à retirer le namespace omc_settings, s'il existe et de mettre celui là
settings.zip