Skip to content

Adminshop: Retour au menu après achat/vente et vente rapide - #1596

Merged
iambibi merged 3 commits into
ServerOpenMC:masterfrom
PuppyTransGirl:fix/adminshop-closing
Nov 2, 2026
Merged

iambibi merged 3 commits into
ServerOpenMC:masterfrom
PuppyTransGirl:fix/adminshop-closing

Conversation

@PuppyTransGirl

Copy link
Copy Markdown
Member

Petit résumé de la PR:

Après avoir confirmé un achat ou une vente dans l'AdminShop, le joueur revient maintenant au menu d'où il vient (catégorie ou variantes) au lieu de voir son inventaire se fermer, et un Shift + clic droit permet de vendre tout un item d'un coup.

É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

Decrivez vos changements

  • ConfirmMenu reçoit un callback onFinish exécuté après la validation, ce qui permet de rouvrir le menu précédent (catégorie, couleurs, bûches ou feuilles).
  • openBuyConfirmMenu et openSellConfirmMenu prennent ce callback en paramètre, et leur Javadoc a été mise à jour.
  • Ajout de AdminShopManager#sellAll et gestion du Shift + clic gauche dans le menu de catégorie et les menus de variantes pour vendre tout le stock d'un item instantanément.
  • Le Shift + clic droit dans ConfirmMenu règle la quantité au maximum vendable, comme l'annonçait déjà la ligne de lore sell_all.
  • La ligne sell_all est affichée dans le lore des items vendables pour que les joueurs connaissent le raccourci.

@PuppyTransGirl PuppyTransGirl added this to the 2.6.0 milestone Nov 2, 2026
@PuppyTransGirl PuppyTransGirl self-assigned this Nov 2, 2026
@PuppyTransGirl PuppyTransGirl added the 🆙 Amélioration Une amélioration sur quelque chose label Nov 2, 2026
@iambibi iambibi modified the milestones: 2.6.0, 2.5.9 Nov 2, 2026
@iambibi
iambibi merged commit b811baf into ServerOpenMC:master Nov 2, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🆙 Amélioration Une amélioration sur quelque chose

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants