Skip to content

Implement open save menu and save menu access management commands - #815

Merged
AerunDev merged 3 commits into
developfrom
581-open-save-menu-command
Sep 2, 2026
Merged

Implement open save menu and save menu access management commands#815
AerunDev merged 3 commits into
developfrom
581-open-save-menu-command

Conversation

@Aelysya

@Aelysya Aelysya commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR adds two commands to the list of available ones, Open Save Menu and Manage Save Menu Access.

closes #580
closes #581
closes #597

Tests to perform

  • The Open Save Menu command does not have any parameters
  • An editor still opens but displays a simple information message

  • The Manage Save Menu Access commands displays a select
  • Modifying the value in the select correctly changes the displayed value on the node and in the JSON
  • Modifying the value from the editor works too

  • The style is satisfactory
image

@Aelysya Aelysya self-assigned this Aug 31, 2026
@Aelysya Aelysya linked an issue Aug 31, 2026 that may be closed by this pull request
6 tasks

@Palbolsky Palbolsky left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Image (désolé pour l'image mais c'était trop tentant 😆)

Rien à dire, j'approuve. GG. 👍

@AerunDev AerunDev left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hey ! J'ai pu tester la PR, et pour moi tout est bon, bien joué !

Une remarque tout de même au niveau du wording entre "Instructions" affiché dans le titre de la bibliothèque de commandes, et "commandes" dans le message dans le cas où un éditeur n'a pas de champ à afficher.
Il faudra qu'on voit si on souhaite harmoniser ou non.

@Aelysya

Aelysya commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator Author

Hey ! J'ai pu tester la PR, et pour moi tout est bon, bien joué !

Une remarque tout de même au niveau du wording entre "Instructions" affiché dans le titre de la bibliothèque de commandes, et "commandes" dans le message dans le cas où un éditeur n'a pas de champ à afficher. Il faudra qu'on voit si on souhaite harmoniser ou non.

J'ai fait un commit pour harmoniser les termes anglais avec le français, du coup j'en ai profité pour changer "commande" par "instruction" partout, ce sera plus cohérent si on utilise les mêmes termes partout dans l'appli.

@Walven

Walven commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Une remarque tout de même au niveau du wording entre "Instructions" affiché dans le titre de la bibliothèque de commandes, et "commandes" dans le message dans le cas où un éditeur n'a pas de champ à afficher. Il faudra qu'on voit si on souhaite harmoniser ou non.

Excellente remarque. Pour clarifier : que le terme reste "Command" dans le code me pose aucun problème.

En revanche, il faudra prendre bien soin de n'utiliser et n'afficher que le terme "Instruction(s)" dans les interfaces.

C'est important pour éviter les homonymes et la confusion. On est dans un univers technique, "Commande" est vaste et fait déjà référence à d'autres objets, les commandes de script ou des commandes "/" dans un terminal par exemple. On construit un flux algorithmique, ce qui est parfaitement une suite d'instructions.

J'essaie moi-même de m'efforcer à utiliser ce terme dans les maquettes et la documentation. Essayons d'harmoniser ça !

@AerunDev
AerunDev merged commit e760e87 into develop Sep 2, 2026
5 checks passed
@AerunDev
AerunDev deleted the 581-open-save-menu-command branch September 2, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

📁 Game Save Commands Open Save Menu Command Save Menu Access Command

4 participants