Development #46555
Permettre de poser une exception qui s'applique sur plusieurs agenda
0%
Description
Des agents qui ont plusieurs agendas perso voudraient pouvoir "poser" leur vacances sans aller ajouter une exception dans chacun des agendas.
J'imagine ça pratique aussi pour une situation du type fermeture exceptionnelle de la mairie tel jour.
Dans le "plusieurs agendas", si ça pouvait prendre indifféremment des agendas réels et virtuels, ça résoudrait #42268.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Sujet changé de Permettre de définir poser une exception qui s'applique sur plusieurs agenda à Permettre de poser une exception qui s'applique sur plusieurs agenda
Mis à jour par Frédéric Péters il y a plus de 3 ans
Récemment je suggérais que les catégories pourraient porter des exceptions, voyant celles-ci comme le moyen déjà présent de "grouper" des agendas.
Une autre approche serait celle de #18904 qui a donné les jours fériés, avoir un espace où se définirait "calendrier des congés de Fred" (qui ne serait pas les settings), et il y aurait alors au niveau des agendas à activer/désactiver ces congés.
Je pense nécessaire de prendre un peu de recul et faire un plan un peu précis, qui assure au final quelque chose de compréhensible.
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Emmanuel Cazenave
On va partir sur un nouveau type d'objet, les "calendriers d'indisponibilité"
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Fichier 0001-start-unavailability-calendars-46555.patch 0001-start-unavailability-calendars-46555.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Coté API j'ai choisi l'implémentation naïve qui consiste à faire semblant que les exceptions sont posés sur tous les guichets, c'est probablement l'option la plus minimaliste en terme de code.
Coté manager j'ai choisi de la même organisation que pour les agendas, avec une vue de détail et une vue settings. La vue détail n'apporte grand chose pour l'instant mais c'est dans l'idée que plus tard ça pourra avoir être le lieu d'un affichage calendaire (genre mensuelle/annuelle) des exceptions.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier global-exceptions.png global-exceptions.png ajouté
Je ne sais pas si ça a vraiment du sens d'afficher l'identifiant à côté de "Paramétrage", on l'affiche sur les agendas parce que c'est exploité par les API.
J'ai cliqué sur "add unavailabilité" ça m'a fait
File "/home/fred/src/eo/venv3-2.2/lib/python3.8/site-packages/django/urls/resolvers.py", line 673, in _reverse_with_prefix
raise NoReverseMatch(msg)
django.urls.exceptions.NoReverseMatch: Reverse for 'chrono-manager-agenda-view' with keyword arguments '{'pk': ''}' not found. 1 pattern(s) tried: ['manage/agendas/(?P<pk>\\d+)/$']
Côté agenda pour moi ça ne doit pas en terme d'interface fonctionner ainsi; ça doit fonctionner comme les jours fériés, cf capture. C'est ce que j'écrivais plus haut : "et il y aurait alors au niveau des agendas à activer/désactiver ces congés.".
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
Frédéric Péters a écrit :
Je ne sais pas si ça a vraiment du sens d'afficher l'identifiant à côté de "Paramétrage", on l'affiche sur les agendas parce que c'est exploité par les API.
Viré.
J'ai cliqué sur "add unavailabilité" ça m'a fait
Branche pas à jour chez toi ou autre subtilité, en tous cas pas de ça chez moi.
Côté agenda pour moi ça ne doit pas en terme d'interface fonctionner ainsi; ça doit fonctionner comme les jours fériés, cf capture.
C'est à prendre au pied de la lettre, ie il faudra aller activer guichet par guichet les calendriers d'indispo ?
Mis à jour par Frédéric Péters il y a plus de 3 ans
Branche pas à jour chez toi ou autre subtilité, en tous cas pas de ça chez moi.
58aa1c55d7db47b4d56fbe020093602aae6f3a46 qui est celle poussée à Date: Wed, 30 Sep 2020 14:46:57 +0200.
Mais capté il y a dans mon environnement hobo une variable qui s'appelle agenda (venant d'une vieille intégration graphique Vincennes...), désolé.
C'est à prendre au pied de la lettre, ie il faudra aller activer guichet par guichet les calendriers d'indispo ?
Oui.
~~
Passant par templates/chrono/manager_agenda_view.html, le titre de la page /manage/unavailability-calendar/1/settings se trouve avec rien derrière le tiret dans son <title> :
{% block page-title-extra-label %}
- {% firstof agenda.label object.label %}
{% endblock %}
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Lié à Development #47393: Calendrier d'indisponibilités : afficher un message lorsqu'une indisponibilité chevauche une réservation ajouté
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Lié à Development #47394: Calendrier d'indisponibilités : gérer l'import/export ajouté
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Lié à Development #47395: Calendrier d'indisponibilités : gérer l'affichage des indisponbilité dans les vues de listing des exceptions ajouté
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Fichier 0001-start-unavailability-calendars-46555.patch 0001-start-unavailability-calendars-46555.patch ajouté
Frédéric Péters a écrit :
Côté agenda pour moi ça ne doit pas en terme d'interface fonctionner ainsi [....]
Voilà.
Passant par templates/chrono/manager_agenda_view.html, le titre de la page /manage/unavailability-calendar/1/settings se trouve avec rien derrière le tiret dans son <title> :
Corrigé.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
On ne liste pas les exceptions du calendrier d'indisponibilité dans la liste des exceptions d'un guichet ?
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
Tu veux dire ça #47395 ? Ou directement sur la vue de settings de l'agenda ?
Mis à jour par Lauréline Guérin il y a plus de 3 ans
non c'est bien ça, #47395 :)
moi ça me semble bon, je passe le ticket en validé ou on attend un coup de tampon de Fred ?
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
Mais valide donc ! On pourra faire des ajustements après.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 03bb2db903fc21a327a5336b9f4dbecdff5b5be4
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Wed Sep 30 17:53:42 2020 +0200
start unavailability calendars (#46555)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution validée
(je ne sais pas où tu l'as poussé ?)
Mais avant de le pousser, tu peux adapter le message de commit ? (genre "add support for unavailability calendars", plutôt que "start").
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
J'ai fait cette modif et envoyé dans master.
commit 7320dcfbe645fd9c120ac415050d05cf28dc2d96
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Wed Sep 30 17:53:42 2020 +0200
add support for unavailability calendars (#46555)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
add support for unavailability calendars (#46555)