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-14963] - Ne pas apporter automatiquement de modification sur la liste des contenants lorsque je procède à une modification transporteur et que le BSFF est au statut SENT #3615

Merged
merged 3 commits into from
Oct 10, 2024

Conversation

benoitguigal
Copy link
Member

@benoitguigal benoitguigal commented Sep 24, 2024

La mise à jour du BSFF pour ajouter un transporteur multi-modal provoquait un changement de l'ordre des contenants avec le message d'erreur : Des champs ont été verrouillés via signature et ne peuvent plus être modifiés : La liste des contenants a été vérouillé via signature et ne peut pas être modifié..

Cela venait du fait que la liste des contenants est renvoyée dans bsff { packagings } avec un tri { numero: 'asc' } mais que ce tri n'était pas fait au moment de la validation sur les champs scellés. Pour résoudre le problème, on ajoute la même option de tri lors de la récupération du BSFF persisté en base en amont de la validation.

  • 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)
  • S'assurer que la numérotation des nouvelles migrations est bien cohérente
  • Informer le data engineer de tout changement de schéma DB

Copy link
Collaborator

@GaelFerrand GaelFerrand left a comment

Choose a reason for hiding this comment

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

Carré

Copy link

@benoitguigal benoitguigal merged commit d00d747 into dev Oct 10, 2024
17 checks passed
@benoitguigal benoitguigal deleted the tra-14963 branch October 10, 2024 18:42
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