Développement #56547
Lors de l'import d'un seul agenda, rediriger vers cet agenda
0%
Description
J'importe un seul agenda, et je dois ensuite le chercher dans la liste des agendas. J'aimerais plutôt être redirigé sur sa page (de paramétrage).
Files
Associated revisions
History
Updated by Lauréline Guérin about 3 years ago
- File 0002-manager-redirect-to-agenda-settings-after-import-565.patch 0002-manager-redirect-to-agenda-settings-after-import-565.patch added
- File 0001-misc-split-some-manager-tests.patch 0001-misc-split-some-manager-tests.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Proposition qui fait ce qui est demandé, mais je n'aime pas trop: on se retrouve sur la page de paramétrage de l'agenda avec un message "un agenda a été créé. aucun agenda n'a été mis à jour.", ça manque de pertinence.
Tant qu'à faire on pourrait aussi rediriger sur la page d'un calendrier d'indispo ou sur la page des raisons d'absence.
On pourrait aussi, au lieu de changer la redirection, changer les messages d'infos, et à la place d'avoir juste des compteurs, lister les agendas/calendriers d'indispo/groupes de raisons d'absence créés/modifiés, avec un lien pour aller sur la page de chaque objet.
Updated by Valentin Deniaud about 3 years ago
Lauréline Guerin a écrit :
Tant qu'à faire on pourrait aussi rediriger sur la page d'un calendrier d'indispo ou sur la page des raisons d'absence.
Oui c'est une bonne idée de généraliser dans ce sens.
changer les messages d'infos, avec un lien pour aller sur la page de chaque objet.
Mais quand on importe un site, on a pas envie d'avoir 100 messages avec 100 liens sur lesquels on ne va pas cliquer. Et quand on importe un objet, le lien devient utile, mais si on clique dessus à chaque fois, autant avoir une redirection ? :)
mais je n'aime pas trop: on se retrouve sur la page de paramétrage de l'agenda avec un message "un agenda a été créé. aucun agenda n'a été mis à jour.", ça manque de pertinence
Oui il faudrait court-circuiter le code qui affiche ces messages, en détectant en amont qu'il n'y a qu'un objet, et avoir un traitement spécifique pour ce cas.
Updated by Lauréline Guérin about 3 years ago
- Status changed from Solution proposée to En cours
allez hop
Oui c'est une bonne idée de généraliser dans ce sens.
Je vais faire ça
autant avoir une redirection
yep
Oui il faudrait court-circuiter le code qui affiche ces messages, en détectant en amont qu'il n'y a qu'un objet, et avoir un traitement spécifique pour ce cas.
zou
Updated by Lauréline Guérin about 3 years ago
- File 0002-manager-redirect-to-agenda-settings-after-import-565.patch 0002-manager-redirect-to-agenda-settings-after-import-565.patch added
- File 0001-misc-split-some-manager-tests.patch 0001-misc-split-some-manager-tests.patch added
- Status changed from En cours to Solution proposée
Updated by Valentin Deniaud about 3 years ago
- Status changed from Solution proposée to Solution validée
Updated by Lauréline Guérin about 3 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit c15db686bdac5c612d91a85e9d26ffa594d11eae Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Sep 3 11:48:17 2021 +0200 manager: redirect to agenda settings after import (#56547) commit 65f1501da534fccd918327fe1017748d12d86153 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Sep 3 10:16:50 2021 +0200 misc: split some manager tests
Updated by Frédéric Péters about 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
manager: redirect to agenda settings after import (#56547)