Project

General

Profile

Development #36167

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

Added by Marie Kuntz 8 months ago. Updated 16 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 Solution déployée 27 Sep 2017
Related to Chrono - Development #29209: matérialiser les sources d'exceptions Solution déployée 19 Dec 2018

History

#1 Updated by Marie Kuntz 8 months 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 8 months ago

Tu as une proposition de nouvelle limite ?

#3 Updated by Frédéric Péters 8 months ago

#4 Updated by Benjamin Dauvergne 8 months 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 8 months 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 8 months 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 8 months 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.

#9 Updated by Marie Kuntz 7 months ago

  • Target version set to Wishlist

#10 Updated by Marie Kuntz 7 months ago

  • Priority changed from Normal to Bas

#11 Updated by Frédéric Péters 6 months ago

#12 Updated by Marie Kuntz 16 days ago

  • Priority changed from Bas to Normal

#13 Updated by Marie Kuntz 16 days ago

  • Target version deleted (Wishlist)

Also available in: Atom PDF