Disable some jdd actions on closed ca#4037
Disable some jdd actions on closed ca#4037christophe-ramet wants to merge 2 commits intodisallow_update_delete_on_closed_affrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## disallow_update_delete_on_closed_af #4037 +/- ##
=======================================================================
+ Coverage 86.47% 86.49% +0.01%
=======================================================================
Files 137 137
Lines 10876 10891 +15
=======================================================================
+ Hits 9405 9420 +15
Misses 1471 1471
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
christophe-ramet
force-pushed
the
disable_some_jdd_actions_on_closed_ca
branch
from
2107fb0 to
b59e552
Compare
Now check if AF is closed or not (so we don't let modify AF or active status). Little fix so we don't call twice the route for listing af.
christophe-ramet
changed the base branch from
develop
to
disallow_update_delete_on_closed_af
jacquesfize
left a comment
jacquesfize
left a comment
There was a problem hiding this comment.
J'ai bien une bad request quand j'essaie de modifier le statut d'un JDD dans un CA fermé.
Par contre, quand j'essaie de faire l'action j'ai un toaster avec une erreur pas compréhensible
et le spinner qui continue de tourner
Il faut remplacer le contenu du toaster avec un message explicite et enlever le spinner quand la modification échoue du statut d'un JDD échoue. Voir tout simplement, désactiver le bouton de changement de statut (en plus du bouton "Editer d'ailleurs)
2 participants
closes #3966
Il n'est plus possible de réactiver un JDD lié à un CA fermé, ni de modifier son CA d'origine si il est déjà fermé.