diff --git a/agreement_legal/models/agreement.py b/agreement_legal/models/agreement.py
index b2c99129a..acde0f53a 100644
--- a/agreement_legal/models/agreement.py
+++ b/agreement_legal/models/agreement.py
@@ -225,7 +225,6 @@ class Agreement(models.Model):
         "parent_agreement_id",
         string="Previous Versions",
         copy=False,
-        domain=[("active", "=", False)],
         context={"active_test": False},
     )
     child_agreements_ids = fields.One2many(
@@ -408,8 +407,7 @@ class Agreement(models.Model):
                     "date_created": fields.Datetime.now(),
                 }
             )
-            # Reset revision to 0 since it's a new version
-            rec.revision = 0
+        return super().write({"revision": 0})
 
     def _get_new_agreement_default_vals(self):
         self.ensure_one()
diff --git a/agreement_legal/tests/test_agreement.py b/agreement_legal/tests/test_agreement.py
index 4b6cb6cd0..93c9465d9 100644
--- a/agreement_legal/tests/test_agreement.py
+++ b/agreement_legal/tests/test_agreement.py
@@ -143,3 +143,8 @@ class TestAgreement(TransactionCase):
         self.assertEqual(
             field[0].get("modifiers", ""), '{"readonly": [["readonly", "=", true]]}'
         )
+
+    def test_action_create_new_version(self):
+        self.test_agreement.create_new_version()
+        self.assertEqual(self.test_agreement.state, "draft")
+        self.assertEqual(len(self.test_agreement.previous_version_agreements_ids), 1)