Configuration du temps de travail par salarié #57
Loading…
Reference in a new issue
No description provided.
Delete branch "23-user-presence-setting"
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?
fixes #55
Migration :
employee_countsuser_presence_settingsChangements :
Solde horaire en début de semaine@ -70,0 +140,4 @@total = Decimal(sum(current_by_user.values())) / 100history.append((effective_date, total))history.reverse()Ça me lève un warning qu'il faille faire un reverse pour que le template affiche quelque chose de correct.
j'ai l'impression que ça devrait plutôt être de la responsabilité de la vue d'organiser la donnée, et encore, ça ne serait tout de même pas vraiment à elle d'ordonner les valeurs, peut-être encore plus loin, dans le template ?
Les fonctions ont été déplacées dans les fichiers qui les utilisent :
endpoint/time_entries.pyetrepositoriesAu passage, j'ai clarifié certaines lignes de code complexes avec des boucles sur users plutôt que des requêtes qui auraient nécessité des commentaires.
@ -566,3 +566,3 @@@propertydef working_employees(self) -> Decimal | None:if self.total_production_days is None:if not self.total_production_days:transformer ça en falsy me semble dangereux, je pense qu'on veut capter les valeurs 0 et donc garder
is None@ -50,0 +114,4 @@def delete_presence_setting(user, setting):"""Delete a presence percentage setting."""if setting.user_id != user.id:abort(404)Reprend plutôt ce qu'il y a dans le flow weekly avec un target_user pour ne pas occuper la variable
userdans le template.Mais cette vérification pourrait en soit disparaitre.
34f5d8a4ab7f8bcdedd6