toulouse-maelis: retirer le paramètre flagCom des payloads (#77547) #247
smihai
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/77547-remove-flagCom-from-payloads"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Ne pas planter parce que Maélis n'attend plus le paramètre flagCom en entrée.
À nouveau désolé pour ce patch de mauvaise qualité (fourre tout).
(A noter que seul le dernier patch passe les tests.)
(à l'heure actuelle je ne peux toujours plus m'inscrire pour tester et compléter les tests sans "tricher")
Sachant cela, pour éviter cela, pourquoi pas des tickets/PR séparées ?
L'objet de ce patch (flagCom) est de permettre à Sigec une montée de version sur INTEG, sans que le connecteur ne plante. Cependant c'est lié à la mise à jour des WSDL.
J'ai groupé les corrections liées aux WSDL dans cette unique PR pour ne pas avoir à les découper (ce qui fait par ailleurs que seul le dernier patch passe les tests).
J'ai fait cela en conscience en sachant que Sigec envoie beaucoup de petits correctifs éparses, que j'ai du mal à suivre, tout en me concentrant sur le suivi général (les tests fonctionnels) ainsi que les derniers développements sur la facturation panier.
J'essaie de comprendre : là on va avoir côté Sigec une version "INTEG" avec des wsdl et une version "TEST" avec une autre version des wsdl ?
Et les deux vont continuer à fonctionner avec le même connecteur sauf qu'en fait pas les tests mais non là je suis perdu.
Oui.
Maélis DEV c'est l'instance hébergé chez Sigec. C'est Sigec qui décide des montée des versions (au fils de leur développements).
Maélis INTEG c'est l'instance hébergé à Toulouse sur laquelle se font les recettes actuellement. Ici c'est nous qui donnons notre aval pour la montée de version (quand les tests de non-régressions passent sur DEV).
Ok et ici j'étais perdu, on a quoi qui va fonctionner où, après ces patchs ?
Une fois ces patchs passés, on pourra marquer comme résolu https://redmine.sigec.fr/issues/2069
(les autres points [tests] sont sans conséquences sur le connecteur)
cf3f2c2a21toadc4fd4939adc4fd4939to0b81087341Mais je ne comprends alors pas ce qu'ils faisaient dans cette PR.
J'ai attrapé les WSDL en entier afin de m'éviter le travaille de découper artificiellement leur mises à jour dans plusieurs PR.
Ces 2 autres patchs font que les tests continuent de fonctionner avec les nouveaux WSDL.
(D'habitude je fais une première PR avec les nouveaux WSDL où je modifie ou retire des tests pour qu'ils fonctionnent. Ici j'ai voulu m'éviter ça, mais ce n'était pas une bonne idée.)