Skip to content

Changements sur les grades #1056

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
gab4000 wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Changements sur les grades #1056

gab4000 wants to merge 12 commits into from
+269 −67

Conversation

@gab4000
Copy link
Contributor

@gab4000 gab4000 commented Nov 1, 2025 •
edited
Loading

Petit résumé de la PR:

  • On peut désormais ajouter et retirer toutes les perms d'un joueur/rank en 1 fois
  • Correctif du reload du menu des permissions
  • On ne peut plus gérer un grades de même niveau/plus important que le sien

É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

  • Ajout des boutons dans les menus de perms
  • Retrait de closeInventory sur le menu des permissions
  • Vérification à chaque modifications si le rank du joueur y est autorisé

This was linked to issues Nov 1, 2025
[BUG] Grades impossible voir perms #1015
Open
[CHANGE] Mettre toutes les permissions d'un coup #1023
Open
iambibi
iambibi requested changes Nov 1, 2025
View reviewed changes
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.

Arrete voir avec tes fourres tout (c'est vraiment pas du tout mais du tout compréhesible par les joueurs)
Met ta pr en draft si pas encore testé

@gab4000 gab4000 marked this pull request as draft November 1, 2025 18:04
@gab4000 gab4000 changed the title Ajout du "tout ajouter/retirer" sur les permissions CHangements sur les grades Nov 1, 2025
@gab4000 gab4000 changed the title CHangements sur les grades Changements sur les grades Nov 1, 2025
@gab4000
Copy link
Contributor Author

gab4000 commented Nov 2, 2025

PR en pause pendant qq semaines

@gab4000 gab4000 marked this pull request as ready for review January 21, 2026 16:28
@gab4000
Copy link
Contributor Author

gab4000 commented Jan 21, 2026

Prêt à être merge

@iambibi
Copy link
Member

iambibi commented Jan 22, 2026

Je review lorsque j'aurais le time, on tente de merge ça samedi en tout cas

iambibi
iambibi requested changes Jan 22, 2026
View reviewed changes
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.

Juste des petits commentaires d'information, si tout est réglé j'update la version du plugin samedi

src/main/java/fr/openmc/core/features/city/menu/CityPermsMenu.java
List<Component> lore = List.of(
Component.text("§e§lCLIQUEZ POUR " + (hasPerm ? "RETIRER" : "AJOUTER") + " CETTE PERMISSION")
);
itemMeta.lore(edit ? lore : List.of());
Copy link
Member

@iambibi iambibi Jan 22, 2026

Choose a reason for hiding this comment

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

Tu retires bien le lore qui te dit Modifier les permissions de ce grade lorsque tu ne peux pas edit?

Copy link
Contributor Author

@gab4000 gab4000 Jan 22, 2026

Choose a reason for hiding this comment

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

Le problème c que jpp tout test, g que 1 seul compte

Copy link
Contributor Author

@gab4000 gab4000 Jan 22, 2026

Choose a reason for hiding this comment

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

Soit je demande a des gens, ce que g déjà fait, ou alors j'essaie de me co en crack

Copy link
Member

@iambibi iambibi Jan 22, 2026

Choose a reason for hiding this comment

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

Le problème c que jpp tout test, g que 1 seul compte

Lol de toute façon je m'en fiche, mais demande des betas testeur au lieu de tt tester sur le serveur

Copy link
Member

@iambibi iambibi Jan 22, 2026

Choose a reason for hiding this comment

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

Le problème c que jpp tout test, g que 1 seul compte

Lol de toute façon je m'en fiche, mais demande des betas testeur au lieu de tt tester sur le serveur

Copy link
Member

@iambibi iambibi Jan 22, 2026

Choose a reason for hiding this comment

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

Soit je demande a des gens, ce que g déjà fait, ou alors j'essaie de me co en crack

Je fais ça perso

Copy link
Member

@iambibi iambibi Jan 23, 2026

Choose a reason for hiding this comment

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

Le problème c que jpp tout test, g que 1 seul compte

donc tu veux jouer au pile ou face? la prochaine fois tu devras tester par contre

Copy link
Contributor Author

@gab4000 gab4000 Jan 23, 2026

Choose a reason for hiding this comment

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

Nan nan tkt, je v test mais pas eu le temps ces derniers jours, je ferrai ça demain soir ou dimanche

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iambibi iambibi iambibi requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[CHANGE] Mettre toutes les permissions d'un coup [BUG] Grades impossible voir perms

2 participants

@gab4000 @iambibi