issue_5603 déclaration du du pipeline notifications_destinataires #5604

Merged
JamesRezo merged 2 commits from cpol/spip_cpol0:issue_5603 into master 1 week ago
cpol commented 1 week ago
  • Testé sur une 4.1.9
  • Je ne sais pas s'il faut incrémenter le numéro de version du paquet ecrire, dites moi si besoin et je repusherai.

Thanks :)

* Testé sur une 4.1.9 * Je ne sais pas s'il faut incrémenter le numéro de version du paquet `ecrire`, dites moi si besoin et je repusherai. Thanks :)
cpol added 2 commits 1 week ago
99932ebb85 fix: ajout pipeline notifications_destinataires
Évite le log d'une erreur lors des appels à ce pipeline

Fix #5603
tcharlss approved these changes 1 week ago
JamesRezo approved these changes 1 week ago
b_b approved these changes 1 week ago
b_b left a comment
Owner

PR parfaite, bon log de commit, un changelog comme il faut, bravo :)

PR parfaite, bon log de commit, un changelog comme il faut, bravo :)
Owner
  • Je ne sais pas s'il faut incrémenter le numéro de version du paquet ecrire, ...

Non, c'est un peu plus complexe que ça.

Testé sur une 4.1.9

Ce sera au mieux reporté dans la branche 4.2.

Ce n'est pas une faille de sécu et la branche 4.1 est en "security fix" jusqu'en aout prochain.

> * Je ne sais pas s'il faut incrémenter le numéro de version du paquet `ecrire`, ... Non, c'est un peu plus complexe que ça. > Testé sur une 4.1.9 Ce sera au mieux reporté dans la branche 4.2. Ce n'est pas une faille de sécu et la branche 4.1 est en "security fix" jusqu'en aout prochain.
JamesRezo merged commit d92d656aa6 into master 1 week ago
cpol commented 1 week ago
Poster

@b_b merci :)

@JamesRezo oui j'avais bien vu que la 4.1 est en security fix only et plus en maintenance active, mais c'était surtout pour prévenir les relecteurs que je ne l'ai pas testé sur une 4.2 et encore moins sur la branche master.
Je vais merger dans ma 4.1 de mon côté, ça ira bien comme jusqu'à la MAJ 4.2 ;)

@b_b merci :) @JamesRezo oui j'avais bien vu que la 4.1 est en security fix only et plus en maintenance active, mais c'était surtout pour prévenir les relecteurs que je ne l'ai pas testé sur une 4.2 et encore moins sur la branche master. Je vais merger dans ma 4.1 de mon côté, ça ira bien comme jusqu'à la MAJ 4.2 ;)
JamesRezo deleted branch issue_5603 1 week ago

Reviewers

tcharlss approved these changes 1 week ago
JamesRezo approved these changes 1 week ago
b_b approved these changes 1 week ago
The pull request has been merged as d92d656aa6.
Sign in to join this conversation.
Loading…
There is no content yet.