[ADD]sale_timesheet_ticket_exclude #11

Merged
LaetitiaElabore merged 1 commits from 16.0-sale-timesheet-ticket-exclude into 16.0 2025-07-15 12:31:10 +00:00
LaetitiaElabore commented 2025-07-08 11:24:35 +00:00 (Migrated from github.com)
No description provided.
mondot (Migrated from github.com) reviewed 2025-07-08 15:44:09 +00:00
mondot (Migrated from github.com) left a comment

Le code est nickel 👌

Le code est nickel :ok_hand:
mondot (Migrated from github.com) commented 2025-07-08 15:32:16 +00:00

J'aurais bien mis un nom de module plus explicite, par ex helpdesk_timesheet_exclude_from_sale_order.

J'aurais bien mis un nom de module plus explicite, par ex `helpdesk_timesheet_exclude_from_sale_order`.
mondot (Migrated from github.com) commented 2025-07-08 15:43:32 +00:00

Je ne sais pas si dans Odoo c'est important nouvelle ligne à la fin des fichiers (si je ne dis pas de bêtises c'est une habitude chez beaucoup de développeurs parce que c'est nécessaire dans certains langages).

Je ne sais pas si dans Odoo c'est important nouvelle ligne à la fin des fichiers (si je ne dis pas de bêtises c'est une habitude chez beaucoup de développeurs parce que c'est nécessaire dans certains langages).
LaetitiaElabore (Migrated from github.com) reviewed 2025-07-15 09:55:12 +00:00
LaetitiaElabore (Migrated from github.com) commented 2025-07-15 09:55:11 +00:00

je suis d'accord avec toi, helpdesk_timesheet_exclude_from_sale_order me parait plus explicite
par la même occasion j'ai passé la catégorie du module de Tools à Helpdesk

je suis d'accord avec toi, `helpdesk_timesheet_exclude_from_sale_order` me parait plus explicite par la même occasion j'ai passé la catégorie du module de `Tools` à `Helpdesk`
LaetitiaElabore (Migrated from github.com) reviewed 2025-07-15 09:55:32 +00:00
LaetitiaElabore (Migrated from github.com) commented 2025-07-15 09:55:32 +00:00

ce n'est pas utile dans Odoo, on peut laisser comme ça

ce n'est pas utile dans Odoo, on peut laisser comme ça
mondot commented 2025-07-15 10:06:57 +00:00 (Migrated from github.com)

/approve

/approve
Sign in to join this conversation.
No description provided.