toulouse-maelis: retirer le cache sur le catalogue général (#78141) #271

Merged

nroche merged 2 commits from wip/78141-parsifal-remove-main-catalog-cache into main 2023-06-23 16:19:10 +02:00

Conversation 2 Commits 2 Files Changed 4 +860 -103

nroche commented 2023-06-02 18:08:17 +02:00

Owner

Copy Link

On veut plus du tout de cache sur le catalogue des activités en fait.

• il y a premier commit pour simplifier un test qui utilisait le cache
• il y a des conversions sur les dates parce que comme on utilisait le catalogue depuis le cache, on avait des chaîne à la place des dates.

On veut plus du tout de cache sur le catalogue des activités en fait. * il y a premier commit pour simplifier un test qui utilisait le cache * il y a des conversions sur les dates parce que comme on utilisait le catalogue depuis le cache, on avait des chaîne à la place des dates.

nroche force-pushed wip/78141-parsifal-remove-main-catalog-cache from 5851194350 to 9c97747643 2023-06-05 10:02:34 +02:00 Compare

nroche force-pushed wip/78141-parsifal-remove-main-catalog-cache from 9c97747643 to 2013586889 2023-06-09 17:55:06 +02:00 Compare

fpeters commented 2023-06-15 09:39:51 +02:00

Owner

Copy Link

On veut plus du tout de cache sur le catalogue des activités en fait.

Je lis le code et il était uniquement employé en cas d'indispo de service, qu'est-ce qui posait problème avec ça ?

> On veut plus du tout de cache sur le catalogue des activités en fait. Je lis le code et il était uniquement employé en cas d'indispo de service, qu'est-ce qui posait problème avec ça ?

nroche commented 2023-06-19 09:38:19 +02:00

Author

Owner

Copy Link

Je lis le code et il était uniquement employé en cas d'indispo de service, qu'est-ce qui posait problème avec ça ?

Désolé, je n'ai pas repris le pourquoi dans la PR.
Sur le ticket :

Laisser au portail la possibilité de modifier l'intervalle des dates à requêter sur le catalogue général.
cf https://redmine.sigec.fr/issues/2506
Dans ces conditions, la mise en cache du catalogue n'est plus possible.

On met en cache le catalogue d'une année.
On ne sait pas (on n'a pas envie de) construire un catalogue d'une durée quelconque à partir des catalogues d'une seule année, que se soit de les concaténer ou de les diviser.

> Je lis le code et il était uniquement employé en cas d'indispo de service, qu'est-ce qui posait problème avec ça ? Désolé, je n'ai pas repris le pourquoi dans la PR. Sur le ticket : > Laisser au portail la possibilité de modifier l'intervalle des dates à requêter sur le catalogue général. > cf https://redmine.sigec.fr/issues/2506 > Dans ces conditions, la mise en cache du catalogue n'est plus possible. On met en cache le catalogue d'une année. On ne sait pas (on n'a pas envie de) construire un catalogue d'une durée quelconque à partir des catalogues d'une seule année, que se soit de les concaténer ou de les diviser.

fpeters approved these changes 2023-06-20 10:07:32 +02:00

fpeters left a comment

Owner

Copy Link

(sur le fond je pense qu'on aurait besoin de cache pour des raisons de disponibilité et perfs mais ça pourra revenir plus tard).

(sur le fond je pense qu'on aurait besoin de cache pour des raisons de disponibilité et perfs mais ça pourra revenir plus tard).

nroche added 2 commits 2023-06-23 16:01:42 +02:00

ad752230d6 toulouse-maelis: [tests] simplify test on activities without nature (#78141)

5e94eb86a3 toulouse-maelis: remove cache on activity catalog (#78141)

nroche merged commit 5e94eb86a3 into main 2023-06-23 16:19:10 +02:00

nroche deleted branch wip/78141-parsifal-remove-main-catalog-cache 2023-06-23 16:19:10 +02:00

Sign in to join this conversation.

Reviewers

No reviewers

fpeters

Labels

Clear labels

No items

No Label

Milestone

Clear milestone

No items

No Milestone

Assignees

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#271

No description provided.