Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TRA-15158] Possibilité de réviser la quantité refusée d'un BSD + correctifs #3961

Open
wants to merge 8 commits into
base: tra-15269
Choose a base branch
from

Conversation

GaelFerrand
Copy link
Collaborator

@GaelFerrand GaelFerrand commented Feb 10, 2025

Contexte

Implémentation front de la révision de la quantityRefused, avec en bonus un petit fix pour ne pas afficher la révision sur les quantités tant que le déchet n'a pas été reçu.

Points de vigilance pour les intégrateurs

Il est possible d'utiliser la révision pour ajouter une quantityRefused à un bordereau qui n'en avait pas. Vous pouvez donc utiliser la révision pour corriger des bordereaux legacy.

Démo

Screencast.from.2025-02-18.14-49-54.webm

Ticket Favro

Permettre de réviser la quantité refusée lors d'un refus partiel

@GaelFerrand GaelFerrand changed the base branch from dev to tra-15269 February 17, 2025 10:44
Copy link

@@ -41,19 +41,21 @@ export function BsddRequestRevisionCancelationInput({ bsdd, onChange }: Props) {
CANCELLABLE_BSDD_STATUSES.includes(bsdd.status) && !isAppendix1;

return (
<>
<div className="fr-mb-4v">
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai ajouté une petite marge au niveau du swtich d'annulation parce que c'était pas beau

image

@@ -33,7 +34,7 @@ const LabelContent = ({
defaultValue?: string | number | React.ReactNode;
suffix?: string;
}) =>
!!value ? (
isDefined(value) ? (
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Attention avec !!, ça marche pas avec zéro. Donc ça n'affichait pas un champ révisé qui passe de zéro à une autre valeur.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant