Repository navigation
[20.0][MIG] hr_timesheet_begin_end: Migration to 20.0 (review of OCA/timesheet#983) - #2
weinni2000 wants to merge 5 commits into
Conversation
|
Sorry @weinni2000, you've used your own review budget of 250,000 diff characters for the last 7 days. You can request another review in 4 days and 23 hours by commenting |
Reviewer's GuideImplementationAdapt duration comparison to the Odoo 20.0 unit API and validate overlapping times by employee, while preserving the existing timesheet validations.
UIUpdate the timesheet and task views to use the current view structure and honor the timesheet readonly modifier for begin/end fields.
ConfigurationBump the module version and register the renamed 20.0 view files.
DocumentationRevise the module documentation and contributor information for the 20.0 release.
TestsUpdate test setup to create an analytic plan and account using the current analytic model structure.
Lock files and vendored codeRegenerate the translation template and rendered module description for the 20.0 documentation and metadata.
Other changesFiles not grouped.
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
61567e4 to
f0607c9
Compare
5e9d76d to
e179165
Compare
The validation for overlapping times was done on user_id, which is the user entering the lines. This should be on employee_id, the employee *for who* the times are entered.
Odoo 20 removes uom.uom.rounding. Use the unit comparison API to keep duration validation consistent with Product Unit precision. Upstream: odoo/odoo@9a4ad7c
f0607c9 to
804b7b8
Compare
Review diff for OCA#983: only the
[MIG]commit (f27e652), compared against the state after the pre-commit auto fixes commit (5e9d76d).🤖 Generated with Claude Code
Summary by Sourcery
Migrate hr_timesheet_begin_end to Odoo 20.0 and align its timesheet validation, views, tests, and documentation with the new platform version.
Enhancements:
Documentation:
Tests: