Us/poc multiple - 2 dernières actions de l'onglet suivi #8433
Conversation
| @@ -13,9 +13,9 @@ def render? | |||
| def operations_for_dossier(dossier) | |||
There was a problem hiding this comment.
@tchak petit doute sur le principe de se baser sur l'etat des dossiers car ça ne fonctionne pas vraiment pour les actions follow / unfollow il me semble
There was a problem hiding this comment.
C'est pour ça qu'on passe tout le dossier dans cette méthode. Pour certains cas il faudra être plus malin que juste l'état.
There was a problem hiding this comment.
ou alors toujours autoriser ces actions pour tous les états (comme ce que tu fais là) et juste rendre l'action idempotente pour follow/unfollow. Je pense que ça serait une bonne option.
There was a problem hiding this comment.
j'ai fait ça 6f93713, je ne sais pas si c'est ce que tu avais en tête ? :)
There was a problem hiding this comment.
Je me suis permis de merger - je te laisse proposer une PR de refacto si au final ça ne te convient pas.
mfo
force-pushed
the
US/poc-multiple-last-2-actions-page-suivis
branch
from
e9e6bfc
to
905fc24
Compare
…peration. sinon c'est un autre instructeur qui est crée
lisa-durand
force-pushed
the
US/poc-multiple-last-2-actions-page-suivis
branch
from
905fc24
to
6f93713
Compare
Mise en place des 2 dernières actions de l'onglet "suivi" : "repasser en construction" et "ne plus suivre"
Voir #8076