entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 39 Releases Activity

misc: change |age_in_working_days so it can return negative values (#71759) #609

Merged
fpeters merged 1 commits from wip/71759-working-days-future into main 2023-09-04 09:25:25 +02:00
Conversation 2 Commits 1 Files Changed 2 +6 -3
fpeters commented 2023-08-15 21:27:47 +02:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/71759-working-days-future from e6d429c502 to 10d38ad316 2023-08-16 07:25:26 +02:00 Compare
fpeters changed title from WIP: misc: change |age_in_working_days so it can return negative values (#71759) to misc: change |age_in_working_days so it can return negative values (#71759) 2023-08-16 08:07:48 +02:00
pmarillonnet approved these changes 2023-08-16 09:40:36 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ok mais je serais pour mettre en avant ce changement auprès des utilisateurs (peut-être via un courriel au club ?)

Ok mais je serais pour mettre en avant ce changement auprès des utilisateurs (peut-être via un courriel au club ?)
fpeters commented 2023-08-16 09:44:25 +02:00
Author
Owner
Copy Link

J'ai imaginé qu'on pourrait repérer les instances utilisant ça, et communiquer sur les cas où ça allait changer en suggérant d'utiliser |abs, mais comme je n'ai pas encore trouvé d'instance où ça poserait problème, ça peut sans doute juste être un petit commentaire dans les notes de mise à jour.

J'ai imaginé qu'on pourrait repérer les instances utilisant ça, et communiquer sur les cas où ça allait changer en suggérant d'utiliser |abs, mais comme je n'ai pas encore trouvé d'instance où ça poserait problème, ça peut sans doute juste être un petit commentaire dans les notes de mise à jour.
pmarillonnet commented 2023-08-16 13:15:51 +02:00
Owner
Copy Link

J'ai imaginé qu'on pourrait repérer les instances utilisant ça, et communiquer sur les cas où ça allait changer en suggérant d'utiliser |abs, mais comme je n'ai pas encore trouvé d'instance où ça poserait problème, ça peut sans doute juste être un petit commentaire dans les notes de mise à jour.

C’est le “tour pas exhaustif” de #71759-7 qui me rassurait moyen et qui me faisait penser qu’il fallait communiquer en amont des notes, mais c’est peut-être suffisant, ok.

> J'ai imaginé qu'on pourrait repérer les instances utilisant ça, et communiquer sur les cas où ça allait changer en suggérant d'utiliser |abs, mais comme je n'ai pas encore trouvé d'instance où ça poserait problème, ça peut sans doute juste être un petit commentaire dans les notes de mise à jour. C’est le “tour pas exhaustif” de #71759-7 qui me rassurait moyen et qui me faisait penser qu’il fallait communiquer en amont des notes, mais c’est peut-être suffisant, ok.
fpeters merged commit 3c411ce651 into main 2023-09-04 09:25:25 +02:00
fpeters deleted branch wip/71759-working-days-future 2023-09-04 09:25:25 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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/wcs#609
No description provided.
Delete Branch "wip/71759-working-days-future"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?