Project

General

Profile

Development #36167

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

Added by Marie Kuntz about 1 year ago. Updated 23 days ago.

Status:
Rejeté
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 Solution déployée 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 about 1 year 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 about 1 year ago

Tu as une proposition de nouvelle limite ?

#3 Updated by Frédéric Péters about 1 year ago

#4 Updated by Benjamin Dauvergne about 1 year 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 about 1 year 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 about 1 year 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 about 1 year 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 about 1 year ago

  • Target version set to Wishlist

#10 Updated by Marie Kuntz about 1 year ago

  • Priority changed from Normal to Bas

#11 Updated by Frédéric Péters about 1 year ago

#12 Updated by Marie Kuntz 7 months ago

  • Priority changed from Bas to Normal

#13 Updated by Marie Kuntz 7 months ago

  • Target version deleted (Wishlist)

#14 Updated by Valentin Deniaud 23 days ago

  • Status changed from Nouveau to Rejeté

Les tickets liés ont été traités, les jours fériés ne doivent plus être importés via ICS, pas besoin de toucher à cette limite donc.

Also available in: Atom PDF