[admin] Ajout d'une page de confirmation après la publication d'une procédure - mise en avant de l'URL à partager #9672
Conversation
lisa-durand
force-pushed
the
add-confirmation-page-for-procedure-publication-ldu
branch
4 times, most recently
from
2c06e80 to
b0f63a9
Compare
lisa-durand
force-pushed
the
add-confirmation-page-for-procedure-publication-ldu
branch
from
b0f63a9 to
ac834b0
Compare
| flash.notice = "Nouvelle version de la démarche publiée" | ||
|
|
||
| redirect_to admin_procedure_path(@procedure) | ||
| end |
There was a problem hiding this comment.
je me demande s'il ne faudrait pas enlever le if @procedure.draft_changed?, d'une part car il me semble inutile, et d'autre part car même si la condition était fausse, ça provoquerait une erreur (car pas de template pour cette action). Ou alors sortir le redirect_to du if pour que quoiqu'il arrive on revienne à la page de la démarche.
Je me dis que ça peut arriver si t'as 2 onglets ouverts sur la même page avec le bouton Publier, tu click une première fois et ça publie, et tu cliques ensuite sur le second
| end | ||
| elsif @procedure.publish_or_reopen!(current_administrateur) | ||
| flash.notice = "Démarche publiée" | ||
| @procedure.publish_revision! |
There was a problem hiding this comment.
T'y es pour rien car ça vient du nommage des méthodes à cause du cas du reopen mais c'est un peu bizarre/pas intuitif d'avoir d'abord publish_or_reopen puis publish_revision si on a modifié le brouillon
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
lisa-durand
deleted the
add-confirmation-page-for-procedure-publication-ldu
branch
closes #9498
Lorsqu'une demarche est publiée et qu'on apporte des modifications on ne passe plus par la page de "publication" (la démarche est déja publiée), il y a par contre un message de confirmation pour plus de sécurité.
(Si on apporte des modifications à une procedure close - on continue de passer par la page de publication)