https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342020-05-19T12:54:32ZRedmine Entr’ouvertChrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2361092020-05-19T12:54:32ZBenjamin Dauvergne
<ul></ul><p>On peut déjà séparer l'implémentation en 2/3 sans toucher aux URLs non ? J'étais déjà pour avant, mais comme pour l'autre j'aimerai bien ne pas avoir des tonnes de rebase à faire sur <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: perfs, structure/algos sur les intervalles (Fermé)" href="https://dev.entrouvert.org/issues/42169">#42169</a>. Si t'en es à vouloir réécrire fillslot je pense que la relecture de <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: perfs, structure/algos sur les intervalles (Fermé)" href="https://dev.entrouvert.org/issues/42169">#42169</a> t'es permise.</p> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2361112020-05-19T12:55:48ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>(elle a relu <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: perfs, structure/algos sur les intervalles (Fermé)" href="https://dev.entrouvert.org/issues/42169">#42169</a>)</p> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2361142020-05-19T13:06:42ZLauréline Guérin
<ul></ul><p>(j'ai relu <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: perfs, structure/algos sur les intervalles (Fermé)" href="https://dev.entrouvert.org/issues/42169">#42169</a> ;) )<br />Je peux attendre que <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: perfs, structure/algos sur les intervalles (Fermé)" href="https://dev.entrouvert.org/issues/42169">#42169</a> soit mergé, et gérer les conflits.<br />J'aimerais que <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: perfs, structure/algos sur les intervalles (Fermé)" href="https://dev.entrouvert.org/issues/42169">#42169</a> et <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: Permettre la configuration d'une date de publication par événement. (Fermé)" href="https://dev.entrouvert.org/issues/40728">#40728</a> passent pour continuer <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: Gestion des ressources (était : liste des créneaux disponibles communs à plusieurs agendas rendez... (Fermé)" href="https://dev.entrouvert.org/issues/38942">#38942</a>, en attendant je splitte)</p> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2367852020-05-22T09:02:39ZLauréline Guérin
<ul><li><strong>Fichier</strong> <a href="/attachments/44784">0005-api-remove-fillslots_url-from-meetings-agenda-detail.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44784/0005-api-remove-fillslots_url-from-meetings-agenda-detail.patch">0005-api-remove-fillslots_url-from-meetings-agenda-detail.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/44785">0004-api-split-fillslot-endpoints-43077.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44785/0004-api-split-fillslot-endpoints-43077.patch">0004-api-split-fillslot-endpoints-43077.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/44786">0003-api-move-serializers-43077.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44786/0003-api-move-serializers-43077.patch">0003-api-move-serializers-43077.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/44787">0002-api-use-APIError-in-fillslots-view-43077.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44787/0002-api-use-APIError-in-fillslots-view-43077.patch">0002-api-use-APIError-in-fillslots-view-43077.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/44788">0001-api-limit-accept-suspend-resize-endpoints-to-events-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44788/0001-api-limit-accept-suspend-resize-endpoints-to-events-.patch">0001-api-limit-accept-suspend-resize-endpoints-to-events-.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>J'ai essayé de réorganiser le code en limitant les spaghettis. J'aurais bien tenté de factoriser un peu plus la méthode <code>fillslots</code> mais j'ai peur que ça soit moins lisible.</p>
<p>Ca fait 4 urls: events/single, events/multiple, meetings/single, meetings/multiple, sachant que pour la réservation multiple d'après la doc c'est que pour les agendas de type events, mais que le code était très permissif.<br />J'ai conservé cette possibilité, avec l'idée d'analyser les logs dans quelques temps pour voir si c'est utilisé ou pas.</p> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2370632020-05-25T08:32:28ZLauréline Guérin
<ul><li><strong>Fichier</strong> <a href="/attachments/44821">0005-api-remove-fillslots_url-from-meetings-agenda-detail.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44821/0005-api-remove-fillslots_url-from-meetings-agenda-detail.patch">0005-api-remove-fillslots_url-from-meetings-agenda-detail.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/44822">0004-api-split-fillslot-endpoints-43077.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44822/0004-api-split-fillslot-endpoints-43077.patch">0004-api-split-fillslot-endpoints-43077.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/44823">0003-api-move-serializers-43077.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44823/0003-api-move-serializers-43077.patch">0003-api-move-serializers-43077.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/44824">0002-api-use-APIError-in-fillslots-view-43077.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44824/0002-api-use-APIError-in-fillslots-view-43077.patch">0002-api-use-APIError-in-fillslots-view-43077.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/44825">0001-api-limit-accept-suspend-resize-endpoints-to-events-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/44825/0001-api-limit-accept-suspend-resize-endpoints-to-events-.patch">0001-api-limit-accept-suspend-resize-endpoints-to-events-.patch</a> ajouté</li></ul> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2370642020-05-25T08:43:47ZLauréline Guérin
<ul></ul><p>(derniers patch: juste une modif dans le 0004 pour déplacer <code>get_places_count</code> dans <code>EventFillSlotsMixin</code>)</p> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2618062020-10-01T14:22:18ZLauréline Guérin
<ul><li><strong>Fichier</strong> <a href="/attachments/48304">0004-api-remove-fillslots_url-from-meetings-agenda-detail.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/48304/0004-api-remove-fillslots_url-from-meetings-agenda-detail.patch">0004-api-remove-fillslots_url-from-meetings-agenda-detail.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/48305">0003-api-split-fillslot-endpoints-43077.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/48305/0003-api-split-fillslot-endpoints-43077.patch">0003-api-split-fillslot-endpoints-43077.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/48306">0002-api-move-serializers-43077.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/48306/0002-api-move-serializers-43077.patch">0002-api-move-serializers-43077.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/48307">0001-api-use-APIError-in-fillslots-view-43077.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/48307/0001-api-use-APIError-in-fillslots-view-43077.patch">0001-api-use-APIError-in-fillslots-view-43077.patch</a> ajouté</li></ul><p>(rebase, gestion des merge conflicts)</p> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2618742020-10-01T16:31:51ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul><p>Un truc a bougé dans les tests.</p> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2618822020-10-01T17:32:06ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Au vu des problèmes récents sur le changement sur l'API /status (résultat <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: gérer les combinaisons id/slug dans l'API de status (Fermé)" href="https://dev.entrouvert.org/issues/46772">#46772</a>), je suis super frileux sur un changement d'URL.</p> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=2619502020-10-02T07:40:51ZLauréline Guérin
<ul><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Rejeté</i></li></ul><p>Allez, je ferme ce ticket; le code a tellement bougé depuis que c'est difficile à maintenir, et je n'étais pas forcément super satisfaite du découpage réalisé. + la frilosité de fred :)</p> Chrono - Development #43077: Split du endpoint fillslotshttps://dev.entrouvert.org/issues/43077?journal_id=4764182023-07-03T07:50:19ZLauréline Guérin
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed" href="/issues/79300">Development #79300</a>: api: splitter Fillslots</i> ajouté</li></ul>