manager: do not show weekday indexes field for exclusion periods (#72625) #26

vdeniaud · 2022-12-20T15:29:40+01:00

vdeniaud commented

2022-12-20 15:29:40 +01:00

L'implem initiale (qui a quand même 9 mois, #45159) a fait apparaître les champs pour choisir la semaine d'une période d'ouverture également côté période d'exclusion d'un agenda virtuel.

Bien sûr il y a 0 test là dessus et ça n'a jamais marché (comportement aléatoire).

Avec le changement d'approche dans #72491, au lieu du comportement aléatoire, on a cette trace.

Voilà pour masquer les champs, j'aurais pu mettre une migration mais j'aime autant aller corriger à la main (à mon avis ça concerne très très peu de tenant, potentiellement 0 en prod, je vais faire un script pour être couvert sur le saas et si on reçoit des traces d'ailleurs ça sera facile d'aller corriger à la main).

L'implem initiale (qui a quand même 9 mois, #45159) a fait apparaître les champs pour choisir la semaine d'une période d'ouverture également côté période d'exclusion d'un agenda virtuel. Bien sûr il y a 0 test là dessus et ça n'a jamais marché (comportement aléatoire). Avec le changement d'approche dans #72491, au lieu du comportement aléatoire, on a cette trace. Voilà pour masquer les champs, j'aurais pu mettre une migration mais j'aime autant aller corriger à la main (à mon avis ça concerne très très peu de tenant, potentiellement 0 en prod, je vais faire un script pour être couvert sur le saas et si on reçoit des traces d'ailleurs ça sera facile d'aller corriger à la main).

vdeniaud added 1 commit 2022-12-20 15:29:41 +01:00

gitea-wip/chrono/pipeline/pr-main This commit looks good Details

4a6780c467 manager: do not show weekday indexes field for exclusion periods (#72625 )

fpeters reviewed 2022-12-20 15:35:46 +01:00

fpeters left a comment

Ok, mais on ne prévoit rien pour réparer les configurations existantes ?

fpeters commented

2022-12-20 15:37:38 +01:00

C'est écrit plus haut (mais ça n'est pa sur l'écran qui affiche le diff et qui permet de faire la relecture) :

Voilà pour masquer les champs, j'aurais pu mettre une migration mais j'aime autant aller corriger à la main (à mon avis ça concerne très très peu de tenant, potentiellement 0 en prod, je vais faire un script pour être couvert sur le saas et si on reçoit des traces d'ailleurs ça sera facile d'aller corriger à la main).

C'est écrit plus haut (mais ça n'est pa sur l'écran qui affiche le diff et qui permet de faire la relecture) : > Voilà pour masquer les champs, j'aurais pu mettre une migration mais j'aime autant aller corriger à la main (à mon avis ça concerne très très peu de tenant, potentiellement 0 en prod, je vais faire un script pour être couvert sur le saas et si on reçoit des traces d'ailleurs ça sera facile d'aller corriger à la main).

fpeters approved these changes 2022-12-20 15:37:48 +01:00

vdeniaud commented

2022-12-20 15:41:07 +01:00

Ok, mais on ne prévoit rien pour réparer les configurations existantes ?

C'est mon dernier paragraphe, je pense que les configurations fautives sont rarissimes et se gèrent tranquillement à la main (et je viens de faire tourner le script et saas prod et test il n'y a que Haute Garonne qui ressort).

> Ok, mais on ne prévoit rien pour réparer les configurations existantes ? C'est mon dernier paragraphe, je pense que les configurations fautives sont rarissimes et se gèrent tranquillement à la main (et je viens de faire tourner le script et saas prod et test il n'y a que Haute Garonne qui ressort).

vdeniaud merged commit 227de93f0c into main

2022-12-20 16:04:52 +01:00

vdeniaud deleted branch wip/72625-api-datetimes-agenda-virtuel-Typ

2022-12-20 16:04:52 +01:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

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/chrono#26