Development #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).
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Fichier 0002-manager-redirect-to-agenda-settings-after-import-565.patch 0002-manager-redirect-to-agenda-settings-after-import-565.patch ajouté
- Fichier 0001-misc-split-some-manager-tests.patch 0001-misc-split-some-manager-tests.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
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.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution proposée à 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
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Fichier 0002-manager-redirect-to-agenda-settings-after-import-565.patch 0002-manager-redirect-to-agenda-settings-after-import-565.patch ajouté
- Fichier 0001-misc-split-some-manager-tests.patch 0001-misc-split-some-manager-tests.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution validée à 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
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: redirect to agenda settings after import (#56547)