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 15626 fix] Ne pas supprimer l'intermédiaire inopinément #3896

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

silto
Copy link
Collaborator

@silto silto commented Jan 10, 2025

Contexte

Dans ma PR précédente pour ce ticket, j'avais fait un fix qui permettait de supprimer un intermédiaire en décochant le toggle correspondant, et de ne pas envoyer d'intermédiaire vide (ne pas envoyer l'entreprise vide qui sert d'initialisation au formulaire). Le fix consistait simplement à ne pas envoyer d'intermédiaire si le toggle "présence d'intermédiaire" était décochée. Le problème est que l'état du toggle n'est initialisé que lors de la visite de l'onglet "autres acteurs", donc lors d'une modification du bordereau sans passer par et onglet le toggle restait à false, et l'intermédiaire était supprimé.
Afin de régler le problème tout en gardant le comportement souhaité, j'ai plutôt utilisé le contenu de l'array d'intermédiaire, pour détecter qu'elle ne contient que l'entreprise vide par défaut, et seulement dans ce cas renvoyer une array d'intermédiaires vide.

Points de vigilance pour les intégrateurs

aucun

Démo

pas de changement de comportement

Ticket Favro

TRA 15626 - Permettre d'ajouter un intermédiaire sur le VHU jusqu'au traitement du bordereau

Checklist

  • Mettre à jour la documentation
  • Mettre à jour le change log
  • Documenter les manipulations à faire lors de la mise en production (sur le ticket Favro de release)
  • Informer le data engineer de tout changement de schéma DB

@silto silto merged commit be6099d into dev Jan 13, 2025
21 checks passed
@silto silto deleted the tra-15626-fix branch January 13, 2025 15:38
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.

3 participants