Skip to content

#patch (2621) Prévenir la perte de données lors de la modification des dates d'action#1433

Merged
superfeedboy merged 12 commits into
developfrom
issue/2621
Mar 25, 2026
Merged

#patch (2621) Prévenir la perte de données lors de la modification des dates d'action#1433
superfeedboy merged 12 commits into
developfrom
issue/2621

Conversation

@ch-benard

Copy link
Copy Markdown
Contributor

🧾 Ticket Trello

https://trello.com/c/OWKMerrd/2621

🛠 Description de la PR

Prévention de la perte de données lors de la modification des dates d'action

🎯 Contexte

Actuellement, lorsqu'un utilisateur modifie les dates de début ou de fin d'une action, les lignes de financement situées en dehors de la nouvelle plage de dates sont supprimées silencieusement sans avertissement. Cela peut entraîner une perte de données involontaire.

🔧 Solution implémentée

Ajout d'une modale de confirmation qui s'affiche avant la suppression des financements lorsque les dates de l'action sont modifiées.

Fonctionnalités
  1. Détection intelligente : La modale ne s'affiche que si des financements vont être effectivement supprimés
  2. Message contextuel : Le message s'adapte selon la date modifiée :
    • Date de début uniquement
    • Date de fin uniquement
    • Les deux dates
  3. Informations détaillées : Affichage des années concernées avec :
    • Nombre de lignes de financement
    • Montants totaux
  4. Actions utilisateur :
    • Confirmer : supprime les financements et soumet le formulaire
    • Annuler : annule la soumission, l'utilisateur peut modifier à nouveau
Composants modifiés
  • Nouveau : ModaleConfirmationSuppressionFinancements.vue - Composant modal de confirmation
  • Modifié : FormDeclarationAction.vue - Intégration de la vérification et affichage de la modale
  • Corrigé : InputActionFinances.vue - Suppression d'une ligne causant une erreur 400

🧪 Tests effectués

Scénarios testés

  • ✅ Modification de la date de début uniquement
  • ✅ Modification de la date de fin uniquement
  • ✅ Modification des deux dates simultanément
  • ✅ Modification de dates sans impact sur les financements (pas de modale)
  • ✅ Confirmation de suppression → financements supprimés, formulaire soumis
  • ✅ Annulation → retour au formulaire sans suppression

📸 Captures d'écran

{B7ACACFD-678E-48AF-9F71-F7F226E561B6}

🚨 Notes pour la mise en production

Liste des choses à faire pour le déploiement (changement de configuration, migrations à faire tourner, etc.), si possible avec les éléments les plus précis possibles (commande exacte à exécuter, etc.).

…ancement (dans le cas d'un changement de date de l'action)

refers to https://trello.com/c/OWKMerrd/2621
refers to https://trello.com/c/OWKMerrd/2621

Supprimer `value.value[strYear] = undefined` qui provoquait des erreurs "400 Bad Request" lors de la soumission du formulaire après modification de la date.

Seule la commande `delete value.value[strYear]` est nécessaire pour supprimer les années hors de la
plage valide.
@ch-benard ch-benard requested a review from superfeedboy March 5, 2026 12:54
@ch-benard ch-benard self-assigned this Mar 5, 2026
@tristanrobert

tristanrobert commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@superfeedboy superfeedboy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Quelques propositions d'amélioration et correction de linting

@sonarqubecloud

Copy link
Copy Markdown

@superfeedboy superfeedboy merged commit 86b9cb8 into develop Mar 25, 2026
7 checks passed
@superfeedboy superfeedboy deleted the issue/2621 branch March 25, 2026 13:34
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