[TRA-15269] La quantité refusée (champ quantityRefused) est désormais obligatoire sur les BSDD #3823
+1,768
−1,665
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
GaelFerrand
commented
Feb 14, 2025
| @@ -352,16 +352,11 @@ describe("Mutation.createFormRevisionRequest", () => { | |||
| } | |||
| } | |||
| }); | |||
| expect(errors).toEqual([ | |||
There was a problem hiding this comment.
J'ai pas compris pourquoi ça bugguait, j'ai dû réécrire le expect
GaelFerrand
requested review from
providenz,
benoitguigal and
Riron
and removed request for
providenz and
benoitguigal
Riron
approved these changes
Feb 17, 2025
|
providenz
approved these changes
Feb 18, 2025
GaelFerrand
commented
Feb 18, 2025
| @@ -1,215 +0,0 @@ | |||
| import React from "react"; | |||
There was a problem hiding this comment.
Je crois que ces fichiers ne sont plus utilisés. C'est ok de les supprimer @kornifex ?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contexte
Il y a quelques mois, j'avais introduit la
quantityRefused, mais elle était facultative.Maintenant elle est obligatoire. C'est-à-dire que pour toutes les mutations d'acceptation, le paramètre est obligatoire en entrée GQL, et pour les mutations de réception + acceptation,
quantityRefusedest obligatoire à partir du moment oùwasteAcceptationStatusest présent.A noter également, cette PR revert un changement introduit il y a 6 mois. Je n'ai plus le contexte mais à l'époque j'avais déjà préparé les révisions, pour accepter la
quantityRefused, et j'avais aussi - je ne sais plus pourquoi - autorisé la révision duwasteAcceptationStatus.Bien que ce soit dans nos plans à moyen terme, mon implémentation était un peu simpliste et mérite plus mûre réflexion (notamment les changements majeurs du genre
ACCEPTED>REFUSEDont des effets de bord pas gérés). Du coup j'ai retiré la possibilité de réviser lewasteAcceptationStatus. Techniquement c'est un breaking change, mais les révisions sont peu utilisées côté API et nous n'avions jamais annoncé cette feature.J'ai aussi fait un gros travail sur les révisions, pour permettre de faire une révision:
quantityRefused(pour modifier un autre champ, ou préciser la quantityRefused)quantityRefusedPoints de vigilance pour les intégrateurs
💥 Breaking change: la quantityRefused est désormais obligatoire sur le BSDD! 💥
Ticket Favro
Rendre obligatoire la quantitéRefusée lorsque le statut de l'acceptation est Refusé partiellement