Skip to content

Conversation

@agateblue
Copy link
Contributor

@hellochloe1989 je te laisse relire / tester le fix si tu as envie (sinon c'est okay, je mergerai dans quelques jours !)

@hellochloe1989
Copy link
Contributor

J'ai pu relire et tester, c'est bon pour moi, merci pour le fix !

@hellochloe1989
Copy link
Contributor

hellochloe1989 commented Oct 1, 2025

On aurait pu aussi revoir le système des actions pour avoir une propriété actionOnUpdate qui s'exécuterait pour les update ou immédiatement après le prefill du champ individuel, et une propriété actionOnPrefillComplete qui s'exécuterait à la fin de tous les prefill. L'action du type d'organisme aurait pu devenir une actionOnUpdate du coup, ca aurait résolu l'écrasement et on aurait gardé la possibilité d'avoir des actions lors du prefill (même si à l'heure actuelle je ne vois pas encore d'utilisation possible des actions au prefill tbh 😅).

Mais cela n'est pas super utile de complexifier les actions pour le moment, surtout qu'elles ne sont pas utilisées tant que ça.
Peut être plus tard qui sait 😁

@agateblue agateblue merged commit 0708671 into main Oct 2, 2025
1 check passed
@agateblue
Copy link
Contributor Author

merci pour la relecture !

@agateblue agateblue deleted the 62-action-set branch October 2, 2025 17:07
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.

3 participants