From 764a1c056f1fd6ae0aef12cd81b0260b5c61d56e Mon Sep 17 00:00:00 2001
From: Carmen Bianca BAKKER <carmen@coopiteasy.be>
Date: Fri, 28 Jun 2024 11:30:27 +0200
Subject: [PATCH 1/6] [FIX] hr_timesheet_begin_end: Test uses timesheet lines
 now

Timesheet lines are characterised by having a project.

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>
---
 .../readme/newsfragments/692.bugfix.1.rst                | 1 +
 hr_timesheet_begin_end/tests/test_timesheet_begin_end.py | 9 ++++-----
 2 files changed, 5 insertions(+), 5 deletions(-)
 create mode 100644 hr_timesheet_begin_end/readme/newsfragments/692.bugfix.1.rst

diff --git a/hr_timesheet_begin_end/readme/newsfragments/692.bugfix.1.rst b/hr_timesheet_begin_end/readme/newsfragments/692.bugfix.1.rst
new file mode 100644
index 0000000000..056c17a06b
--- /dev/null
+++ b/hr_timesheet_begin_end/readme/newsfragments/692.bugfix.1.rst
@@ -0,0 +1 @@
+Fixed the test to use timesheet lines instead of bare analytic lines.
diff --git a/hr_timesheet_begin_end/tests/test_timesheet_begin_end.py b/hr_timesheet_begin_end/tests/test_timesheet_begin_end.py
index a70b9b73b5..85ee99d01a 100644
--- a/hr_timesheet_begin_end/tests/test_timesheet_begin_end.py
+++ b/hr_timesheet_begin_end/tests/test_timesheet_begin_end.py
@@ -9,17 +9,16 @@ class TestBeginEnd(common.TransactionCase):
     def setUp(self):
         super(TestBeginEnd, self).setUp()
         self.timesheet_line_model = self.env["account.analytic.line"]
-        self.analytic = self.env.ref("analytic.analytic_administratif")
-        self.user = self.env.ref("base.user_root")
+        self.project = self.env.ref("project.project_project_1")
+        self.employee = self.env.ref("hr.employee_qdp")
         self.base_line = {
             "name": "test",
             "date": fields.Date.today(),
             "time_start": 10.0,
             "time_stop": 12.0,
-            "user_id": self.user.id,
             "unit_amount": 2.0,
-            "account_id": self.analytic.id,
-            "amount": -60.0,
+            "project_id": self.project.id,
+            "employee_id": self.employee.id,
         }
 
     def test_onchange(self):

From 0985b0d3de701186b6dcdd2f5d18b70cae5205a6 Mon Sep 17 00:00:00 2001
From: Carmen Bianca BAKKER <carmen@coopiteasy.be>
Date: Mon, 17 Jun 2024 12:10:09 +0200
Subject: [PATCH 2/6] [REF] hr_timesheet_begin_end: Make more extensible

The separation of checks into separate methods is needed because I want
to disable one check in another module. This makes the module more
extensible.

The unit_amount_from_start_stop method also makes the module more
extensible.

I have also moved the onchange to the top of the file, according to the
OCA contribution guidelines.

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>
---
 .../models/account_analytic_line.py           | 75 ++++++++++++-------
 .../readme/newsfragments/692.feature.1.rst    |  1 +
 2 files changed, 50 insertions(+), 26 deletions(-)
 create mode 100644 hr_timesheet_begin_end/readme/newsfragments/692.feature.1.rst

diff --git a/hr_timesheet_begin_end/models/account_analytic_line.py b/hr_timesheet_begin_end/models/account_analytic_line.py
index e49c22ad41..40bbbeac0f 100644
--- a/hr_timesheet_begin_end/models/account_analytic_line.py
+++ b/hr_timesheet_begin_end/models/account_analytic_line.py
@@ -1,5 +1,6 @@
 # Copyright 2015 Camptocamp SA - Guewen Baconnier
 # Copyright 2017 Tecnativa, S.L. - Luis M. Ontalba
+# Copyright 2024 Coop IT Easy SC - Carmen Bianca BAKKER
 # License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
 
 from datetime import timedelta
@@ -15,16 +16,14 @@ class AccountAnalyticLine(models.Model):
     time_start = fields.Float(string="Begin Hour")
     time_stop = fields.Float(string="End Hour")
 
-    @api.constrains("time_start", "time_stop", "unit_amount")
-    def _check_time_start_stop(self):
-        for line in self:
-            value_to_html = self.env["ir.qweb.field.float_time"].value_to_html
-            start = timedelta(hours=line.time_start)
-            stop = timedelta(hours=line.time_stop)
-            if stop < start:
-                value_to_html(line.time_start, None)
-                value_to_html(line.time_stop, None)
+    @api.onchange("time_start", "time_stop", "project_id")
+    def onchange_hours_start_stop(self):
+        self.unit_amount = self.unit_amount_from_start_stop()
 
+    def _validate_start_before_stop(self):
+        value_to_html = self.env["ir.qweb.field.float_time"].value_to_html
+        for line in self:
+            if line.time_stop < line.time_start:
                 raise exceptions.ValidationError(
                     _(
                         "The beginning hour (%(html_start)s) must "
@@ -35,7 +34,11 @@ def _check_time_start_stop(self):
                         "html_stop": value_to_html(line.time_stop, None),
                     }
                 )
-            hours = (stop - start).seconds / 3600
+
+    def _validate_unit_amount_equal_to_time_diff(self):
+        value_to_html = self.env["ir.qweb.field.float_time"].value_to_html
+        for line in self:
+            hours = line.unit_amount_from_start_stop()
             rounding = self.env.ref("uom.product_uom_hour").rounding
             if hours and float_compare(
                 hours, line.unit_amount, precision_rounding=rounding
@@ -50,16 +53,21 @@ def _check_time_start_stop(self):
                         "html_hours": value_to_html(hours, None),
                     }
                 )
-            # check if lines overlap
-            others = self.search(
-                [
-                    ("id", "!=", line.id),
-                    ("employee_id", "=", line.employee_id.id),
-                    ("date", "=", line.date),
-                    ("time_start", "<", line.time_stop),
-                    ("time_stop", ">", line.time_start),
-                ]
-            )
+
+    def _overlap_domain(self):
+        self.ensure_one()
+        return [
+            ("id", "!=", self.id),
+            ("employee_id", "=", self.employee_id.id),
+            ("date", "=", self.date),
+            ("time_start", "<", self.time_stop),
+            ("time_stop", ">", self.time_start),
+        ]
+
+    def _validate_no_overlap(self):
+        value_to_html = self.env["ir.qweb.field.float_time"].value_to_html
+        for line in self:
+            others = self.search(line._overlap_domain())
             if others:
                 message = _("Lines can't overlap:\n")
                 message += "\n".join(
@@ -74,13 +82,28 @@ def _check_time_start_stop(self):
                 )
                 raise exceptions.ValidationError(message)
 
-    @api.onchange("time_start", "time_stop")
-    def onchange_hours_start_stop(self):
-        start = timedelta(hours=self.time_start)
-        stop = timedelta(hours=self.time_stop)
+    @api.constrains("time_start", "time_stop", "unit_amount")
+    def _check_time_start_stop(self):
+        lines = self.filtered(lambda line: line.project_id)
+        lines._validate_start_before_stop()
+        lines._validate_unit_amount_equal_to_time_diff()
+        lines._validate_no_overlap()
+
+    @api.model
+    def _hours_from_start_stop(self, time_start, time_stop):
+        start = timedelta(hours=time_start)
+        stop = timedelta(hours=time_stop)
         if stop < start:
-            return
-        self.unit_amount = (stop - start).seconds / 3600
+            # Invalid case, but return something sensible.
+            return 0
+        return (stop - start).seconds / 3600
+
+    def unit_amount_from_start_stop(self):
+        self.ensure_one()
+        # Don't handle non-timesheet lines.
+        if not self.project_id:
+            return 0
+        return self._hours_from_start_stop(self.time_start, self.time_stop)
 
     def merge_timesheets(self):  # pragma: no cover
         """This method is needed in case hr_timesheet_sheet is installed"""
diff --git a/hr_timesheet_begin_end/readme/newsfragments/692.feature.1.rst b/hr_timesheet_begin_end/readme/newsfragments/692.feature.1.rst
new file mode 100644
index 0000000000..366ce40d00
--- /dev/null
+++ b/hr_timesheet_begin_end/readme/newsfragments/692.feature.1.rst
@@ -0,0 +1 @@
+Refactored the module to be more extensible.

From 55c67c254f68a64a42a395a4e4d2545a36406392 Mon Sep 17 00:00:00 2001
From: Carmen Bianca BAKKER <carmen@coopiteasy.be>
Date: Mon, 17 Jun 2024 17:21:49 +0200
Subject: [PATCH 3/6] [REF] hr_timesheet_begin_end: Refactor unit_amount to be
 a compute field

Instead of using onchange, which is less convenient in Odoo 16.

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>
---
 .../models/account_analytic_line.py           | 26 +++++++++++--
 .../readme/newsfragments/692.feature.2.rst    |  1 +
 .../tests/test_timesheet_begin_end.py         | 37 +++++++++++++++----
 3 files changed, 53 insertions(+), 11 deletions(-)
 create mode 100644 hr_timesheet_begin_end/readme/newsfragments/692.feature.2.rst

diff --git a/hr_timesheet_begin_end/models/account_analytic_line.py b/hr_timesheet_begin_end/models/account_analytic_line.py
index 40bbbeac0f..70a6e4dcf8 100644
--- a/hr_timesheet_begin_end/models/account_analytic_line.py
+++ b/hr_timesheet_begin_end/models/account_analytic_line.py
@@ -16,9 +16,29 @@ class AccountAnalyticLine(models.Model):
     time_start = fields.Float(string="Begin Hour")
     time_stop = fields.Float(string="End Hour")
 
-    @api.onchange("time_start", "time_stop", "project_id")
-    def onchange_hours_start_stop(self):
-        self.unit_amount = self.unit_amount_from_start_stop()
+    # Override to be a computed field.
+    unit_amount = fields.Float(
+        compute="_compute_unit_amount",
+        store=True,
+        readonly=False,
+        # This default is a workaround for a bizarre situation: if a line is
+        # created with a time range but WITHOUT defining unit_amount, then you
+        # would expect unit_amount to be computed from the range. But this never
+        # happens, and it is instead set to default value 0. Subsequently the
+        # constraint _validate_unit_amount_equal_to_time_diff kicks in and
+        # raises an exception.
+        #
+        # By setting the default to None, the computation is correctly
+        # triggered. If nothing is computed, None falls back to 0.
+        default=None,
+    )
+
+    @api.depends("time_start", "time_stop", "project_id")
+    def _compute_unit_amount(self):
+        # Do not compute/adjust the unit_amount of non-timesheets.
+        lines = self.filtered(lambda line: line.project_id)
+        for line in lines:
+            line.unit_amount = line.unit_amount_from_start_stop()
 
     def _validate_start_before_stop(self):
         value_to_html = self.env["ir.qweb.field.float_time"].value_to_html
diff --git a/hr_timesheet_begin_end/readme/newsfragments/692.feature.2.rst b/hr_timesheet_begin_end/readme/newsfragments/692.feature.2.rst
new file mode 100644
index 0000000000..185296c901
--- /dev/null
+++ b/hr_timesheet_begin_end/readme/newsfragments/692.feature.2.rst
@@ -0,0 +1 @@
+Changed ``unit_amount`` into a computed (stored, writeable) field.
diff --git a/hr_timesheet_begin_end/tests/test_timesheet_begin_end.py b/hr_timesheet_begin_end/tests/test_timesheet_begin_end.py
index 85ee99d01a..7992565aea 100644
--- a/hr_timesheet_begin_end/tests/test_timesheet_begin_end.py
+++ b/hr_timesheet_begin_end/tests/test_timesheet_begin_end.py
@@ -21,18 +21,39 @@ def setUp(self):
             "employee_id": self.employee.id,
         }
 
-    def test_onchange(self):
-        line = self.timesheet_line_model.new(
-            {"name": "test", "time_start": 10.0, "time_stop": 12.0}
-        )
-        line.onchange_hours_start_stop()
-        self.assertEqual(line.unit_amount, 2)
+    def test_compute_unit_amount(self):
+        line = self.base_line.copy()
+        del line["unit_amount"]
+        line_record = self.timesheet_line_model.create(line)
+        self.assertEqual(line_record.unit_amount, 2)
+        line_record.time_stop = 14.0
+        self.assertEqual(line_record.unit_amount, 4)
+
+    def test_compute_unit_amount_no_compute_if_no_times(self):
+        line = self.base_line.copy()
+        del line["time_start"]
+        del line["time_stop"]
+        line_record = self.timesheet_line_model.create(line)
+        self.assertEqual(line_record.unit_amount, 2.0)
+        line_record.unit_amount = 3.0
+        self.assertEqual(line_record.unit_amount, 3.0)
+
+    def test_compute_unit_amount_to_zero(self):
+        line = self.base_line.copy()
+        del line["unit_amount"]
+        line_record = self.timesheet_line_model.create(line)
+        self.assertEqual(line_record.unit_amount, 2)
+        line_record.write({"time_start": 0, "time_stop": 0})
+        self.assertEqual(line_record.unit_amount, 0)
 
-    def test_onchange_no_update(self):
+    def test_compute_unit_amount_to_zero_no_record(self):
+        # Cannot create/save this model because it breaks a constraint, so using
+        # .new().
         line = self.timesheet_line_model.new(
             {"name": "test", "time_start": 13.0, "time_stop": 12.0}
         )
-        line.onchange_hours_start_stop()
+        self.assertEqual(line.unit_amount, 0)
+        line.time_stop = 10.0
         self.assertEqual(line.unit_amount, 0)
 
     def test_check_begin_before_end(self):

From b601c05f1a4b6ec0408cdddc306090b44f11ff80 Mon Sep 17 00:00:00 2001
From: Carmen Bianca BAKKER <carmen@coopiteasy.be>
Date: Fri, 28 Jun 2024 14:45:06 +0200
Subject: [PATCH 4/6] [FIX] Add hr_timesheet_begin_end to rebel modules

This is a tricky one. _Technically_ there is nothing that makes
hr_timesheet_begin_end incompatible with the other modules here.
However, when the project_id on a timesheet line is changed, unit_amount
is rerecomputed. This has some consequences.

In the sale_timesheet tests, the project_id of a timesheet line
is (sometimes?) recomputed. Subsequently, unit_amount is recomputed and
set to 0 when it _shouldn't_ be. Under a normal flow this wouldn't be a
problem; time_start and time_stop would be populated to correctly
recompute unit_amount. But in the tests, they are not.

One solution is to, in addition to not recomputing the unit_amount of
non-timesheet lines, not recompute the unit_amount of timesheet lines
whose time_start and time_stop are both set to 00:00. However, this
means that resetting these values to 00:00 does not correctly set
unit_amount back to 0, which seems erroneous to me.

Marking this module as a rebel module seems like the best course of
action to me, even though it would be preferable to test this in
conjunction with the other modules.

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>
---
 .copier-answers.yml        |  3 ++-
 .github/workflows/test.yml | 21 ++++++++++++---------
 .gitignore                 |  9 +++++++++
 3 files changed, 23 insertions(+), 10 deletions(-)

diff --git a/.copier-answers.yml b/.copier-answers.yml
index fd762c0179..e3f412721a 100644
--- a/.copier-answers.yml
+++ b/.copier-answers.yml
@@ -1,5 +1,5 @@
 # Do NOT update manually; changes here will be overwritten by Copier
-_commit: v1.21.1
+_commit: '1.23'
 _src_path: gh:oca/oca-addons-repo-template
 ci: GitHub
 convert_readme_fragments_to_markdown: false
@@ -17,6 +17,7 @@ org_name: Odoo Community Association (OCA)
 org_slug: OCA
 rebel_module_groups:
 - sale_timesheet_rounded
+- hr_timesheet_begin_end
 repo_description: 'TODO: add repo description.'
 repo_name: timesheet
 repo_slug: timesheet
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 3bad198c1f..9f621e11c4 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -37,18 +37,25 @@ jobs:
         include:
           - container: ghcr.io/oca/oca-ci/py3.10-odoo16.0:latest
             include: "sale_timesheet_rounded"
-            makepot: "true"
             name: test with Odoo
           - container: ghcr.io/oca/oca-ci/py3.10-ocb16.0:latest
             include: "sale_timesheet_rounded"
             name: test with OCB
+            makepot: "true"
           - container: ghcr.io/oca/oca-ci/py3.10-odoo16.0:latest
-            exclude: "sale_timesheet_rounded"
+            include: "hr_timesheet_begin_end"
+            name: test with Odoo
+          - container: ghcr.io/oca/oca-ci/py3.10-ocb16.0:latest
+            include: "hr_timesheet_begin_end"
+            name: test with OCB
             makepot: "true"
+          - container: ghcr.io/oca/oca-ci/py3.10-odoo16.0:latest
+            exclude: "sale_timesheet_rounded,hr_timesheet_begin_end"
             name: test with Odoo
           - container: ghcr.io/oca/oca-ci/py3.10-ocb16.0:latest
-            exclude: "sale_timesheet_rounded"
+            exclude: "sale_timesheet_rounded,hr_timesheet_begin_end"
             name: test with OCB
+            makepot: "true"
     services:
       postgres:
         image: postgres:12.0
@@ -79,9 +86,5 @@ jobs:
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
       - name: Update .pot files
-        run:
-          oca_export_and_push_pot https://x-access-token:${{ secrets.GIT_PUSH_TOKEN
-          }}@github.com/${{ github.repository }}
-        if:
-          ${{ matrix.makepot == 'true' && github.event_name == 'push' &&
-          github.repository_owner == 'OCA' }}
+        run: oca_export_and_push_pot https://x-access-token:${{ secrets.GIT_PUSH_TOKEN }}@github.com/${{ github.repository }}
+        if: ${{ matrix.makepot == 'true' && github.event_name == 'push' && github.repository_owner == 'OCA' }}
diff --git a/.gitignore b/.gitignore
index 0090721f5d..2b045db399 100644
--- a/.gitignore
+++ b/.gitignore
@@ -25,6 +25,15 @@ var/
 *.egg
 *.eggs
 
+# Debian packages
+*.deb
+
+# Redhat packages
+*.rpm
+
+# MacOS packages
+*.dmg
+
 # Installer logs
 pip-log.txt
 pip-delete-this-directory.txt

From 38be3be990b8e336e5168ce0d4a1432712e31bb2 Mon Sep 17 00:00:00 2001
From: Carmen Bianca BAKKER <carmen@coopiteasy.be>
Date: Mon, 17 Jun 2024 17:06:38 +0200
Subject: [PATCH 5/6] [ADD] hr_timesheet_begin_end_no_end

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>
---
 hr_timesheet_begin_end_no_end/README.rst      |  88 ++++
 hr_timesheet_begin_end_no_end/__init__.py     |   5 +
 hr_timesheet_begin_end_no_end/__manifest__.py |  18 +
 .../models/__init__.py                        |   5 +
 .../models/account_analytic_line.py           |  16 +
 .../readme/CONTRIBUTORS.rst                   |   3 +
 .../readme/DESCRIPTION.rst                    |   3 +
 .../static/description/index.html             | 427 ++++++++++++++++++
 .../tests/__init__.py                         |   5 +
 .../tests/test_timesheet_begin_end_no_end.py  |  53 +++
 .../odoo/addons/hr_timesheet_begin_end_no_end |   1 +
 setup/hr_timesheet_begin_end_no_end/setup.py  |   6 +
 12 files changed, 630 insertions(+)
 create mode 100644 hr_timesheet_begin_end_no_end/README.rst
 create mode 100644 hr_timesheet_begin_end_no_end/__init__.py
 create mode 100644 hr_timesheet_begin_end_no_end/__manifest__.py
 create mode 100644 hr_timesheet_begin_end_no_end/models/__init__.py
 create mode 100644 hr_timesheet_begin_end_no_end/models/account_analytic_line.py
 create mode 100644 hr_timesheet_begin_end_no_end/readme/CONTRIBUTORS.rst
 create mode 100644 hr_timesheet_begin_end_no_end/readme/DESCRIPTION.rst
 create mode 100644 hr_timesheet_begin_end_no_end/static/description/index.html
 create mode 100644 hr_timesheet_begin_end_no_end/tests/__init__.py
 create mode 100644 hr_timesheet_begin_end_no_end/tests/test_timesheet_begin_end_no_end.py
 create mode 120000 setup/hr_timesheet_begin_end_no_end/odoo/addons/hr_timesheet_begin_end_no_end
 create mode 100644 setup/hr_timesheet_begin_end_no_end/setup.py

diff --git a/hr_timesheet_begin_end_no_end/README.rst b/hr_timesheet_begin_end_no_end/README.rst
new file mode 100644
index 0000000000..5e65ba1e93
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/README.rst
@@ -0,0 +1,88 @@
+==========================================
+Timesheet - Begin/End Hours with empty end
+==========================================
+
+.. 
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! This file is generated by oca-gen-addon-readme !!
+   !! changes will be overwritten.                   !!
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! source digest: sha256:6f9bf57eb39a36725b852081172b71f26e16fed507227ba22b894d0f1ed0d8e5
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+
+.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
+    :target: https://odoo-community.org/page/development-status
+    :alt: Beta
+.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
+    :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+    :alt: License: AGPL-3
+.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Ftimesheet-lightgray.png?logo=github
+    :target: https://github.com/OCA/timesheet/tree/16.0/hr_timesheet_begin_end_no_end
+    :alt: OCA/timesheet
+.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
+    :target: https://translation.odoo-community.org/projects/timesheet-16-0/timesheet-16-0-hr_timesheet_begin_end_no_end
+    :alt: Translate me on Weblate
+.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/timesheet&target_branch=16.0
+    :alt: Try me on Runboat
+
+|badge1| |badge2| |badge3| |badge4| |badge5|
+
+Allow to insert begin hour without end hour on the timesheet. The benefit of
+doing this is that you can save your begin hour at the start of your day/task,
+and come back later to save the end hour.
+
+**Table of contents**
+
+.. contents::
+   :local:
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues <https://github.com/OCA/timesheet/issues>`_.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+`feedback <https://github.com/OCA/timesheet/issues/new?body=module:%20hr_timesheet_begin_end_no_end%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+
+Do not contact contributors directly about support or help with technical issues.
+
+Credits
+=======
+
+Authors
+~~~~~~~
+
+* Coop IT Easy SC
+
+Contributors
+~~~~~~~~~~~~
+
+* `Coop IT Easy SC <https://coopiteasy.be>`_:
+
+  * Carmen Bianca BAKKER
+
+Maintainers
+~~~~~~~~~~~
+
+This module is maintained by the OCA.
+
+.. image:: https://odoo-community.org/logo.png
+   :alt: Odoo Community Association
+   :target: https://odoo-community.org
+
+OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.
+
+.. |maintainer-carmenbianca| image:: https://github.com/carmenbianca.png?size=40px
+    :target: https://github.com/carmenbianca
+    :alt: carmenbianca
+
+Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
+
+|maintainer-carmenbianca| 
+
+This module is part of the `OCA/timesheet <https://github.com/OCA/timesheet/tree/16.0/hr_timesheet_begin_end_no_end>`_ project on GitHub.
+
+You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/hr_timesheet_begin_end_no_end/__init__.py b/hr_timesheet_begin_end_no_end/__init__.py
new file mode 100644
index 0000000000..3eb78877c5
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/__init__.py
@@ -0,0 +1,5 @@
+# SPDX-FileCopyrightText: 2024 Coop IT Easy SC
+#
+# SPDX-License-Identifier: AGPL-3.0-or-later
+
+from . import models
diff --git a/hr_timesheet_begin_end_no_end/__manifest__.py b/hr_timesheet_begin_end_no_end/__manifest__.py
new file mode 100644
index 0000000000..ad6d33bf14
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/__manifest__.py
@@ -0,0 +1,18 @@
+# SPDX-FileCopyrightText: 2024 Coop IT Easy SC
+#
+# SPDX-License-Identifier: AGPL-3.0-or-later
+
+{
+    "name": "Timesheet - Begin/End Hours with empty end",
+    "summary": (
+        "Allow to insert begin hour without end hour, which the user may fill"
+        " in later."
+    ),
+    "version": "16.0.1.0.0",
+    "category": "Human Resources",
+    "website": "https://github.com/OCA/timesheet",
+    "author": "Coop IT Easy SC, Odoo Community Association (OCA)",
+    "maintainers": ["carmenbianca"],
+    "license": "AGPL-3",
+    "depends": ["hr_timesheet_begin_end"],
+}
diff --git a/hr_timesheet_begin_end_no_end/models/__init__.py b/hr_timesheet_begin_end_no_end/models/__init__.py
new file mode 100644
index 0000000000..84f4adcd2b
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/models/__init__.py
@@ -0,0 +1,5 @@
+# SPDX-FileCopyrightText: 2024 Coop IT Easy SC
+#
+# SPDX-License-Identifier: AGPL-3.0-or-later
+
+from . import account_analytic_line
diff --git a/hr_timesheet_begin_end_no_end/models/account_analytic_line.py b/hr_timesheet_begin_end_no_end/models/account_analytic_line.py
new file mode 100644
index 0000000000..5a0fcdd75e
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/models/account_analytic_line.py
@@ -0,0 +1,16 @@
+# SPDX-FileCopyrightText: 2024 Coop IT Easy SC
+#
+# SPDX-License-Identifier: AGPL-3.0-or-later
+
+
+from odoo import models
+
+
+class AccountAnalyticLine(models.Model):
+    _inherit = "account.analytic.line"
+
+    def _validate_start_before_stop(self):
+        for line in self:
+            if line.time_stop:
+                super(AccountAnalyticLine, line)._validate_start_before_stop()
+        return
diff --git a/hr_timesheet_begin_end_no_end/readme/CONTRIBUTORS.rst b/hr_timesheet_begin_end_no_end/readme/CONTRIBUTORS.rst
new file mode 100644
index 0000000000..f1ac675779
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/readme/CONTRIBUTORS.rst
@@ -0,0 +1,3 @@
+* `Coop IT Easy SC <https://coopiteasy.be>`_:
+
+  * Carmen Bianca BAKKER
diff --git a/hr_timesheet_begin_end_no_end/readme/DESCRIPTION.rst b/hr_timesheet_begin_end_no_end/readme/DESCRIPTION.rst
new file mode 100644
index 0000000000..8376256322
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/readme/DESCRIPTION.rst
@@ -0,0 +1,3 @@
+Allow to insert begin hour without end hour on the timesheet. The benefit of
+doing this is that you can save your begin hour at the start of your day/task,
+and come back later to save the end hour.
diff --git a/hr_timesheet_begin_end_no_end/static/description/index.html b/hr_timesheet_begin_end_no_end/static/description/index.html
new file mode 100644
index 0000000000..76647fd432
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/static/description/index.html
@@ -0,0 +1,427 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
+<head>
+<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+<meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
+<title>Timesheet - Begin/End Hours with empty end</title>
+<style type="text/css">
+
+/*
+:Author: David Goodger (goodger@python.org)
+:Id: $Id: html4css1.css 8954 2022-01-20 10:10:25Z milde $
+:Copyright: This stylesheet has been placed in the public domain.
+
+Default cascading style sheet for the HTML output of Docutils.
+
+See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
+customize this style sheet.
+*/
+
+/* used to remove borders from tables and images */
+.borderless, table.borderless td, table.borderless th {
+  border: 0 }
+
+table.borderless td, table.borderless th {
+  /* Override padding for "table.docutils td" with "! important".
+     The right padding separates the table cells. */
+  padding: 0 0.5em 0 0 ! important }
+
+.first {
+  /* Override more specific margin styles with "! important". */
+  margin-top: 0 ! important }
+
+.last, .with-subtitle {
+  margin-bottom: 0 ! important }
+
+.hidden {
+  display: none }
+
+.subscript {
+  vertical-align: sub;
+  font-size: smaller }
+
+.superscript {
+  vertical-align: super;
+  font-size: smaller }
+
+a.toc-backref {
+  text-decoration: none ;
+  color: black }
+
+blockquote.epigraph {
+  margin: 2em 5em ; }
+
+dl.docutils dd {
+  margin-bottom: 0.5em }
+
+object[type="image/svg+xml"], object[type="application/x-shockwave-flash"] {
+  overflow: hidden;
+}
+
+/* Uncomment (and remove this text!) to get bold-faced definition list terms
+dl.docutils dt {
+  font-weight: bold }
+*/
+
+div.abstract {
+  margin: 2em 5em }
+
+div.abstract p.topic-title {
+  font-weight: bold ;
+  text-align: center }
+
+div.admonition, div.attention, div.caution, div.danger, div.error,
+div.hint, div.important, div.note, div.tip, div.warning {
+  margin: 2em ;
+  border: medium outset ;
+  padding: 1em }
+
+div.admonition p.admonition-title, div.hint p.admonition-title,
+div.important p.admonition-title, div.note p.admonition-title,
+div.tip p.admonition-title {
+  font-weight: bold ;
+  font-family: sans-serif }
+
+div.attention p.admonition-title, div.caution p.admonition-title,
+div.danger p.admonition-title, div.error p.admonition-title,
+div.warning p.admonition-title, .code .error {
+  color: red ;
+  font-weight: bold ;
+  font-family: sans-serif }
+
+/* Uncomment (and remove this text!) to get reduced vertical space in
+   compound paragraphs.
+div.compound .compound-first, div.compound .compound-middle {
+  margin-bottom: 0.5em }
+
+div.compound .compound-last, div.compound .compound-middle {
+  margin-top: 0.5em }
+*/
+
+div.dedication {
+  margin: 2em 5em ;
+  text-align: center ;
+  font-style: italic }
+
+div.dedication p.topic-title {
+  font-weight: bold ;
+  font-style: normal }
+
+div.figure {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+div.footer, div.header {
+  clear: both;
+  font-size: smaller }
+
+div.line-block {
+  display: block ;
+  margin-top: 1em ;
+  margin-bottom: 1em }
+
+div.line-block div.line-block {
+  margin-top: 0 ;
+  margin-bottom: 0 ;
+  margin-left: 1.5em }
+
+div.sidebar {
+  margin: 0 0 0.5em 1em ;
+  border: medium outset ;
+  padding: 1em ;
+  background-color: #ffffee ;
+  width: 40% ;
+  float: right ;
+  clear: right }
+
+div.sidebar p.rubric {
+  font-family: sans-serif ;
+  font-size: medium }
+
+div.system-messages {
+  margin: 5em }
+
+div.system-messages h1 {
+  color: red }
+
+div.system-message {
+  border: medium outset ;
+  padding: 1em }
+
+div.system-message p.system-message-title {
+  color: red ;
+  font-weight: bold }
+
+div.topic {
+  margin: 2em }
+
+h1.section-subtitle, h2.section-subtitle, h3.section-subtitle,
+h4.section-subtitle, h5.section-subtitle, h6.section-subtitle {
+  margin-top: 0.4em }
+
+h1.title {
+  text-align: center }
+
+h2.subtitle {
+  text-align: center }
+
+hr.docutils {
+  width: 75% }
+
+img.align-left, .figure.align-left, object.align-left, table.align-left {
+  clear: left ;
+  float: left ;
+  margin-right: 1em }
+
+img.align-right, .figure.align-right, object.align-right, table.align-right {
+  clear: right ;
+  float: right ;
+  margin-left: 1em }
+
+img.align-center, .figure.align-center, object.align-center {
+  display: block;
+  margin-left: auto;
+  margin-right: auto;
+}
+
+table.align-center {
+  margin-left: auto;
+  margin-right: auto;
+}
+
+.align-left {
+  text-align: left }
+
+.align-center {
+  clear: both ;
+  text-align: center }
+
+.align-right {
+  text-align: right }
+
+/* reset inner alignment in figures */
+div.align-right {
+  text-align: inherit }
+
+/* div.align-center * { */
+/*   text-align: left } */
+
+.align-top    {
+  vertical-align: top }
+
+.align-middle {
+  vertical-align: middle }
+
+.align-bottom {
+  vertical-align: bottom }
+
+ol.simple, ul.simple {
+  margin-bottom: 1em }
+
+ol.arabic {
+  list-style: decimal }
+
+ol.loweralpha {
+  list-style: lower-alpha }
+
+ol.upperalpha {
+  list-style: upper-alpha }
+
+ol.lowerroman {
+  list-style: lower-roman }
+
+ol.upperroman {
+  list-style: upper-roman }
+
+p.attribution {
+  text-align: right ;
+  margin-left: 50% }
+
+p.caption {
+  font-style: italic }
+
+p.credits {
+  font-style: italic ;
+  font-size: smaller }
+
+p.label {
+  white-space: nowrap }
+
+p.rubric {
+  font-weight: bold ;
+  font-size: larger ;
+  color: maroon ;
+  text-align: center }
+
+p.sidebar-title {
+  font-family: sans-serif ;
+  font-weight: bold ;
+  font-size: larger }
+
+p.sidebar-subtitle {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+p.topic-title {
+  font-weight: bold }
+
+pre.address {
+  margin-bottom: 0 ;
+  margin-top: 0 ;
+  font: inherit }
+
+pre.literal-block, pre.doctest-block, pre.math, pre.code {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+pre.code .ln { color: grey; } /* line numbers */
+pre.code, code { background-color: #eeeeee }
+pre.code .comment, code .comment { color: #5C6576 }
+pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
+pre.code .literal.string, code .literal.string { color: #0C5404 }
+pre.code .name.builtin, code .name.builtin { color: #352B84 }
+pre.code .deleted, code .deleted { background-color: #DEB0A1}
+pre.code .inserted, code .inserted { background-color: #A3D289}
+
+span.classifier {
+  font-family: sans-serif ;
+  font-style: oblique }
+
+span.classifier-delimiter {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+span.interpreted {
+  font-family: sans-serif }
+
+span.option {
+  white-space: nowrap }
+
+span.pre {
+  white-space: pre }
+
+span.problematic {
+  color: red }
+
+span.section-subtitle {
+  /* font-size relative to parent (h1..h6 element) */
+  font-size: 80% }
+
+table.citation {
+  border-left: solid 1px gray;
+  margin-left: 1px }
+
+table.docinfo {
+  margin: 2em 4em }
+
+table.docutils {
+  margin-top: 0.5em ;
+  margin-bottom: 0.5em }
+
+table.footnote {
+  border-left: solid 1px black;
+  margin-left: 1px }
+
+table.docutils td, table.docutils th,
+table.docinfo td, table.docinfo th {
+  padding-left: 0.5em ;
+  padding-right: 0.5em ;
+  vertical-align: top }
+
+table.docutils th.field-name, table.docinfo th.docinfo-name {
+  font-weight: bold ;
+  text-align: left ;
+  white-space: nowrap ;
+  padding-left: 0 }
+
+/* "booktabs" style (no vertical lines) */
+table.docutils.booktabs {
+  border: 0px;
+  border-top: 2px solid;
+  border-bottom: 2px solid;
+  border-collapse: collapse;
+}
+table.docutils.booktabs * {
+  border: 0px;
+}
+table.docutils.booktabs th {
+  border-bottom: thin solid;
+  text-align: left;
+}
+
+h1 tt.docutils, h2 tt.docutils, h3 tt.docutils,
+h4 tt.docutils, h5 tt.docutils, h6 tt.docutils {
+  font-size: 100% }
+
+ul.auto-toc {
+  list-style-type: none }
+
+</style>
+</head>
+<body>
+<div class="document" id="timesheet-begin-end-hours-with-empty-end">
+<h1 class="title">Timesheet - Begin/End Hours with empty end</h1>
+
+<!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! This file is generated by oca-gen-addon-readme !!
+!! changes will be overwritten.                   !!
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! source digest: sha256:6f9bf57eb39a36725b852081172b71f26e16fed507227ba22b894d0f1ed0d8e5
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/timesheet/tree/16.0/hr_timesheet_begin_end_no_end"><img alt="OCA/timesheet" src="https://img.shields.io/badge/github-OCA%2Ftimesheet-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/timesheet-16-0/timesheet-16-0-hr_timesheet_begin_end_no_end"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/timesheet&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p>Allow to insert begin hour without end hour on the timesheet. The benefit of
+doing this is that you can save your begin hour at the start of your day/task,
+and come back later to save the end hour.</p>
+<p><strong>Table of contents</strong></p>
+<div class="contents local topic" id="contents">
+<ul class="simple">
+<li><a class="reference internal" href="#bug-tracker" id="toc-entry-1">Bug Tracker</a></li>
+<li><a class="reference internal" href="#credits" id="toc-entry-2">Credits</a><ul>
+<li><a class="reference internal" href="#authors" id="toc-entry-3">Authors</a></li>
+<li><a class="reference internal" href="#contributors" id="toc-entry-4">Contributors</a></li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-5">Maintainers</a></li>
+</ul>
+</li>
+</ul>
+</div>
+<div class="section" id="bug-tracker">
+<h1><a class="toc-backref" href="#toc-entry-1">Bug Tracker</a></h1>
+<p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/timesheet/issues">GitHub Issues</a>.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+<a class="reference external" href="https://github.com/OCA/timesheet/issues/new?body=module:%20hr_timesheet_begin_end_no_end%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<p>Do not contact contributors directly about support or help with technical issues.</p>
+</div>
+<div class="section" id="credits">
+<h1><a class="toc-backref" href="#toc-entry-2">Credits</a></h1>
+<div class="section" id="authors">
+<h2><a class="toc-backref" href="#toc-entry-3">Authors</a></h2>
+<ul class="simple">
+<li>Coop IT Easy SC</li>
+</ul>
+</div>
+<div class="section" id="contributors">
+<h2><a class="toc-backref" href="#toc-entry-4">Contributors</a></h2>
+<ul class="simple">
+<li><a class="reference external" href="https://coopiteasy.be">Coop IT Easy SC</a>:<ul>
+<li>Carmen Bianca BAKKER</li>
+</ul>
+</li>
+</ul>
+</div>
+<div class="section" id="maintainers">
+<h2><a class="toc-backref" href="#toc-entry-5">Maintainers</a></h2>
+<p>This module is maintained by the OCA.</p>
+<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
+<p>OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.</p>
+<p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainer</a>:</p>
+<p><a class="reference external image-reference" href="https://github.com/carmenbianca"><img alt="carmenbianca" src="https://github.com/carmenbianca.png?size=40px" /></a></p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/timesheet/tree/16.0/hr_timesheet_begin_end_no_end">OCA/timesheet</a> project on GitHub.</p>
+<p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
+</div>
+</div>
+</div>
+</body>
+</html>
diff --git a/hr_timesheet_begin_end_no_end/tests/__init__.py b/hr_timesheet_begin_end_no_end/tests/__init__.py
new file mode 100644
index 0000000000..eae8ab8978
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/tests/__init__.py
@@ -0,0 +1,5 @@
+# SPDX-FileCopyrightText: 2024 Coop IT Easy SC
+#
+# SPDX-License-Identifier: AGPL-3.0-or-later
+
+from . import test_timesheet_begin_end_no_end
diff --git a/hr_timesheet_begin_end_no_end/tests/test_timesheet_begin_end_no_end.py b/hr_timesheet_begin_end_no_end/tests/test_timesheet_begin_end_no_end.py
new file mode 100644
index 0000000000..0274dee6e0
--- /dev/null
+++ b/hr_timesheet_begin_end_no_end/tests/test_timesheet_begin_end_no_end.py
@@ -0,0 +1,53 @@
+# SPDX-FileCopyrightText: 2024 Coop IT Easy SC
+#
+# SPDX-License-Identifier: AGPL-3.0-or-later
+
+from odoo import exceptions, fields
+from odoo.tests.common import TransactionCase
+
+
+class TestBeginEndNoEnd(TransactionCase):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.project = cls.env.ref("project.project_project_1")
+        cls.employee = cls.env.ref("hr.employee_qdp")
+
+    def base_line(self):
+        return {
+            "name": "test",
+            "date": fields.Date.today(),
+            "time_start": 10.0,
+            "time_stop": 12.0,
+            "project_id": self.project.id,
+            "employee_id": self.employee.id,
+        }
+
+    def test_check_end_is_zero(self):
+        line = self.base_line()
+        line.update({"time_stop": 0})
+        # No error.
+        self.env["account.analytic.line"].create(line)
+
+    def test_check_end_is_not_zero(self):
+        line = self.base_line()
+        line_record = self.env["account.analytic.line"].create(line)
+        with self.assertRaises(exceptions.ValidationError):
+            line_record.time_stop = 1
+
+    def test_check_end_is_not_zero_no_record(self):
+        line = self.base_line()
+        line.update({"time_stop": 1})
+        with self.assertRaises(exceptions.ValidationError):
+            self.env["account.analytic.line"].create(line)
+
+    def test_no_overlap(self):
+        line_1 = self.base_line()
+        line_1.update({"time_stop": 0, "unit_amount": 0})
+        line_2 = line_1.copy()
+        line_2.update({"time_start": 11.0})
+        line_3 = self.base_line()
+        # Able to create all without overlap, because time_stop is 0 for two of
+        # them, and they don't count, even though their start times are
+        # overlapped with line_3.
+        self.env["account.analytic.line"].create([line_1, line_2, line_3])
diff --git a/setup/hr_timesheet_begin_end_no_end/odoo/addons/hr_timesheet_begin_end_no_end b/setup/hr_timesheet_begin_end_no_end/odoo/addons/hr_timesheet_begin_end_no_end
new file mode 120000
index 0000000000..3bde95d631
--- /dev/null
+++ b/setup/hr_timesheet_begin_end_no_end/odoo/addons/hr_timesheet_begin_end_no_end
@@ -0,0 +1 @@
+../../../../hr_timesheet_begin_end_no_end
\ No newline at end of file
diff --git a/setup/hr_timesheet_begin_end_no_end/setup.py b/setup/hr_timesheet_begin_end_no_end/setup.py
new file mode 100644
index 0000000000..28c57bb640
--- /dev/null
+++ b/setup/hr_timesheet_begin_end_no_end/setup.py
@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)

From 41d45b32b5b518a6d15fe741f5b718c627899758 Mon Sep 17 00:00:00 2001
From: Carmen Bianca BAKKER <carmen@coopiteasy.be>
Date: Fri, 28 Jun 2024 14:58:11 +0200
Subject: [PATCH 6/6] [FIX] hr_timesheet_begin_end_no_end: Added as rebel
 module

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>
---
 .copier-answers.yml        |  1 +
 .github/workflows/test.yml | 11 +++++++++--
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/.copier-answers.yml b/.copier-answers.yml
index e3f412721a..d9fa087bf5 100644
--- a/.copier-answers.yml
+++ b/.copier-answers.yml
@@ -18,6 +18,7 @@ org_slug: OCA
 rebel_module_groups:
 - sale_timesheet_rounded
 - hr_timesheet_begin_end
+- hr_timesheet_begin_end_no_end
 repo_description: 'TODO: add repo description.'
 repo_name: timesheet
 repo_slug: timesheet
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 9f621e11c4..ada789fb60 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -50,10 +50,17 @@ jobs:
             name: test with OCB
             makepot: "true"
           - container: ghcr.io/oca/oca-ci/py3.10-odoo16.0:latest
-            exclude: "sale_timesheet_rounded,hr_timesheet_begin_end"
+            include: "hr_timesheet_begin_end_no_end"
             name: test with Odoo
           - container: ghcr.io/oca/oca-ci/py3.10-ocb16.0:latest
-            exclude: "sale_timesheet_rounded,hr_timesheet_begin_end"
+            include: "hr_timesheet_begin_end_no_end"
+            name: test with OCB
+            makepot: "true"
+          - container: ghcr.io/oca/oca-ci/py3.10-odoo16.0:latest
+            exclude: "sale_timesheet_rounded,hr_timesheet_begin_end,hr_timesheet_begin_end_no_end"
+            name: test with Odoo
+          - container: ghcr.io/oca/oca-ci/py3.10-ocb16.0:latest
+            exclude: "sale_timesheet_rounded,hr_timesheet_begin_end,hr_timesheet_begin_end_no_end"
             name: test with OCB
             makepot: "true"
     services: