Permet de créer un modèle depuis un événement, l'écrasement devenant une option - #26
Merged
Merged
Conversation
…une option La version précédente ne savait qu'écraser : sans modèle existant, le bouton était désactivé et renvoyait l'utilisateur en créer un ailleurs, avant de revenir y verser la check-list. C'était le cas d'usage principal — faire d'un événement mis au point un modèle réutilisable — qui était le seul impossible. « Copier dans les modèles » propose maintenant les deux, la création par défaut. Un nouveau modèle reprend le titre et la description de l'événement en plus de ses tâches : ce sont les seuls champs qu'un modèle porte. Le remplacement d'un modèle existant reste offert, précédé de sa confirmation nommant le modèle et le nombre de tâches perdues, et protégé par le verrou de version. Le bouton n'est plus désactivé que si la check-list est vide, seul cas où il n'y a réellement rien à copier. Côté MCP, overwrite_template_from_event devient save_event_as_template et couvre les deux chemins. La confirmation n'est exigée que pour un remplacement : créer un modèle ne détruit rien. Vérifié sur un PostgreSQL 16 avec les migrations du projet : création reprenant titre, description et les 3 tâches avec leurs unités d'affichage, écrasement passant de 1 à 3 tâches, et refus d'une écriture sur version périmée. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014SfQYBU4xXTeSEHKhHQXdD
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Le défaut de conception
La version précédente (#23) ne savait qu'écraser. Sans modèle existant, le bouton était désactivé et l'interface renvoyait l'utilisateur en créer un ailleurs, à la main, avant de revenir y verser la check-list.
Autrement dit : le cas d'usage principal — faire d'un événement qu'on vient de mettre au point un modèle réutilisable — était le seul que la fonctionnalité ne couvrait pas. Signalé en revue, corrigé ici.
Ce que ça devient
Un bouton, « Copier dans les modèles », qui propose les deux, la création par défaut :
Le bouton n'est plus désactivé que si la check-list est vide — le seul cas où il n'y a réellement rien à copier.
Serveur MCP
overwrite_template_from_eventdevientsave_event_as_templateet couvre les deux chemins : sanstemplateIdil crée, avec il remplace.confirmn'est exigé que pour un remplacement — créer un modèle ne détruit rien, l'exiger serait du bruit.Vérification
Rejoué sur un PostgreSQL 16 avec les migrations du projet, sur un événement de 3 tâches et un modèle existant de 1 tâche :
{"title":"Réserver le photobooth","description":"Six mois à l'avance","leadTimeDays":180,"leadTimeUnit":"months","leadTimeValue":6}— unités d'affichage conservéestsc --noEmit,npm run lintetnext buildpassent.Generated by Claude Code