https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342021-02-23T11:08:49ZRedmine Entr’ouvertChrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=2893542021-02-23T11:08:49ZValentin Deniaud
<ul></ul><p>Pour le moment c'est normal, quand on a pas de borne temporelle max on ne montre pas les évènements récurrents.</p>
<p>Si il y a une date de réservation max, pas de problèmes on l'utilise automatiquement. Mais en son absence il faut spécifier date_end à l'API /datetimes/ (ça devra être précisé le jour où il y aura de la doc).</p>
<p>Tu aurais attendu quel comportement ?</p>
<p>(par contre actuellement le comportement est incohérent si on est dans la situation où certaines récurrences sont en base et pas d'autres, /datetimes/ va montrer une vue partielle bizarre, je peux patcher ça ici)</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=2893732021-02-23T12:50:21ZLauréline Guérin
<ul></ul><p>ce que j'observe:<br />dans un agenda avec un Délai de réservation maximal > 0,<br />- un event récurrent sans date de fin affiche dans la vue mensuelle des events réservables et des events en dehors de la période d'inscription<br />- les récurrences dans la période de réservation de cet event remontent dans l'API datetimes</p>
<p>dans un agenda avec un Délai de réservation maximal == 0,<br />- un event récurrent sans date de fin n'affiche dans la vue mensuelle que des events réservables<br />- aucune récurrence ne remonte dans l'API datetimes</p>
<p>Il y a une incohérence entre l'UI et l'API, mais si je comprends bien c'est le comportement souhaité ?</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=2893822021-02-23T13:12:32ZValentin Deniaud
<ul></ul><p>Lauréline Guerin a écrit :</p>
<blockquote>
<p>c'est le comportement souhaité ?</p>
</blockquote>
<p>Ça m'a paru le seul comportement possible, puisqu'on ne peut pas retourner un nombre infini d'évènements dans du JSON, et mettre une coupure arbitraire genre 1 an ne m'a pas paru censé.</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=2893832021-02-23T13:15:03ZLauréline Guérin
<ul></ul><p>Du coup on en revient presque à l'idée d'imposer une date de fin de récurrence :)<br />Sachant que les activités récurrentes sont généralement mises en place sur une année scolaire</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=2893882021-02-23T13:25:23ZValentin Deniaud
<ul></ul><p>Je ne vois pas quel est le problème derrière ce ticket, il y a quel cas d'usage ?</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=2893932021-02-23T13:46:40ZLauréline Guérin
<ul></ul><p>Le problème:</p>
<p>On a un événement récurrent. En fonction de plusieurs paramètres (présence ou non d'un délai max de réservation, d'une date de réservation, ou d'un filtre date_end), l'API datetimes peut ne pas renvoyer de slots.</p>
<p>Ca me paraît bancal.</p>
<p>Dans un ticket d'optimisation de l'API datetimes Benjamin proposait de limiter la période couverte (et faire évoluer w.c.s. pour qu'il ne récupère que les slots d'une période donnée), parce que renvoyer la terre entière ce n'est pas forcément pertinent - je ne sais plus si c'était dans un commentaire, par email ou sur jabber; peut-être qu'il faudrait regarder de ce côté-là pour avoir un comportement stable ?</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=2893992021-02-23T14:08:11ZValentin Deniaud
<ul></ul><p>Dac mais comme le cas où on ne renvoie rien est bien précis, agenda sans délai max de réservation + évènement récurrent sans date de fin de récurrence, j'imaginais possible de le documenter : « dans ce cas, précisez un paramètre date_end sinon l'API ne sait pas quoi renvoyer » (on pourrait ajouter « et vous non plus », d'où ma question sur les cas d'usages, je n'en vois pas).</p>
<p>Mais oui je comprends maintenant que ce comportement c'est un peu ignorer silencieusement une erreur au lieu de la signaler : pour faire simple avant d'en venir à l'idée de Benj, on pourrait lever une APIError si on détecte qu'on est dans ce cas, en demandant explicitement un date_end dans le message d'erreur ?</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=2894052021-02-23T14:16:24ZLauréline Guérin
<ul></ul><p>Comment tu gèrerais le cas où dans un même agenda tu as des récurrents avec date de fin, des events non récurrents, et des récurrences sans limite, et pas de end_date passé en filtre ?</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=2894212021-02-23T14:52:39ZValentin Deniaud
<ul></ul><p>Oui on voit venir le cas où on a un agenda en prod, pas de délai de réservation max et des évènements avec fin de récurrence, et où on a pas envie qu'ajouter un évènement sans fin de récurrence fasse d'un coup planter tous les appels à /datetimes/. Donc mauvaise idée l'APIError.</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=3700642022-04-04T14:20:32ZValentin Deniaud
<ul><li><strong>Assigné à</strong> mis à <i>Valentin Deniaud</i></li></ul> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=3700652022-04-04T14:22:31ZValentin Deniaud
<ul><li><strong>Fichier</strong> <a href="/attachments/64023">0001-tests-ensure-datetimes-works-with-infinite-recurring.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/64023/0001-tests-ensure-datetimes-works-with-infinite-recurring.patch">0001-tests-ensure-datetimes-works-with-infinite-recurring.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>Ce n'est plus un problème maintenant, ça m'irait de fermer sans même merger ce test.</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=3700672022-04-04T14:24:21ZLauréline Guérin
<ul></ul><p>yes tu peux fermer</p> Chrono - Development #51360: Evénements récurrents - délais de réservationhttps://dev.entrouvert.org/issues/51360?journal_id=3700732022-04-04T14:30:41ZValentin Deniaud
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Fermé</i></li></ul>