Project

General

Profile

Development #36167

Lever la limite de 600 jours lors de l'import des exceptions

Added by Marie Kuntz 30 days ago. Updated 29 days ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
16 Sep 2019
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

Description

Parce qu'on peut importer des jours fériés sur plusieurs années par exemple


Related issues

Related to Chrono - Development #28767: Import d'un ICS : prise en charge de la règle RRULE Nouveau 07 Dec 2018
Related to Chrono - Development #19033: Pagination dans la liste des exceptions Nouveau 27 Sep 2017

History

#1 Updated by Marie Kuntz 30 days ago

  • Related to Development #28767: Import d'un ICS : prise en charge de la règle RRULE added

#2 Updated by Frédéric Péters 30 days ago

Tu as une proposition de nouvelle limite ?

#3 Updated by Frédéric Péters 30 days ago

#4 Updated by Benjamin Dauvergne 30 days ago

J'aimerai que le lien soit fait avec #29209 et que donc l'approche choisie (puisqu'on ne gère pas explicitement de récurrence) soit de rejouer régulièrement une source, remplissant en permanence les 600 prochains jours, ce qui me parait large.

Je répète juste ce que je disais sur #28767.

#5 Updated by Marie Kuntz 29 days ago

Frédéric Péters a écrit :

Tu as une proposition de nouvelle limite ?

10 ans soit 3600 jours (je pense aux jours fériés, j'entends bien que ça ne concernera jamais les exceptions ponctuelles de type vacances)

#6 Updated by Marie Kuntz 29 days ago

Benjamin Dauvergne a écrit :

J'aimerai que le lien soit fait avec #29209 et que donc l'approche choisie (puisqu'on ne gère pas explicitement de récurrence) soit de rejouer régulièrement une source, remplissant en permanence les 600 prochains jours, ce qui me parait large.

Je répète juste ce que je disais sur #28767.

J'avoue que je ne comprends pas pourquoi on rajoute de la complexité (rejouer un fichier de manière récurrente pour importer les exceptions qui sont déjà toutes dans un fichier) alors qu'il "suffit" d'importer tous les événements.
De plus le fichier peut changer, si on le joue de manière récurrente, cela veut dire qu'on doit pouvoir le modifier en cours de route.
Pour moi, cet aspect là est couvert par la synchronisation d'un calendrier externe, pas par l'import d'un fichier.
Ou alors je n'ai rien compris.

#7 Updated by Benjamin Dauvergne 29 days ago

Marie Kuntz a écrit :

Benjamin Dauvergne a écrit :

J'aimerai que le lien soit fait avec #29209 et que donc l'approche choisie (puisqu'on ne gère pas explicitement de récurrence) soit de rejouer régulièrement une source, remplissant en permanence les 600 prochains jours, ce qui me parait large.

Je répète juste ce que je disais sur #28767.

J'avoue que je ne comprends pas pourquoi on rajoute de la complexité (rejouer un fichier de manière récurrente pour importer les exceptions qui sont déjà toutes dans un fichier) alors qu'il "suffit" d'importer tous les événements.
De plus le fichier peut changer, si on le joue de manière récurrente, cela veut dire qu'on doit pouvoir le modifier en cours de route.

Justement actuellement on ne peut pas le modifier, c'est l'objet du ticket que je pointe, tant que qu'on ne conserve pas un lien entre un fichier .ics et les exceptions qu'on crée avec on ne peut pas les supprimer/modifier.

Pour moi, cet aspect là est couvert par la synchronisation d'un calendrier externe, pas par l'import d'un fichier.
Ou alors je n'ai rien compris.

La limite de 600 jours avec un calendrier externe ayant des évènements récurrents est la même, je connais le code sur ces aspects de chrono vraiment bien.

Also available in: Atom PDF