[IMP] project_working_time_task_portal: display billable_remaining_hours instead of remaining_hours in project kanban view #25

2025-08-20T10:49:22Z

mondot commented

2025-08-20 10:49:22 +00:00

(Migrated from github.com)

https://lokavaluto.fr/web?debug=1#id=2526&cids=1&menu_id=144&action=977&model=project.task&view_type=form

LaetitiaElabore commented

2025-08-21 14:25:28 +00:00

(Migrated from github.com)

le module project_working_time_task_portal porte un nom qui suggère que ça concerne le portail, c'est pour ça que j'avais fait un module project_task_billable_hours pour la gestion de l'affichage des temps non facturable dans le back-end au niveau des taches. Ce choix pourrait être mis en question sans problème, mais si on veut garder la même logique, il faudrait un module project_project_billable_hours

le module `project_working_time_task_portal` porte un nom qui suggère que ça concerne le portail, c'est pour ça que j'avais fait un module `project_task_billable_hours` pour la gestion de l'affichage des temps non facturable dans le back-end au niveau des taches. Ce choix pourrait être mis en question sans problème, mais si on veut garder la même logique, il faudrait un module `project_project_billable_hours`

mondot commented

2025-09-01 08:20:52 +00:00

(Migrated from github.com)

Il y a effectivement un méli mélo dans le module project_working_time_task_portal (avant mes modifications) : on modifie des vues du portail portal_tasks_list et des vues du backoffice project_working_time_view_task_form.

Je ne suis pas fan de multiplier les modules pour des mini features comme celles-ci (car souvent on les veut toutes). Est-ce qu'on ne voudrait pas faire un module unifié project_task_billable_hours ?

Il y a effectivement un méli mélo dans le module `project_working_time_task_portal` (avant mes modifications) : on modifie des vues du portail `portal_tasks_list` et des vues du backoffice `project_working_time_view_task_form`. Je ne suis pas fan de multiplier les modules pour des mini features comme celles-ci (car souvent on les veut toutes). Est-ce qu'on ne voudrait pas faire un module unifié `project_task_billable_hours` ?

LaetitiaElabore commented

2025-09-01 08:29:20 +00:00

(Migrated from github.com)

je suis d'accord avec toi, l'idéal serait de renommer le module et d'y mettre toutes les fonctionnalités qu'on veut ensemble. Mais si on change le nom d'un module déjà déployé chez des clients, on est obligé de l'associer à un script de migration, du coup ça fait un peu usine à gaz pour pas grand chose. A moins que le script de migration soit simple. Alors pourquoi pas

mondot commented

2025-09-04 15:27:13 +00:00

(Migrated from github.com)

je suis d'accord avec toi, l'idéal serait de renommer le module et d'y mettre toutes les fonctionnalités qu'on veut ensemble. Mais si on change le nom d'un module déjà déployé chez des clients, on est obligé de l'associer à un script de migration, du coup ça fait un peu usine à gaz pour pas grand chose. A moins que le script de migration soit simple. Alors pourquoi pas

Ok j'ai voulu faire la fusion mais après discussion avec Stephan je me suis rendu compte que c'était trop compliqué car on ne sait pas lesquels de nos clients ont ces modules installés.

Du coup j'ai préféré mettre à jour project_task_billable_hours, qui a une dépendance à project_working_time_task_portal donc c'est ok. J'ai créé une nouvelle PR pour ces changements : https://github.com/elabore-coop/project-tools/pull/26.

> je suis d'accord avec toi, l'idéal serait de renommer le module et d'y mettre toutes les fonctionnalités qu'on veut ensemble. Mais si on change le nom d'un module déjà déployé chez des clients, on est obligé de l'associer à un script de migration, du coup ça fait un peu usine à gaz pour pas grand chose. A moins que le script de migration soit simple. Alors pourquoi pas Ok j'ai voulu faire la fusion mais après discussion avec Stephan je me suis rendu compte que c'était trop compliqué car on ne sait pas lesquels de nos clients ont ces modules installés. Du coup j'ai préféré mettre à jour `project_task_billable_hours`, qui a une dépendance à `project_working_time_task_portal` donc c'est ok. J'ai créé une nouvelle PR pour ces changements : https://github.com/elabore-coop/project-tools/pull/26.

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Elabore/project-tools#25