toulouse-maelis: [tests] rafraîchir les tests et les trames de tests sur les factures et les paniers (#76836) #225

nroche · 2023-04-20T17:25:57+02:00

nroche commented

2023-04-20 17:25:57 +02:00

Corriger les numéros des régies sur les tests des factures
et rafraîchir les trames SOAP des paniers avec les trames obtenues dernièrement.

Corriger les numéros des régies sur les tests des factures et rafraîchir les trames SOAP des paniers avec les trames obtenues dernièrement.

fpeters commented

2023-04-20 18:20:40 +02:00

Ça peut passer sans "toulouse-maelis: correct invoice log diff order (#76828)", ou cette PR en dépend ?

fpeters changed title from ~~tououse-maelis: [tests] rafraîchir les tests et les trames de tests sur les factures et les paniers~~ to toulouse-maelis: [tests] rafraîchir les tests et les trames de tests sur les factures et les paniers

2023-04-20 18:21:24 +02:00

nroche commented

2023-04-20 21:41:15 +02:00

Ça peut passer sans "toulouse-maelis: correct invoice log diff order (#76828)", ou cette PR en dépend ?

Ça peut passer sans, si je ré-introduis l'erreur corrigée dans les patchs suivants.
Dis-moi si tu veux que je fasses ça.

> Ça peut passer sans "toulouse-maelis: correct invoice log diff order (#76828)", ou cette PR en dépend ? Ça peut passer sans, si je ré-introduis l'erreur corrigée dans les patchs suivants. Dis-moi si tu veux que je fasses ça.

fpeters commented

2023-04-21 08:24:36 +02:00

Ça peut passer sans "toulouse-maelis: correct invoice log diff order (#76828)", ou cette PR en dépend ?

Ça peut passer sans, si je ré-introduis l'erreur corrigée dans les patchs suivants.
Dis-moi si tu veux que je fasses ça.

Non le propos, toujours, c'est minimiser le boulot : pour y arriver il ne faut certainement pas reprendre modifier des choses, il faut juste, quand tu ajoutes la PR, écrire "ok il y a ici un commit sans rapport c'est telle autre PR qu'il faut d'abord relire".

>> Ça peut passer sans "toulouse-maelis: correct invoice log diff order (#76828)", ou cette PR en dépend ? > Ça peut passer sans, si je ré-introduis l'erreur corrigée dans les patchs suivants. Dis-moi si tu veux que je fasses ça. Non le propos, toujours, c'est minimiser le boulot : pour y arriver il ne faut certainement pas reprendre modifier des choses, il faut juste, quand tu ajoutes la PR, écrire "ok il y a ici un commit sans rapport c'est telle autre PR qu'il faut d'abord relire".

bdauvergne approved these changes 2023-04-21 12:25:02 +02:00

bdauvergne left a comment

Le truc du diff est passé sur une autre PR il me semble.

nroche changed title from ~~toulouse-maelis: [tests] rafraîchir les tests et les trames de tests sur les factures et les paniers~~ to toulouse-maelis: [tests] rafraîchir les tests et les trames de tests sur les factures et les paniers (#76836)

2023-04-21 13:00:55 +02:00

nroche force-pushed wip/76836-parsifal-update-facturation-tests from 1d35827094 to 9535c6c68f

2023-04-21 13:02:16 +02:00

Compare

nroche merged commit 9535c6c68f into main

2023-04-21 13:51:39 +02:00

nroche deleted branch wip/76836-parsifal-update-facturation-tests

2023-04-21 13:51:39 +02:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

3 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#225