feat: Ajouter des appels de notifications normalisées pour tous les objets lors d'insertion et modifs fix #4874 #5304

Open
rastapopoulos wants to merge 3 commits from dev/issue_4874_normaliser_notifs into master
Owner

Ajoute ces notifs dans editer_objet générique
et dans les objets qui ont des fichiers dédiés (quelle horreur ces fichiers dédiés énormes qui doublonnent une bonne partie du editer_objet générique, on pourrait pas en supprimer un maximum et déplacer les spécificités qu'en pipeline ?)

Possible à continuer dans les plugins dist qui ont des fichiers d'action dédiés aussi (pfff)

Les anciens appels sont toujours là pour instituertruc moche, à déprécier

Ajoute ces notifs dans editer_objet générique et dans les objets qui ont des fichiers dédiés (quelle horreur ces fichiers dédiés énormes qui doublonnent une bonne partie du editer_objet générique, on pourrait pas en supprimer un maximum et déplacer les spécificités qu'en pipeline ?) Possible à continuer dans les plugins dist qui ont des fichiers d'action dédiés aussi (pfff) Les anciens appels sont toujours là pour instituertruc moche, à déprécier
rastapopoulos added 2 commits 4 weeks ago
b_b approved these changes 4 weeks ago
Owner

On a besoin de notification tel que "article_inserer" en plus de "objet_inserer" qui gèrerait le dispatch ? (question?)

On a besoin de notification tel que "article_inserer" en plus de "objet_inserer" qui gèrerait le dispatch ? (question?)
Poster
Owner

Bé ni plus ni moins que "instituer" qui était bien par objet aussi ?

Pour les notifs ça peut quand même être très différents suivant les objets donc ça me parait toujours utile d'avoir des notifs faciles à implémenter pour TEL objet précis.

Mais par contre c'est sûrement une bonne idée aussi (en plus) d'ajouter des appels de notifs génériques avec "objet" et ça pour tout du coup, pas que "inserer".

Donc quand ya un événement yorait des notifs potentielles (car c'est toujours que potentielles par défaut) :

  • patate_inserer + objet_inserer
  • patate_modifier + objet_modifier
  • patate_instituer + objet_instituer

Ça pourrait être ajouté dans la foulée oui.

Bé ni plus ni moins que "instituer" qui était bien par objet aussi ? Pour les notifs ça peut quand même être très différents suivant les objets donc ça me parait toujours utile d'avoir des notifs faciles à implémenter pour TEL objet précis. Mais par contre c'est sûrement une bonne idée aussi (en plus) d'ajouter des appels de notifs génériques avec "objet" et ça pour tout du coup, pas que "inserer". Donc quand ya un événement yorait des notifs *potentielles* (car c'est toujours que potentielles par défaut) : - patate_inserer + objet_inserer - patate_modifier + objet_modifier - patate_instituer + objet_instituer Ça pourrait être ajouté dans la foulée oui.
Owner

Ça pourrait être ajouté dans la foulée oui.

Du coup c'est encore en WIP ou on peut approuver maintenant ?

> Ça pourrait être ajouté dans la foulée oui. Du coup c'est encore en WIP ou on peut approuver maintenant ?
Poster
Owner

Bé non, ça c'est une demande/réflexion qui a été dites après coup. Ça c'est pas fait

Bé non, ça c'est une demande/réflexion qui a été dites après coup. Ça c'est pas fait
rastapopoulos added 1 commit 1 week ago
Poster
Owner

Et là c'est ajouté (les génériques objet_XXX en plus de celles par objet)

Et là c'est ajouté (les génériques objet_XXX en plus de celles par objet)
b_b added 3 commits 1 week ago
tcharlss approved these changes 1 week ago

Reviewers

b_b approved these changes 4 weeks ago
tcharlss approved these changes 1 week ago
This Pull Request is blocked because it's outdated.
This branch is out-of-date with the base branch
Sign in to join this conversation.
Loading…
There is no content yet.