From 72869bbfaa2f531f97c92e2ad2f6fdf6d3904180 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Thu, 20 Aug 2026 11:49:07 +0200
Subject: [PATCH 01/14] [IMP] git: git challenge completed!

---
 git-challenge.txt | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)
 create mode 100644 git-challenge.txt

diff --git a/git-challenge.txt b/git-challenge.txt
new file mode 100644
index 00000000000..799c7e11c12
--- /dev/null
+++ b/git-challenge.txt
@@ -0,0 +1,29 @@
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⣀⣤⣴⣶⡾⠿⠛⠻⠶⣦⣄⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢠⡞⠉⢠⡿⠁⠀⠀⣰⡾⠓⠂⠉⢻⣦⡀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢠⡟⠀⣰⡿⠀⠀⠀⣼⠏⠀⠀⠀⢀⣠⣿⣿⣦⡀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⣾⣧⣶⣿⣇⠀⢀⣼⡏⠀⠀⠀⣰⣿⣿⢻⣿⣿⣿⣦⡀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢠⣶⣶⡄⠀⠀⠀⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢸⣿⣿⣿⣿⣿⣶⣾⣿⠁⠀⠀⠀⣠⣾⠏⢸⣿⣿⣿⣿⣿⣶⣶⣶⣶⣶⣶⠶⠚⠛⠛⠛⠻⠾⠿⢿⣇⡀⠀⠀⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢿⣿⣿⣿⣿⣿⣿⡿⢷⣤⣤⣾⡿⠋⠀⣸⣿⣿⣿⣿⣿⣿⣿⣿⣿⡟⠁⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠉⠳⢦⡀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠘⢿⣿⣿⣿⣿⣿⣿⡀⠉⠉⠉⠀⢀⣴⣿⣿⣿⣿⣿⣿⣿⣿⣿⡿⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠙⢷⣄⣀⣀⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠈⠻⣿⣿⣿⣿⣿⣿⣷⣶⣶⣾⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⡇⠀⠀⠀⠀⠀⣰⣶⣦⣄⠀⠀⠀⠀⠀⠀⠀⠀⠙⣿⣿⠇
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠈⠛⢿⣿⣿⣿⣿⣿⣿⣿⣿⣿⡿⣫⣿⣿⣿⣿⣿⣿⣿⡇⠀⠀⠀⠀⠀⢻⣿⣟⢹⣆⠀⠀⠀⠀⠀⠀⠀⠀⠘⣧⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢀⣽⣿⣿⣿⣿⠿⠟⣋⣥⣾⣿⣿⣿⣿⣿⣿⣿⣿⡇⠀⠀⠀⠀⠀⠀⠙⠿⠮⢭⣆⠀⠀⠀⠀⠀⠀⠀⠀⢹⡀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢀⣴⠟⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⡇⠀⠀⠀⠀⠀⡇⠀⢀⣄⠀⠀⠀⠀⣠⣄⡀⠀⠀⠀⠘⡇
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢀⣴⡟⠁⠀⢻⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣷⠀⠀⠀⠀⠀⣧⠀⠸⣿⣇⠀⠀⠀⢱⡛⠉⣿⣦⠀⢠⡇
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⣠⣿⠋⠀⠀⠀⠘⢿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣧⠀⠀⠀⠀⠸⣷⡀⠻⣿⣷⣦⠄⠸⣿⣿⣿⡿⢀⡾⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢀⣼⠟⠁⠀⠀⠀⠀⠀⠈⠻⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣧⡀⠀⠀⠀⠈⠛⠷⠦⠭⠥⠀⠀⠈⠉⢁⣰⠟⠀⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢀⣴⡾⠋⠀⠀⠀⠀⠀⠀⠀⠀⠀⠈⠙⠿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣦⣄⡀⠀⠀⠀⠀⠀⠀⠀⣀⣤⠶⠋⠁⠀⠀⠀
+⠀⠀⠀⠀⠀⠀⠀⠀⢀⣴⡿⠋⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠈⠙⠛⠿⢿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣶⣶⣶⠶⠶⠛⠉⠀⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⠀⠀⢠⣾⠏⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠈⠉⠛⠿⣿⣿⠿⣿⣿⣿⣿⡿⣫⣾⣷⣶⣶⡖⠛⠓⡆⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⠀⢠⣋⡏⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠉⠛⢷⣯⡍⠉⣰⣿⡿⠿⣿⠟⢳⡾⠁⢳⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⢠⡾⣿⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠙⢿⣼⣿⣿⠀⠀⣹⠀⠀⢷⡀⣸⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⡰⢻⣴⢿⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⣼⣿⣿⣿⣤⣴⣿⣤⣤⣾⠿⠋⠀⠀⠀⠀⠀⠀
+⠀⠀⢠⠞⠁⢸⣇⣾⡆⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⣼⡿⠿⠿⠟⠛⠋⠉⠉⠁⠀⠀⠀⠀⠀⠀⠀⠀⠀
+⠀⣴⡏⠀⠀⠀⢿⠃⣿⡄⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⣀⣀⣀⣀⣀⣀⣀⣤⡾⠃⠀⠀⠀⠀⠀⣀⣤⣤⣤⣀⡀⠀⠀⠀⠀⠀⠀⠀
+⣸⣿⣧⠀⠀⠀⠘⢷⡏⣿⣄⡀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢀⣀⣤⣶⣿⣿⡉⠥⠿⠛⠒⠚⣻⣄⣀⣀⠀⣠⣤⣾⣏⢉⡏⠀⠈⣿⣦⡀⠀⠀⠀⠀⠀
+⣿⣿⣿⣆⠀⠀⠀⠈⠻⢿⡍⢻⡦⣤⣀⣀⠀⠀⠀⠀⣀⣀⣠⣤⣤⠶⣾⣯⣡⠔⠋⠉⠀⢀⣠⣴⣶⣿⠟⠉⠀⣨⡿⢋⣽⣿⡿⠟⠛⠛⠶⣾⠁⠀⢱⠀⠀⠀⠀⠀
+⢻⣿⣿⣿⣦⠀⠀⠀⠀⠀⠉⠚⠷⢤⣿⣿⣉⣹⣿⣏⠙⣿⣅⣽⡷⠴⠛⠉⠀⠀⠀⢠⣾⣿⣿⣿⣿⠏⠀⠀⣼⡟⠀⣾⣿⠋⠀⠀⠀⠀⠀⠈⢳⡖⢻⠀⠀⠀⠀⠀
+⠈⢻⣿⣿⣿⣷⡀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠉⠉⠉⠉⠉⠁⠀⠀⠀⠀⠀⠀⠀⠀⢠⣿⣿⣿⣿⣿⣿⠀⠀⢠⣿⡇⠀⣿⡇⠀⠀⠀⠀⠀⠀⠀⢸⣷⡞⠀⠀⠀⠀⠀
+⠀⠀⢹⣿⣿⣿⣿⣶⣄⡀⢀⡴⣶⣶⣶⣶⣤⣤⣤⣀⡀⠀⠀⠀⠀⠀⠀⠀⠀⠀⢸⣿⣿⣿⣿⣿⣿⠀⠀⢸⣿⣇⠀⢿⣧⡀⠀⠀⠀⠀⠀⢀⣾⡟⠁⠀⠀⠀⠀⠀
+⠀⠀⠀⠻⣿⣿⣿⣿⣿⠿⠋⠀⠀⠀⠀⠀⠉⠉⠙⠛⠛⠿⠷⢶⣦⣤⣀⡀⠀⠀⠀⢻⣿⣿⣿⣿⣿⣆⠀⠈⢿⣿⣦⡀⠻⢿⣦⣤⣤⣤⣶⡿⠋⠀⠀⠀⠀⠀⠀⠀
+⠀⠀⠀⠀⠀⠉⠉⠉⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠉⠉⠛⠻⠿⠿⢿⣿⣿⣿⣿⣿⠦⠤⠤⢙⣻⣿⡦⠤⠽⠛⠛⠛⠉⠀⠀⠀⠀⠀⠀⠀⠀⠀

From f1c66d24d650c15970d35c6c66ee7c44de3e6f27 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Mon, 17 Aug 2026 14:13:41 +0200
Subject: [PATCH 02/14] [ADD] estate: init

---
 estate/__init__.py     |  0
 estate/__manifest__.py | 11 +++++++++++
 2 files changed, 11 insertions(+)
 create mode 100644 estate/__init__.py
 create mode 100644 estate/__manifest__.py

diff --git a/estate/__init__.py b/estate/__init__.py
new file mode 100644
index 00000000000..e69de29bb2d
diff --git a/estate/__manifest__.py b/estate/__manifest__.py
new file mode 100644
index 00000000000..de284a8292c
--- /dev/null
+++ b/estate/__manifest__.py
@@ -0,0 +1,11 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+
+{
+    "name": "estate",
+    "application": True,
+    "depends": [
+        "base",
+    ],
+}

From 2f0f8cce12505c2083b0d1671621d96f2f7fb138 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Mon, 17 Aug 2026 14:56:28 +0200
Subject: [PATCH 03/14] [IMP] estate: chapter 3

---
 estate/__init__.py               |  4 ++++
 estate/models/__init__.py        |  1 +
 estate/models/estate_property.py | 22 ++++++++++++++++++++++
 3 files changed, 27 insertions(+)
 create mode 100644 estate/models/__init__.py
 create mode 100644 estate/models/estate_property.py

diff --git a/estate/__init__.py b/estate/__init__.py
index e69de29bb2d..dc5e6b693d1 100644
--- a/estate/__init__.py
+++ b/estate/__init__.py
@@ -0,0 +1,4 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+from . import models
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
new file mode 100644
index 00000000000..5e1963c9d2f
--- /dev/null
+++ b/estate/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_property
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
new file mode 100644
index 00000000000..066e93676a7
--- /dev/null
+++ b/estate/models/estate_property.py
@@ -0,0 +1,22 @@
+from odoo import fields, models
+
+
+class EstateProperty(models.Model):
+    _name = "estate.property"
+    _description = "An estate property"
+
+    name = fields.Char(required=True)
+    description = fields.Text()
+    postcode = fields.Char()
+    date_availability = fields.Date()
+    expected_price = fields.Float(required=True)
+    selling_price = fields.Float()
+    bedrooms = fields.Integer()
+    living_area = fields.Integer()
+    facades = fields.Integer()
+    garage = fields.Boolean()
+    garden = fields.Boolean()
+    garden_area = fields.Boolean()
+    garden_orientation = fields.Selection(
+        [("north", "North"), ("south", "South"), ("east", "East"), ("west", "West")]
+    )

From c9fd1508bc66ba2d08edbced8c5750303a764572 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Mon, 17 Aug 2026 15:39:59 +0200
Subject: [PATCH 04/14] [IMP] estate: chapter 4

---
 estate/__manifest__.py              | 3 +++
 estate/security/ir.model.access.csv | 2 ++
 2 files changed, 5 insertions(+)
 create mode 100644 estate/security/ir.model.access.csv

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index de284a8292c..16615ed8997 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -8,4 +8,7 @@
     "depends": [
         "base",
     ],
+    "data": [
+        "security/ir.model.access.csv",
+    ],
 }
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
new file mode 100644
index 00000000000..98f4671fb0d
--- /dev/null
+++ b/estate/security/ir.model.access.csv
@@ -0,0 +1,2 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+estate.access_estate_property,access_estate_property,estate.model_estate_property,base.group_user,1,1,1,1

From fb75f1ca9d67b610af1b6aeb22253a66e5493f46 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Mon, 17 Aug 2026 16:49:52 +0200
Subject: [PATCH 05/14] [IMP] estate: chapter 5

---
 estate/__manifest__.py                 |  2 ++
 estate/models/estate_property.py       | 22 +++++++++++++++++++---
 estate/views/estate_menus.xml          |  8 ++++++++
 estate/views/estate_property_views.xml |  8 ++++++++
 4 files changed, 37 insertions(+), 3 deletions(-)
 create mode 100644 estate/views/estate_menus.xml
 create mode 100644 estate/views/estate_property_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 16615ed8997..7fd887aab61 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -10,5 +10,7 @@
     ],
     "data": [
         "security/ir.model.access.csv",
+        "views/estate_property_views.xml",
+        "views/estate_menus.xml",
     ],
 }
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 066e93676a7..dec86c0f4ea 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,5 @@
 from odoo import fields, models
+from odoo.tools.date_utils import add
 
 
 class EstateProperty(models.Model):
@@ -8,10 +9,12 @@ class EstateProperty(models.Model):
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()
-    date_availability = fields.Date()
+    date_availability = fields.Date(
+        copy=False, default=lambda _: add(fields.Date.today(), months=+3)
+    )
     expected_price = fields.Float(required=True)
-    selling_price = fields.Float()
-    bedrooms = fields.Integer()
+    selling_price = fields.Float(copy=False, readonly=True)
+    bedrooms = fields.Integer(default=2)
     living_area = fields.Integer()
     facades = fields.Integer()
     garage = fields.Boolean()
@@ -20,3 +23,16 @@ class EstateProperty(models.Model):
     garden_orientation = fields.Selection(
         [("north", "North"), ("south", "South"), ("east", "East"), ("west", "West")]
     )
+    active = fields.Boolean(default=True)
+    state = fields.Selection(
+        [
+            ("new", "New"),
+            ("received", "Offer Received"),
+            ("accepted", "Offer Accepted"),
+            ("sold", "Sold"),
+            ("cancelled", "Cancelled"),
+        ],
+        required=True,
+        copy=False,
+        default="new",
+    )
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
new file mode 100644
index 00000000000..c6cd401b4a8
--- /dev/null
+++ b/estate/views/estate_menus.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+    <menuitem id="estate_menu_root" name="Real Estate">
+        <menuitem id="estate_first_level_menu" name="Advertisements">
+            <menuitem id="estate_model_menu_action" action="estate_model_action"/>
+        </menuitem>
+    </menuitem>
+</odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
new file mode 100644
index 00000000000..420bdc65197
--- /dev/null
+++ b/estate/views/estate_property_views.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_model_action" model="ir.actions.act_window">
+        <field name="name">Properties</field>
+        <field name="res_model">estate.property</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>

From 03075f51238414ab44eb556c49c3284a7c02ea83 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Tue, 18 Aug 2026 10:14:26 +0200
Subject: [PATCH 06/14] [IMP] estate: chapter 6

---
 estate/__manifest__.py           |  2 ++
 estate/models/estate_property.py |  2 +-
 estate/views/estate_form.xml     | 38 ++++++++++++++++++++++++++++++++
 estate/views/estate_list.xml     | 34 ++++++++++++++++++++++++++++
 4 files changed, 75 insertions(+), 1 deletion(-)
 create mode 100644 estate/views/estate_form.xml
 create mode 100644 estate/views/estate_list.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 7fd887aab61..312c426c783 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -11,6 +11,8 @@
     "data": [
         "security/ir.model.access.csv",
         "views/estate_property_views.xml",
+        "views/estate_list.xml",
+        "views/estate_form.xml",
         "views/estate_menus.xml",
     ],
 }
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index dec86c0f4ea..3301229596a 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -19,7 +19,7 @@ class EstateProperty(models.Model):
     facades = fields.Integer()
     garage = fields.Boolean()
     garden = fields.Boolean()
-    garden_area = fields.Boolean()
+    garden_area = fields.Integer()
     garden_orientation = fields.Selection(
         [("north", "North"), ("south", "South"), ("east", "East"), ("west", "West")]
     )
diff --git a/estate/views/estate_form.xml b/estate/views/estate_form.xml
new file mode 100644
index 00000000000..5bfca1003a0
--- /dev/null
+++ b/estate/views/estate_form.xml
@@ -0,0 +1,38 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_view_form" model="ir.ui.view">
+        <field name="name">Properties</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <form string="estate">
+                <sheet>
+                    <field name="name" string="" required="True" placeholder="My new house" class="fs-1 mb-5" />
+                    <group>
+                        <group>
+                            <field name="postcode" />
+                            <field name="date_availability" string="Available From" />
+                        </group>
+                        <group>
+                            <field name="expected_price" required="True" />
+                            <field name="selling_price" />
+                        </group>
+                    </group>
+                    <notebook>
+                        <page string="Description">
+                            <group>
+                                <field name="description" />
+                                <field name="bedrooms" />
+                                <field name="living_area" string="Living Area (sqm)" />
+                                <field name="facades" />
+                                <field name="garage" />
+                                <field name="garden" />
+                                <field name="garden_area" string="Garden Area (sqm)" />
+                                <field name="garden_orientation" />
+                            </group>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_list.xml b/estate/views/estate_list.xml
new file mode 100644
index 00000000000..aee0471fa54
--- /dev/null
+++ b/estate/views/estate_list.xml
@@ -0,0 +1,34 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_view_search" model="ir.ui.view">
+        <field name="name">Search</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <search string="Delivery Carrier">
+                <field name="name" string="Estate" />
+                <field name="selling_price"/>
+                <separator/>
+                <filter string="Available" name="available" domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]"/>
+                <group>
+                    <filter string="Postcode" name="postcode" context="{'group_by':'postcode', 'residual_visible':True}"/>
+                </group>
+            </search>
+        </field>
+    </record>
+
+    <record id="estate_view_tree" model="ir.ui.view">
+        <field name="name">Properties</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <list string="estate">
+                <field name="name" />
+                <field name="postcode" />
+                <field name="bedrooms" />
+                <field name="living_area" />
+                <field name="expected_price" />
+                <field name="selling_price" />
+                <field name="date_availability" />
+            </list>
+        </field>
+    </record>
+</odoo>

From 5cd1ce7a2f1796d33081a1d21ba350bee42facf2 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Tue, 18 Aug 2026 11:55:02 +0200
Subject: [PATCH 07/14] [IMP] estate: chapter 7

---
 estate/__manifest__.py                 |  5 +-
 estate/models/__init__.py              |  3 +
 estate/models/estate_offer.py          | 15 +++++
 estate/models/estate_property.py       | 16 +++++
 estate/models/estate_tag.py            |  8 +++
 estate/models/estate_type.py           |  8 +++
 estate/security/ir.model.access.csv    |  3 +
 estate/views/estate_form.xml           | 38 ------------
 estate/views/estate_list.xml           | 34 -----------
 estate/views/estate_menus.xml          |  8 ++-
 estate/views/estate_offer_views.xml    | 31 ++++++++++
 estate/views/estate_property_views.xml | 84 +++++++++++++++++++++++++-
 estate/views/estate_tag_views.xml      |  8 +++
 estate/views/estate_type_views.xml     |  8 +++
 14 files changed, 192 insertions(+), 77 deletions(-)
 create mode 100644 estate/models/estate_offer.py
 create mode 100644 estate/models/estate_tag.py
 create mode 100644 estate/models/estate_type.py
 delete mode 100644 estate/views/estate_form.xml
 delete mode 100644 estate/views/estate_list.xml
 create mode 100644 estate/views/estate_offer_views.xml
 create mode 100644 estate/views/estate_tag_views.xml
 create mode 100644 estate/views/estate_type_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 312c426c783..73e0e310abe 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -11,8 +11,9 @@
     "data": [
         "security/ir.model.access.csv",
         "views/estate_property_views.xml",
-        "views/estate_list.xml",
-        "views/estate_form.xml",
+        "views/estate_type_views.xml",
+        "views/estate_tag_views.xml",
+        "views/estate_offer_views.xml",
         "views/estate_menus.xml",
     ],
 }
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 5e1963c9d2f..8d8063038f5 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1 +1,4 @@
 from . import estate_property
+from . import estate_type
+from . import estate_tag
+from . import estate_offer
diff --git a/estate/models/estate_offer.py b/estate/models/estate_offer.py
new file mode 100644
index 00000000000..9e1194a45a9
--- /dev/null
+++ b/estate/models/estate_offer.py
@@ -0,0 +1,15 @@
+from odoo import fields, models
+
+
+class EstateOffer(models.Model):
+    _name = "estate.offer"
+    _description = "An estate offer"
+
+    price = fields.Float()
+    status = fields.Selection(
+        [("accepted", "Accepted"), ("refused", "Refused")], copy=False
+    )
+
+    # Foreign fields
+    partner_id = fields.Many2one("res.partner", required=True)
+    property_id = fields.Many2one("estate.property", required=True)
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 3301229596a..2c27050a437 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -36,3 +36,19 @@ class EstateProperty(models.Model):
         copy=False,
         default="new",
     )
+
+    # Foreign fields
+    property_type_id = fields.Many2one(
+        "estate.type", required=True, ondelete="restrict"
+    )
+    buyer_id = fields.Many2one(
+        "res.partner", required=False, copy=False, ondelete="restrict"
+    )
+    seller_id = fields.Many2one(
+        "res.users",
+        required=True,
+        default=lambda self: self.env.user,
+        ondelete="restrict",
+    )
+    tag_ids = fields.Many2many("estate.tag")
+    offer_ids = fields.One2many("estate.offer", "property_id")
diff --git a/estate/models/estate_tag.py b/estate/models/estate_tag.py
new file mode 100644
index 00000000000..5ce1c31faf6
--- /dev/null
+++ b/estate/models/estate_tag.py
@@ -0,0 +1,8 @@
+from odoo import fields, models
+
+
+class EstateTag(models.Model):
+    _name = "estate.tag"
+    _description = "An estate tag"
+
+    name = fields.Char(required=True)
diff --git a/estate/models/estate_type.py b/estate/models/estate_type.py
new file mode 100644
index 00000000000..848488dfb59
--- /dev/null
+++ b/estate/models/estate_type.py
@@ -0,0 +1,8 @@
+from odoo import fields, models
+
+
+class EstateType(models.Model):
+    _name = "estate.type"
+    _description = "An estate type"
+
+    name = fields.Char(required=True)
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index 98f4671fb0d..7aa3583474b 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -1,2 +1,5 @@
 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
 estate.access_estate_property,access_estate_property,estate.model_estate_property,base.group_user,1,1,1,1
+estate.access_estate_type,access_estate_type,estate.model_estate_type,base.group_user,1,1,1,1
+estate.access_estate_tag,access_estate_tag,estate.model_estate_tag,base.group_user,1,1,1,1
+estate.access_estate_offer,access_estate_offer,estate.model_estate_offer,base.group_user,1,1,1,1
diff --git a/estate/views/estate_form.xml b/estate/views/estate_form.xml
deleted file mode 100644
index 5bfca1003a0..00000000000
--- a/estate/views/estate_form.xml
+++ /dev/null
@@ -1,38 +0,0 @@
-<?xml version="1.0"?>
-<odoo>
-    <record id="estate_view_form" model="ir.ui.view">
-        <field name="name">Properties</field>
-        <field name="model">estate.property</field>
-        <field name="arch" type="xml">
-            <form string="estate">
-                <sheet>
-                    <field name="name" string="" required="True" placeholder="My new house" class="fs-1 mb-5" />
-                    <group>
-                        <group>
-                            <field name="postcode" />
-                            <field name="date_availability" string="Available From" />
-                        </group>
-                        <group>
-                            <field name="expected_price" required="True" />
-                            <field name="selling_price" />
-                        </group>
-                    </group>
-                    <notebook>
-                        <page string="Description">
-                            <group>
-                                <field name="description" />
-                                <field name="bedrooms" />
-                                <field name="living_area" string="Living Area (sqm)" />
-                                <field name="facades" />
-                                <field name="garage" />
-                                <field name="garden" />
-                                <field name="garden_area" string="Garden Area (sqm)" />
-                                <field name="garden_orientation" />
-                            </group>
-                        </page>
-                    </notebook>
-                </sheet>
-            </form>
-        </field>
-    </record>
-</odoo>
diff --git a/estate/views/estate_list.xml b/estate/views/estate_list.xml
deleted file mode 100644
index aee0471fa54..00000000000
--- a/estate/views/estate_list.xml
+++ /dev/null
@@ -1,34 +0,0 @@
-<?xml version="1.0"?>
-<odoo>
-    <record id="estate_view_search" model="ir.ui.view">
-        <field name="name">Search</field>
-        <field name="model">estate.property</field>
-        <field name="arch" type="xml">
-            <search string="Delivery Carrier">
-                <field name="name" string="Estate" />
-                <field name="selling_price"/>
-                <separator/>
-                <filter string="Available" name="available" domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]"/>
-                <group>
-                    <filter string="Postcode" name="postcode" context="{'group_by':'postcode', 'residual_visible':True}"/>
-                </group>
-            </search>
-        </field>
-    </record>
-
-    <record id="estate_view_tree" model="ir.ui.view">
-        <field name="name">Properties</field>
-        <field name="model">estate.property</field>
-        <field name="arch" type="xml">
-            <list string="estate">
-                <field name="name" />
-                <field name="postcode" />
-                <field name="bedrooms" />
-                <field name="living_area" />
-                <field name="expected_price" />
-                <field name="selling_price" />
-                <field name="date_availability" />
-            </list>
-        </field>
-    </record>
-</odoo>
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index c6cd401b4a8..b3f9748ea5b 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -1,8 +1,12 @@
 <?xml version="1.0"?>
 <odoo>
     <menuitem id="estate_menu_root" name="Real Estate">
-        <menuitem id="estate_first_level_menu" name="Advertisements">
-            <menuitem id="estate_model_menu_action" action="estate_model_action"/>
+        <menuitem id="estate_first_menu_advertisements" name="Advertisements">
+            <menuitem id="estate_second_advertisement_advertisements" action="estate_property_action"/>
+        </menuitem>
+        <menuitem id="estate_first_menu_settings" name="Settings">
+            <menuitem id="estate_second_settings_types" action="estate_type_action"/>
+            <menuitem id="estate_second_settings_tags" action="estate_tag_action"/>
         </menuitem>
     </menuitem>
 </odoo>
diff --git a/estate/views/estate_offer_views.xml b/estate/views/estate_offer_views.xml
new file mode 100644
index 00000000000..4e985114e9b
--- /dev/null
+++ b/estate/views/estate_offer_views.xml
@@ -0,0 +1,31 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_offer_list" model="ir.ui.view">
+        <field name="name">Offers</field>
+        <field name="model">estate.offer</field>
+        <field name="arch" type="xml">
+            <list>
+                <field name="price" />
+                <field name="partner_id" />
+                <field name="status" />
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_offer_form" model="ir.ui.view">
+        <field name="name">Offers</field>
+        <field name="model">estate.offer</field>
+        <field name="arch" type="xml">
+            <form>
+                <sheet>
+                    <group>
+                        <field name="price" />
+                        <field name="partner_id" />
+                        <field name="status" />
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+</odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 420bdc65197..46a92a276c9 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -1,8 +1,90 @@
 <?xml version="1.0"?>
 <odoo>
-    <record id="estate_model_action" model="ir.actions.act_window">
+    <record id="estate_property_action" model="ir.actions.act_window">
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
     </record>
+
+    <record id="estate_property_search" model="ir.ui.view">
+        <field name="name">Search</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <search string="Delivery Carrier">
+                <field name="name" string="Estate" />
+                <field name="selling_price"/>
+                <separator/>
+                <filter string="Available" name="available" domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]"/>
+                <group>
+                    <filter string="Postcode" name="postcode" context="{'group_by':'postcode', 'residual_visible':True}"/>
+                </group>
+            </search>
+        </field>
+    </record>
+
+    <record id="estate_property_list" model="ir.ui.view">
+        <field name="name">Properties</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <list>
+                <field name="name" />
+                <field name="postcode" />
+                <field name="bedrooms" />
+                <field name="living_area" />
+                <field name="expected_price" />
+                <field name="selling_price" />
+                <field name="date_availability" />
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_property_form" model="ir.ui.view">
+        <field name="name">Properties</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <form>
+                <sheet>
+                    <group class="mb-5">
+                        <field name="name" string="" required="True" placeholder="My new house" class="fs-1" />
+                        <field name="tag_ids" string="" widget="many2many_tags"/>
+                    </group>
+                    <group>
+                        <group>
+                            <field name="property_type_id" string="Property Type" />
+                            <field name="postcode" />
+                            <field name="date_availability" string="Available From" />
+                        </group>
+                        <group>
+                            <field name="expected_price" required="True" />
+                            <field name="selling_price" />
+                        </group>
+                    </group>
+                    <notebook>
+                        <page string="Description">
+                            <group>
+                                <field name="description" />
+                                <field name="bedrooms" />
+                                <field name="living_area" string="Living Area (sqm)" />
+                                <field name="facades" />
+                                <field name="garage" />
+                                <field name="garden" />
+                                <field name="garden_area" string="Garden Area (sqm)" />
+                                <field name="garden_orientation" />
+                            </group>
+                        </page>
+                        <page string="Offers">
+                            <field name="offer_ids" />
+                        </page>
+                        <page string="Other Info">
+                            <group>
+                                <field name="seller_id" string="Salesman" />
+                                <field name="buyer_id" />
+                            </group>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
 </odoo>
diff --git a/estate/views/estate_tag_views.xml b/estate/views/estate_tag_views.xml
new file mode 100644
index 00000000000..c2f9bb2d1ea
--- /dev/null
+++ b/estate/views/estate_tag_views.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_tag_action" model="ir.actions.act_window">
+        <field name="name">Tags</field>
+        <field name="res_model">estate.tag</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_type_views.xml b/estate/views/estate_type_views.xml
new file mode 100644
index 00000000000..1bb09714a7c
--- /dev/null
+++ b/estate/views/estate_type_views.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_type_action" model="ir.actions.act_window">
+        <field name="name">Types</field>
+        <field name="res_model">estate.type</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>

From 1e20bf5d0a1b04bde5e556152a9fb475b9d4e03d Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Tue, 18 Aug 2026 14:09:32 +0200
Subject: [PATCH 08/14] [IMP] estate: chapter 8

---
 estate/models/estate_offer.py          | 21 ++++++++++++++++++++-
 estate/models/estate_property.py       | 25 +++++++++++++++++++++++--
 estate/views/estate_offer_views.xml    |  4 ++++
 estate/views/estate_property_views.xml |  2 ++
 4 files changed, 49 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_offer.py b/estate/models/estate_offer.py
index 9e1194a45a9..0c0ef456925 100644
--- a/estate/models/estate_offer.py
+++ b/estate/models/estate_offer.py
@@ -1,4 +1,5 @@
-from odoo import fields, models
+from odoo import api, fields, models
+from odoo.tools.date_utils import add
 
 
 class EstateOffer(models.Model):
@@ -9,7 +10,25 @@ class EstateOffer(models.Model):
     status = fields.Selection(
         [("accepted", "Accepted"), ("refused", "Refused")], copy=False
     )
+    validity = fields.Integer(default=7)
 
     # Foreign fields
     partner_id = fields.Many2one("res.partner", required=True)
     property_id = fields.Many2one("estate.property", required=True)
+
+    # Computed fields
+    date_deadline = fields.Date(
+        compute="_compute_date_deadline", inverse="_inverse_date_deadline"
+    )
+
+    @api.depends("validity")
+    def _compute_date_deadline(self):
+        for record in self:
+            record.date_deadline = add(
+                record.create_date if record.create_date else fields.Date.today(),
+                days=+record.validity,
+            )
+
+    def _inverse_date_deadline(self):
+        for record in self:
+            record.validity = (record.date_deadline - fields.Date.today()).days
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 2c27050a437..2de6ccda610 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,5 @@
 from odoo import fields, models
+from odoo.orm.models import api
 from odoo.tools.date_utils import add
 
 
@@ -12,8 +13,8 @@ class EstateProperty(models.Model):
     date_availability = fields.Date(
         copy=False, default=lambda _: add(fields.Date.today(), months=+3)
     )
-    expected_price = fields.Float(required=True)
-    selling_price = fields.Float(copy=False, readonly=True)
+    expected_price = fields.Integer(required=True)
+    selling_price = fields.Integer(copy=False, readonly=True)
     bedrooms = fields.Integer(default=2)
     living_area = fields.Integer()
     facades = fields.Integer()
@@ -52,3 +53,23 @@ class EstateProperty(models.Model):
     )
     tag_ids = fields.Many2many("estate.tag")
     offer_ids = fields.One2many("estate.offer", "property_id")
+
+    # Computed fields
+    total_area = fields.Integer(compute="_compute_total_area")
+    best_price = fields.Integer(compute="_compute_best_price")
+
+    @api.depends("living_area", "garden_area")
+    def _compute_total_area(self):
+        for record in self:
+            record.total_area = record.living_area + record.garden_area
+
+    @api.depends("offer_ids.price")
+    def _compute_best_price(self):
+        for record in self:
+            # Fallback to 0 if no records are present
+            record.best_price = max(record.offer_ids.mapped("price") + [0])
+
+    @api.onchange("garden")
+    def _onchange_garden(self):
+        self.garden_area = 10 if self.garden else None
+        self.garden_orientation = "north" if self.garden else None
diff --git a/estate/views/estate_offer_views.xml b/estate/views/estate_offer_views.xml
index 4e985114e9b..8437fb41646 100644
--- a/estate/views/estate_offer_views.xml
+++ b/estate/views/estate_offer_views.xml
@@ -7,6 +7,8 @@
             <list>
                 <field name="price" />
                 <field name="partner_id" />
+                <field name="validity" />
+                <field name="date_deadline" />
                 <field name="status" />
             </list>
         </field>
@@ -21,6 +23,8 @@
                     <group>
                         <field name="price" />
                         <field name="partner_id" />
+                        <field name="validity" />
+                        <field name="date_deadline" />
                         <field name="status" />
                     </group>
                 </sheet>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 46a92a276c9..97daa71650f 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -56,6 +56,7 @@
                         </group>
                         <group>
                             <field name="expected_price" required="True" />
+                                <field name="best_price" />
                             <field name="selling_price" />
                         </group>
                     </group>
@@ -70,6 +71,7 @@
                                 <field name="garden" />
                                 <field name="garden_area" string="Garden Area (sqm)" />
                                 <field name="garden_orientation" />
+                                <field name="total_area" string="Total Area (sqm)" />
                             </group>
                         </page>
                         <page string="Offers">

From 6ebc63ffaac2e90d95860b7091d2316da7510495 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Tue, 18 Aug 2026 15:08:04 +0200
Subject: [PATCH 09/14] [IMP] estate: chapter 9

---
 estate/models/estate_offer.py          | 19 ++++++++++
 estate/models/estate_property.py       | 49 ++++++++++++++++++++++++--
 estate/views/estate_offer_views.xml    |  2 ++
 estate/views/estate_property_views.xml |  7 +++-
 4 files changed, 74 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_offer.py b/estate/models/estate_offer.py
index 0c0ef456925..fac8af09e3e 100644
--- a/estate/models/estate_offer.py
+++ b/estate/models/estate_offer.py
@@ -1,4 +1,5 @@
 from odoo import api, fields, models
+from odoo.exceptions import UserError
 from odoo.tools.date_utils import add
 
 
@@ -32,3 +33,21 @@ def _compute_date_deadline(self):
     def _inverse_date_deadline(self):
         for record in self:
             record.validity = (record.date_deadline - fields.Date.today()).days
+
+    # Actions
+
+    def set_status_accepted(self):
+        for record in self:
+            for offer in self.property_id.offer_ids:
+                if offer.status == "accepted":
+                    raise UserError("Only one offer can be accepted")
+
+            record.status = "accepted"
+
+        return True
+
+    def set_status_refused(self):
+        for record in self:
+            record.status = "refused"
+
+        return True
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 2de6ccda610..5d33d3712a9 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,5 @@
 from odoo import fields, models
+from odoo.exceptions import UserError
 from odoo.orm.models import api
 from odoo.tools.date_utils import add
 
@@ -14,7 +15,9 @@ class EstateProperty(models.Model):
         copy=False, default=lambda _: add(fields.Date.today(), months=+3)
     )
     expected_price = fields.Integer(required=True)
-    selling_price = fields.Integer(copy=False, readonly=True)
+    selling_price = fields.Integer(
+        copy=False, readonly=True, compute="_compute_selling_price"
+    )
     bedrooms = fields.Integer(default=2)
     living_area = fields.Integer()
     facades = fields.Integer()
@@ -43,7 +46,11 @@ class EstateProperty(models.Model):
         "estate.type", required=True, ondelete="restrict"
     )
     buyer_id = fields.Many2one(
-        "res.partner", required=False, copy=False, ondelete="restrict"
+        "res.partner",
+        required=False,
+        copy=False,
+        ondelete="restrict",
+        compute="_compute_buyer_id",
     )
     seller_id = fields.Many2one(
         "res.users",
@@ -58,6 +65,24 @@ class EstateProperty(models.Model):
     total_area = fields.Integer(compute="_compute_total_area")
     best_price = fields.Integer(compute="_compute_best_price")
 
+    @api.depends("offer_ids")
+    def _compute_selling_price(self):
+        for record in self:
+            record.selling_price = None
+            for offer in record.offer_ids:
+                if offer.status == "accepted":
+                    record.selling_price = offer.price
+                    break
+
+    @api.depends("offer_ids")
+    def _compute_buyer_id(self):
+        for record in self:
+            record.buyer_id = None
+            for offer in record.offer_ids:
+                if offer.status == "accepted":
+                    record.buyer_id = offer.partner_id
+                    break
+
     @api.depends("living_area", "garden_area")
     def _compute_total_area(self):
         for record in self:
@@ -73,3 +98,23 @@ def _compute_best_price(self):
     def _onchange_garden(self):
         self.garden_area = 10 if self.garden else None
         self.garden_orientation = "north" if self.garden else None
+
+    # Actions
+
+    def set_state_cancel(self):
+        for record in self:
+            if record.state == "sold":
+                raise UserError("A sold property cannot be cancelled")
+
+            record.state = "cancelled"
+
+        return True
+
+    def set_state_sold(self):
+        for record in self:
+            if record.state == "cancelled":
+                raise UserError("A cancelled property cannot be sold")
+
+            record.state = "sold"
+
+        return True
diff --git a/estate/views/estate_offer_views.xml b/estate/views/estate_offer_views.xml
index 8437fb41646..a51c568b5dc 100644
--- a/estate/views/estate_offer_views.xml
+++ b/estate/views/estate_offer_views.xml
@@ -9,6 +9,8 @@
                 <field name="partner_id" />
                 <field name="validity" />
                 <field name="date_deadline" />
+                <button type="object" string="Accept" name="set_status_accepted" icon="fa-check" />
+                <button type="object" string="Refuse" name="set_status_refused" icon="fa-close" />
                 <field name="status" />
             </list>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 97daa71650f..95f18ad3fd8 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -43,6 +43,10 @@
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form>
+                <header>
+                    <button type="object" name="set_state_cancel" string="Cancel" />
+                    <button type="object" name="set_state_sold" string="Sold" />
+                </header>
                 <sheet>
                     <group class="mb-5">
                         <field name="name" string="" required="True" placeholder="My new house" class="fs-1" />
@@ -50,13 +54,14 @@
                     </group>
                     <group>
                         <group>
+                            <field name="state" readonly="True" />
                             <field name="property_type_id" string="Property Type" />
                             <field name="postcode" />
                             <field name="date_availability" string="Available From" />
                         </group>
                         <group>
                             <field name="expected_price" required="True" />
-                                <field name="best_price" />
+                            <field name="best_price" />
                             <field name="selling_price" />
                         </group>
                     </group>

From a26dc4edb2fe0b1a0f6fdca31c3c17a63d09706c Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Tue, 18 Aug 2026 16:11:09 +0200
Subject: [PATCH 10/14] [IMP] estate: chapter 10

---
 estate/models/estate_offer.py    | 17 ++++++++++++++++-
 estate/models/estate_property.py | 18 +++++++++++++-----
 estate/models/estate_tag.py      |  6 ++++++
 estate/models/estate_type.py     |  6 ++++++
 4 files changed, 41 insertions(+), 6 deletions(-)

diff --git a/estate/models/estate_offer.py b/estate/models/estate_offer.py
index fac8af09e3e..6b4f80d4346 100644
--- a/estate/models/estate_offer.py
+++ b/estate/models/estate_offer.py
@@ -1,6 +1,7 @@
 from odoo import api, fields, models
-from odoo.exceptions import UserError
+from odoo.exceptions import UserError, ValidationError
 from odoo.tools.date_utils import add
+from odoo.tools.float_utils import float_compare
 
 
 class EstateOffer(models.Model):
@@ -22,6 +23,12 @@ class EstateOffer(models.Model):
         compute="_compute_date_deadline", inverse="_inverse_date_deadline"
     )
 
+    # Constraints
+    _check_price = models.Constraint(
+        "CHECK(price > 0)",
+        "Offer price must be stricly positive",
+    )
+
     @api.depends("validity")
     def _compute_date_deadline(self):
         for record in self:
@@ -38,6 +45,14 @@ def _inverse_date_deadline(self):
 
     def set_status_accepted(self):
         for record in self:
+            if (
+                float_compare(record.price, record.property_id.expected_price * 0.9, 2)
+                == -1
+            ):
+                raise ValidationError(
+                    "Offer has to be at least 90% of the expected price"
+                )
+
             for offer in self.property_id.offer_ids:
                 if offer.status == "accepted":
                     raise UserError("Only one offer can be accepted")
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 5d33d3712a9..e88df0eb9b7 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,7 +1,8 @@
 from odoo import fields, models
-from odoo.exceptions import UserError
+from odoo.exceptions import UserError, ValidationError
 from odoo.orm.models import api
 from odoo.tools.date_utils import add
+from odoo.tools.float_utils import float_compare, float_is_zero
 
 
 class EstateProperty(models.Model):
@@ -14,9 +15,9 @@ class EstateProperty(models.Model):
     date_availability = fields.Date(
         copy=False, default=lambda _: add(fields.Date.today(), months=+3)
     )
-    expected_price = fields.Integer(required=True)
-    selling_price = fields.Integer(
-        copy=False, readonly=True, compute="_compute_selling_price"
+    expected_price = fields.Float(required=True)
+    selling_price = fields.Float(
+        copy=False, readonly=True, compute="_compute_selling_price", store=True
     )
     bedrooms = fields.Integer(default=2)
     living_area = fields.Integer()
@@ -65,10 +66,17 @@ class EstateProperty(models.Model):
     total_area = fields.Integer(compute="_compute_total_area")
     best_price = fields.Integer(compute="_compute_best_price")
 
+    # Constraints
+    _check_expected_price = models.Constraint(
+        "CHECK(expected_price > 0)",
+        "Expected price must be stricly positive",
+    )
+
+    # Computations
     @api.depends("offer_ids")
     def _compute_selling_price(self):
         for record in self:
-            record.selling_price = None
+            record.selling_price = 0
             for offer in record.offer_ids:
                 if offer.status == "accepted":
                     record.selling_price = offer.price
diff --git a/estate/models/estate_tag.py b/estate/models/estate_tag.py
index 5ce1c31faf6..e47751a155c 100644
--- a/estate/models/estate_tag.py
+++ b/estate/models/estate_tag.py
@@ -6,3 +6,9 @@ class EstateTag(models.Model):
     _description = "An estate tag"
 
     name = fields.Char(required=True)
+
+    # Constraints
+    _check_name = models.Constraint(
+        "UNIQUE(name)",
+        "A tag name must be unique",
+    )
diff --git a/estate/models/estate_type.py b/estate/models/estate_type.py
index 848488dfb59..65e082a58ea 100644
--- a/estate/models/estate_type.py
+++ b/estate/models/estate_type.py
@@ -6,3 +6,9 @@ class EstateType(models.Model):
     _description = "An estate type"
 
     name = fields.Char(required=True)
+
+    # Constraints
+    _check_name = models.Constraint(
+        "UNIQUE(name)",
+        "A type must be unique",
+    )

From fc251aa5b0252ec7a5be77123394d47106469d0b Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Wed, 19 Aug 2026 11:34:30 +0200
Subject: [PATCH 11/14] [IMP] estate: chapter 11

---
 estate/__manifest__.py                 |   2 +-
 estate/models/estate_offer.py          |  16 ++++
 estate/models/estate_property.py       |  10 ++-
 estate/models/estate_tag.py            |   2 +
 estate/models/estate_type.py           |  16 +++-
 estate/views/estate_offer_views.xml    |  32 +++----
 estate/views/estate_property_views.xml | 111 +++++++++++++------------
 estate/views/estate_tag_views.xml      |  13 ++-
 estate/views/estate_type_views.xml     |  45 ++++++++++
 9 files changed, 167 insertions(+), 80 deletions(-)

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 73e0e310abe..5faa40a5720 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -11,9 +11,9 @@
     "data": [
         "security/ir.model.access.csv",
         "views/estate_property_views.xml",
+        "views/estate_offer_views.xml",
         "views/estate_type_views.xml",
         "views/estate_tag_views.xml",
-        "views/estate_offer_views.xml",
         "views/estate_menus.xml",
     ],
 }
diff --git a/estate/models/estate_offer.py b/estate/models/estate_offer.py
index 6b4f80d4346..2a2fe123ad9 100644
--- a/estate/models/estate_offer.py
+++ b/estate/models/estate_offer.py
@@ -7,6 +7,7 @@
 class EstateOffer(models.Model):
     _name = "estate.offer"
     _description = "An estate offer"
+    _order = "price desc"
 
     price = fields.Float()
     status = fields.Selection(
@@ -23,6 +24,11 @@ class EstateOffer(models.Model):
         compute="_compute_date_deadline", inverse="_inverse_date_deadline"
     )
 
+    # Related fields
+    property_type_id = fields.Many2one(
+        related="property_id.property_type_id", store=True
+    )
+
     # Constraints
     _check_price = models.Constraint(
         "CHECK(price > 0)",
@@ -45,6 +51,11 @@ def _inverse_date_deadline(self):
 
     def set_status_accepted(self):
         for record in self:
+            if record.status == "accepted" or record.status == "refused":
+                raise UserError(
+                    "The state of an offer cannot be changed once it is set"
+                )
+
             if (
                 float_compare(record.price, record.property_id.expected_price * 0.9, 2)
                 == -1
@@ -63,6 +74,11 @@ def set_status_accepted(self):
 
     def set_status_refused(self):
         for record in self:
+            if record.status == "accepted" or record.status == "refused":
+                raise UserError(
+                    "The state of an offer cannot be changed once it is set"
+                )
+
             record.status = "refused"
 
         return True
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index e88df0eb9b7..1a7e00cc475 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,13 +1,13 @@
 from odoo import fields, models
-from odoo.exceptions import UserError, ValidationError
+from odoo.exceptions import UserError
 from odoo.orm.models import api
 from odoo.tools.date_utils import add
-from odoo.tools.float_utils import float_compare, float_is_zero
 
 
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "An estate property"
+    _order = "id desc"
 
     name = fields.Char(required=True)
     description = fields.Text()
@@ -17,7 +17,7 @@ class EstateProperty(models.Model):
     )
     expected_price = fields.Float(required=True)
     selling_price = fields.Float(
-        copy=False, readonly=True, compute="_compute_selling_price", store=True
+        copy=False, readonly=True, compute="_compute_selling_price"
     )
     bedrooms = fields.Integer(default=2)
     living_area = fields.Integer()
@@ -113,6 +113,8 @@ def set_state_cancel(self):
         for record in self:
             if record.state == "sold":
                 raise UserError("A sold property cannot be cancelled")
+            if record.state == "cancelled":
+                raise UserError("A cancelled property cannot be cancelled again")
 
             record.state = "cancelled"
 
@@ -122,6 +124,8 @@ def set_state_sold(self):
         for record in self:
             if record.state == "cancelled":
                 raise UserError("A cancelled property cannot be sold")
+            if record.state == "sold":
+                raise UserError("A sold property cannot be sold again")
 
             record.state = "sold"
 
diff --git a/estate/models/estate_tag.py b/estate/models/estate_tag.py
index e47751a155c..776dcfadf38 100644
--- a/estate/models/estate_tag.py
+++ b/estate/models/estate_tag.py
@@ -4,8 +4,10 @@
 class EstateTag(models.Model):
     _name = "estate.tag"
     _description = "An estate tag"
+    _order = "name"
 
     name = fields.Char(required=True)
+    color = fields.Integer()
 
     # Constraints
     _check_name = models.Constraint(
diff --git a/estate/models/estate_type.py b/estate/models/estate_type.py
index 65e082a58ea..79e8355e01e 100644
--- a/estate/models/estate_type.py
+++ b/estate/models/estate_type.py
@@ -1,14 +1,28 @@
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class EstateType(models.Model):
     _name = "estate.type"
     _description = "An estate type"
+    _order = "sequence, name"
 
     name = fields.Char(required=True)
+    sequence = fields.Integer(required=True, default=1)
+
+    # Foreign fields
+    property_ids = fields.One2many("estate.property", "property_type_id")
+    offer_ids = fields.One2many("estate.offer", "property_type_id")
+
+    # Computed fields
+    offer_count = fields.Integer(compute="_compute_offer_count")
 
     # Constraints
     _check_name = models.Constraint(
         "UNIQUE(name)",
         "A type must be unique",
     )
+
+    @api.depends("offer_ids")
+    def _compute_offer_count(self):
+        for record in self:
+            record.offer_count = len(record.offer_ids)
diff --git a/estate/views/estate_offer_views.xml b/estate/views/estate_offer_views.xml
index a51c568b5dc..eee1ab5a5eb 100644
--- a/estate/views/estate_offer_views.xml
+++ b/estate/views/estate_offer_views.xml
@@ -1,37 +1,25 @@
 <?xml version="1.0"?>
 <odoo>
+    <record id="estate_offer_action" model="ir.actions.act_window">
+        <field name="name">Offers</field>
+        <field name="res_model">estate.offer</field>
+        <field name="view_mode">list</field>
+        <field name="domain">[('property_type_id', '=', active_id)]</field>
+    </record>
+
     <record id="estate_offer_list" model="ir.ui.view">
         <field name="name">Offers</field>
         <field name="model">estate.offer</field>
         <field name="arch" type="xml">
-            <list>
+            <list editable="top" decoration-success="status == 'accepted'" decoration-danger="status == 'refused'">
                 <field name="price" />
                 <field name="partner_id" />
                 <field name="validity" />
                 <field name="date_deadline" />
-                <button type="object" string="Accept" name="set_status_accepted" icon="fa-check" />
-                <button type="object" string="Refuse" name="set_status_refused" icon="fa-close" />
+                <button type="object" string="Accept" name="set_status_accepted" icon="fa-check" invisible="status != ''" />
+                <button type="object" string="Refuse" name="set_status_refused" icon="fa-close" invisible="status != ''" />
                 <field name="status" />
             </list>
         </field>
     </record>
-
-    <record id="estate_offer_form" model="ir.ui.view">
-        <field name="name">Offers</field>
-        <field name="model">estate.offer</field>
-        <field name="arch" type="xml">
-            <form>
-                <sheet>
-                    <group>
-                        <field name="price" />
-                        <field name="partner_id" />
-                        <field name="validity" />
-                        <field name="date_deadline" />
-                        <field name="status" />
-                    </group>
-                </sheet>
-            </form>
-        </field>
-    </record>
-
 </odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 95f18ad3fd8..a57f9c6e6d0 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -4,36 +4,39 @@
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
+        <field name="context">{'search_default_available': True}</field>
     </record>
 
-    <record id="estate_property_search" model="ir.ui.view">
-        <field name="name">Search</field>
-        <field name="model">estate.property</field>
+    <record id="estate_property_search" model="ir.ui.view"> 
+        <field name="name">Search</field> 
+        <field name="model">estate.property</field> 
         <field name="arch" type="xml">
-            <search string="Delivery Carrier">
-                <field name="name" string="Estate" />
-                <field name="selling_price"/>
-                <separator/>
-                <filter string="Available" name="available" domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]"/>
-                <group>
-                    <filter string="Postcode" name="postcode" context="{'group_by':'postcode', 'residual_visible':True}"/>
-                </group>
-            </search>
-        </field>
-    </record>
+        <search>
+            <field name="name" string="Estate" />
+            <field name="selling_price" />
+            <field string="Living Area (sqm)" name="living_area" filter_domain="[('living_area', '>=', self)]" />
+            <separator />
+            <filter string="Available" name="available" domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]"/> 
+            <group> 
+                <filter string="Postcode" name="postcode" context="{'group_by':'postcode', 'residual_visible':True}"/> 
+            </group> 
+        </search> 
+        </field> 
+    </record> 
 
     <record id="estate_property_list" model="ir.ui.view">
         <field name="name">Properties</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
-            <list>
+            <list decoration-muted="state == 'sold'" decoration-success="state == 'received'" decoration-bf="state == 'accepted'">
                 <field name="name" />
                 <field name="postcode" />
                 <field name="bedrooms" />
                 <field name="living_area" />
                 <field name="expected_price" />
                 <field name="selling_price" />
-                <field name="date_availability" />
+                <field name="date_availability" optional="True" />
+                <field name="state" column_invisible="True" />
             </list>
         </field>
     </record>
@@ -44,51 +47,55 @@
         <field name="arch" type="xml">
             <form>
                 <header>
-                    <button type="object" name="set_state_cancel" string="Cancel" />
+                    <group invisible="state == 'sold' or state == 'cancelled'">
                     <button type="object" name="set_state_sold" string="Sold" />
+                    <button type="object" name="set_state_cancel" string="Cancel" />
+                    </group>
+                    <field name="state" string="" widget="statusbar" statusbar_visible="new,received,accepted,sold" />
                 </header>
                 <sheet>
-                    <group class="mb-5">
-                        <field name="name" string="" required="True" placeholder="My new house" class="fs-1" />
-                        <field name="tag_ids" string="" widget="many2many_tags"/>
-                    </group>
-                    <group>
-                        <group>
-                            <field name="state" readonly="True" />
-                            <field name="property_type_id" string="Property Type" />
-                            <field name="postcode" />
-                            <field name="date_availability" string="Available From" />
+                    <div>
+                        <group class="mb-5">
+                            <field name="name" string="" required="True" placeholder="My new house" class="fs-1" />
+                            <field name="tag_ids" string="" widget="many2many_tags" options="{'color_field': 'color'}" />
                         </group>
                         <group>
-                            <field name="expected_price" required="True" />
-                            <field name="best_price" />
-                            <field name="selling_price" />
-                        </group>
-                    </group>
-                    <notebook>
-                        <page string="Description">
                             <group>
-                                <field name="description" />
-                                <field name="bedrooms" />
-                                <field name="living_area" string="Living Area (sqm)" />
-                                <field name="facades" />
-                                <field name="garage" />
-                                <field name="garden" />
-                                <field name="garden_area" string="Garden Area (sqm)" />
-                                <field name="garden_orientation" />
-                                <field name="total_area" string="Total Area (sqm)" />
+                                <field name="property_type_id" string="Property Type" options="{'no_quick_create': True, 'no_create_edit': True}" />
+                                <field name="postcode" />
+                                <field name="date_availability" string="Available From" />
                             </group>
-                        </page>
-                        <page string="Offers">
-                            <field name="offer_ids" />
-                        </page>
-                        <page string="Other Info">
                             <group>
-                                <field name="seller_id" string="Salesman" />
-                                <field name="buyer_id" />
+                                <field name="expected_price" required="True" />
+                                <field name="best_price" />
+                                <field name="selling_price" />
                             </group>
-                        </page>
-                    </notebook>
+                        </group>
+                        <notebook>
+                            <page string="Description">
+                                <group>
+                                    <field name="description" />
+                                    <field name="bedrooms" />
+                                    <field name="living_area" string="Living Area (sqm)" />
+                                    <field name="facades" />
+                                    <field name="garage" />
+                                    <field name="garden" />
+                                    <field name="garden_area" string="Garden Area (sqm)" invisible="not garden" />
+                                    <field name="garden_orientation" invisible="not garden" />
+                                    <field name="total_area" string="Total Area (sqm)" />
+                                </group>
+                            </page>
+                            <page string="Offers">
+                                <field name="offer_ids" readonly="state == 'sold' or state == 'cancelled'" />
+                            </page>
+                            <page string="Other Info">
+                                <group>
+                                    <field name="seller_id" string="Salesman" />
+                                    <field name="buyer_id" />
+                                </group>
+                            </page>
+                        </notebook>
+                    </div>
                 </sheet>
             </form>
         </field>
diff --git a/estate/views/estate_tag_views.xml b/estate/views/estate_tag_views.xml
index c2f9bb2d1ea..f2435d7b573 100644
--- a/estate/views/estate_tag_views.xml
+++ b/estate/views/estate_tag_views.xml
@@ -3,6 +3,17 @@
     <record id="estate_tag_action" model="ir.actions.act_window">
         <field name="name">Tags</field>
         <field name="res_model">estate.tag</field>
-        <field name="view_mode">list,form</field>
+        <field name="view_mode">list</field>
+    </record>
+
+    <record id="estate_tag_list" model="ir.ui.view">
+        <field name="name">Type</field>
+        <field name="model">estate.tag</field>
+        <field name="arch" type="xml">
+            <list editable="top">
+                <field name="name" />
+                <field name="color" widget="color_picker" />
+            </list>
+        </field>
     </record>
 </odoo>
diff --git a/estate/views/estate_type_views.xml b/estate/views/estate_type_views.xml
index 1bb09714a7c..f40c97d6660 100644
--- a/estate/views/estate_type_views.xml
+++ b/estate/views/estate_type_views.xml
@@ -5,4 +5,49 @@
         <field name="res_model">estate.type</field>
         <field name="view_mode">list,form</field>
     </record>
+
+    <record id="estate_type_list" model="ir.ui.view">
+        <field name="name">Type</field>
+        <field name="model">estate.type</field>
+        <field name="arch" type="xml">
+            <list>
+                <field name="sequence" widget="handle" />
+                <field name="name" />
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_type_form" model="ir.ui.view">
+        <field name="name">Type</field>
+        <field name="model">estate.type</field>
+        <field name="arch" type="xml">
+            <form>
+                <sheet>
+                    <div class="oe_button_box" name="button_box">
+                        <button class="oe_stat_button" type="action" name="%(estate_offer_action)d" icon="fa-money">
+                            <div class="o_form_field o_stat_info">
+                                <span class="o_stat_text">Offers</span>
+                                <span class="o_stat_value">
+                                    <field name="offer_count" widget='monetary'/>
+                                </span>
+                            </div>
+                        </button>
+                    </div>
+
+                    <field name="name" string="" required="True" placeholder="House" class="mb-5 fs-1" />
+                    <notebook>
+                        <page string="Properties">
+                            <field name="property_ids">
+                                <list>
+                                    <field name="name" />
+                                    <field name="expected_price" />
+                                    <field name="state" />
+                                </list>
+                            </field>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
 </odoo>

From 0c4927a28f050435d4872253e7f7d661dca0a74b Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Wed, 19 Aug 2026 14:14:20 +0200
Subject: [PATCH 12/14] [IMP] estate: chapter 12

---
 estate/__manifest__.py                 |  1 +
 estate/models/__init__.py              |  1 +
 estate/models/estate_offer.py          | 11 +++++++++++
 estate/models/estate_property.py       | 12 +++++++++++-
 estate/models/res_user.py              | 24 ++++++++++++++++++++++++
 estate/views/estate_property_views.xml |  2 +-
 estate/views/res_user.xml              | 17 +++++++++++++++++
 7 files changed, 66 insertions(+), 2 deletions(-)
 create mode 100644 estate/models/res_user.py
 create mode 100644 estate/views/res_user.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 5faa40a5720..63095a970c4 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -15,5 +15,6 @@
         "views/estate_type_views.xml",
         "views/estate_tag_views.xml",
         "views/estate_menus.xml",
+        "views/res_user.xml",
     ],
 }
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 8d8063038f5..12614b21242 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -2,3 +2,4 @@
 from . import estate_type
 from . import estate_tag
 from . import estate_offer
+from . import res_user
diff --git a/estate/models/estate_offer.py b/estate/models/estate_offer.py
index 2a2fe123ad9..26d0f28b578 100644
--- a/estate/models/estate_offer.py
+++ b/estate/models/estate_offer.py
@@ -82,3 +82,14 @@ def set_status_refused(self):
             record.status = "refused"
 
         return True
+
+    # Overwrites
+
+    @api.model
+    def create(self, vals_list):
+        for vals in vals_list:
+            property = self.env["estate.property"].browse(vals["property_id"])
+            if property.state == "new":
+                property.state = "received"
+
+        super().create(vals_list)
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 1a7e00cc475..6ed0991742a 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,5 +1,5 @@
 from odoo import fields, models
-from odoo.exceptions import UserError
+from odoo.exceptions import UserError, ValidationError
 from odoo.orm.models import api
 from odoo.tools.date_utils import add
 
@@ -130,3 +130,13 @@ def set_state_sold(self):
             record.state = "sold"
 
         return True
+
+    # Overwrites
+
+    @api.ondelete(at_uninstall=False)
+    def onDelete(self):
+        for record in self:
+            if record.state not in ["new", "cancelled"]:
+                raise ValidationError(
+                    "Only properties in the state new or cancelled can be deleted"
+                )
diff --git a/estate/models/res_user.py b/estate/models/res_user.py
new file mode 100644
index 00000000000..21ec8d9b4b0
--- /dev/null
+++ b/estate/models/res_user.py
@@ -0,0 +1,24 @@
+from odoo import fields, models
+
+
+class User(models.Model):
+    _inherit = "res.users"
+
+    # Foreign fields
+
+    property_ids = fields.One2many(
+        "estate.property",
+        "seller_id",
+    )
+
+    # Computed fields
+
+    available_property_ids = fields.One2many(
+        "estate.property", compute="_compute_filtered_properties"
+    )
+
+    def _compute_filtered_properties(self):
+        for record in self:
+            record.available_property_ids = record.property_ids.filtered(
+                lambda x: x.state not in ("sold", "cancelled")
+            )
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index a57f9c6e6d0..0768f051b37 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -16,7 +16,7 @@
             <field name="selling_price" />
             <field string="Living Area (sqm)" name="living_area" filter_domain="[('living_area', '>=', self)]" />
             <separator />
-            <filter string="Available" name="available" domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]"/> 
+            <filter string="Available" name="available" domain="[('state', 'not in', ('sold', 'cancelled'))]"/> 
             <group> 
                 <filter string="Postcode" name="postcode" context="{'group_by':'postcode', 'residual_visible':True}"/> 
             </group> 
diff --git a/estate/views/res_user.xml b/estate/views/res_user.xml
new file mode 100644
index 00000000000..485530c2e58
--- /dev/null
+++ b/estate/views/res_user.xml
@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <data>
+    <record id="res_users_view_form" model="ir.ui.view">
+        <field name="name">res.users.view.form.inherit.estate.property</field>
+        <field name="model">res.users</field>
+        <field name="inherit_id" ref="base.view_users_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//page[@name='page_security']" position="after">
+                <page string="Properties">
+                    <field name="available_property_ids" />
+                </page>
+            </xpath>
+        </field>
+    </record>
+    </data>
+</odoo>

From 85ae848d9a5c1e797b51e677be8ef7d86839c472 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Wed, 19 Aug 2026 15:29:40 +0200
Subject: [PATCH 13/14] [IMP] estate: chapter 13

---
 estate/views/estate_offer_views.xml      |  4 +--
 estate/views/estate_property_views.xml   |  2 +-
 estate_account/__init__.py               |  1 +
 estate_account/__manifest__.py           | 14 ++++++++++
 estate_account/models/__init__.py        |  1 +
 estate_account/models/estate_property.py | 35 ++++++++++++++++++++++++
 6 files changed, 54 insertions(+), 3 deletions(-)
 create mode 100644 estate_account/__init__.py
 create mode 100644 estate_account/__manifest__.py
 create mode 100644 estate_account/models/__init__.py
 create mode 100644 estate_account/models/estate_property.py

diff --git a/estate/views/estate_offer_views.xml b/estate/views/estate_offer_views.xml
index eee1ab5a5eb..4fa3cabbe16 100644
--- a/estate/views/estate_offer_views.xml
+++ b/estate/views/estate_offer_views.xml
@@ -16,8 +16,8 @@
                 <field name="partner_id" />
                 <field name="validity" />
                 <field name="date_deadline" />
-                <button type="object" string="Accept" name="set_status_accepted" icon="fa-check" invisible="status != ''" />
-                <button type="object" string="Refuse" name="set_status_refused" icon="fa-close" invisible="status != ''" />
+                <button type="object" string="Accept" name="set_status_accepted" icon="fa-check" invisible="status" />
+                <button type="object" string="Refuse" name="set_status_refused" icon="fa-close" invisible="status" />
                 <field name="status" />
             </list>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 0768f051b37..c993d25c751 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -86,7 +86,7 @@
                                 </group>
                             </page>
                             <page string="Offers">
-                                <field name="offer_ids" readonly="state == 'sold' or state == 'cancelled'" />
+                                <field name="offer_ids" readonly="state in ('sold', 'cancelled')" />
                             </page>
                             <page string="Other Info">
                                 <group>
diff --git a/estate_account/__init__.py b/estate_account/__init__.py
new file mode 100644
index 00000000000..0650744f6bc
--- /dev/null
+++ b/estate_account/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
new file mode 100644
index 00000000000..e5a733ca713
--- /dev/null
+++ b/estate_account/__manifest__.py
@@ -0,0 +1,14 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+
+{
+    "name": "estate_account",
+    "application": False,
+    "depends": [
+        "base",
+        "account",
+        "estate",
+    ],
+    "data": [],
+}
diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py
new file mode 100644
index 00000000000..5e1963c9d2f
--- /dev/null
+++ b/estate_account/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_property
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py
new file mode 100644
index 00000000000..2b15afa78ed
--- /dev/null
+++ b/estate_account/models/estate_property.py
@@ -0,0 +1,35 @@
+from odoo import Command, models
+
+
+class EstateProperty(models.Model):
+    _inherit = "estate.property"
+
+    def set_state_sold(self):
+        result = super().set_state_sold()
+
+        for record in self:
+            invoice = self.env["account.move"].create(
+                {
+                    "partner_id": record.buyer_id.id,
+                    "move_type": "out_invoice",
+                    "invoice_line_ids": [
+                        Command.create(
+                            {
+                                "name": "Down Payment",
+                                "quantity": 1,
+                                "price_unit": record.selling_price * 0.06,
+                            }
+                        ),
+                        Command.create(
+                            {
+                                "name": "Administrative Fees",
+                                "quantity": 1,
+                                "price_unit": 100.00,
+                            }
+                        ),
+                    ],
+                }
+            )
+            invoice.action_post()
+
+        return result

From 0180936abcfdc67d0c1f465ed4238a4e7a41e618 Mon Sep 17 00:00:00 2001
From: Vincent Vallaeys <vincent@vallaeys.com>
Date: Wed, 19 Aug 2026 16:11:35 +0200
Subject: [PATCH 14/14] [IMP] estate: chapter 14

[IMP] estate: refuse other offers once one gets accepted

[CLN] estate: cleanup + linting

[PERF] estate: make invoices in batch

[FIX] estate: typo in variable name

[LINT] estate: adhere to guidelines for xmlid

[FIX] estate: return the super create on overwrite

[LINT] estate: remove unnecessary invisible field

	modified:   estate/views/estate_property_views.xml
	new file:   git-challenge.txt

	modified:   estate/views/estate_property_views.xml

[IMP] estate: minor test cases

[FIX]: review
---
 estate/__init__.py                       |  1 -
 estate/__manifest__.py                   |  4 +-
 estate/models/estate_offer.py            | 56 +++++++++++++-----------
 estate/models/estate_property.py         | 31 +++++++++----
 estate/tests/__init__.py                 |  2 +
 estate/tests/common.py                   | 17 +++++++
 estate/tests/test_estate_offer.py        | 15 +++++++
 estate/tests/test_estate_property.py     | 35 +++++++++++++++
 estate/views/estate_menus.xml            | 10 ++---
 estate/views/estate_offer_views.xml      | 10 ++---
 estate/views/estate_property_views.xml   | 51 +++++++++++++++------
 estate/views/estate_tag_views.xml        |  6 +--
 estate/views/estate_type_views.xml       | 12 ++---
 estate_account/__manifest__.py           |  6 +--
 estate_account/models/estate_property.py | 11 +++--
 15 files changed, 190 insertions(+), 77 deletions(-)
 create mode 100644 estate/tests/__init__.py
 create mode 100644 estate/tests/common.py
 create mode 100644 estate/tests/test_estate_offer.py
 create mode 100644 estate/tests/test_estate_property.py

diff --git a/estate/__init__.py b/estate/__init__.py
index dc5e6b693d1..d6210b1285d 100644
--- a/estate/__init__.py
+++ b/estate/__init__.py
@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
 from . import models
diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 63095a970c4..90aaf27f813 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -1,7 +1,5 @@
-# -*- coding: utf-8 -*-
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
-
 {
     "name": "estate",
     "application": True,
@@ -17,4 +15,6 @@
         "views/estate_menus.xml",
         "views/res_user.xml",
     ],
+    "author": "Odoo S.A.",
+    "license": "LGPL-3",
 }
diff --git a/estate/models/estate_offer.py b/estate/models/estate_offer.py
index 26d0f28b578..cb1cc35fd10 100644
--- a/estate/models/estate_offer.py
+++ b/estate/models/estate_offer.py
@@ -43,44 +43,47 @@ def _compute_date_deadline(self):
                 days=+record.validity,
             )
 
+    @api.onchange("date_deadline")
     def _inverse_date_deadline(self):
         for record in self:
             record.validity = (record.date_deadline - fields.Date.today()).days
 
     # Actions
 
-    def set_status_accepted(self):
-        for record in self:
-            if record.status == "accepted" or record.status == "refused":
-                raise UserError(
-                    "The state of an offer cannot be changed once it is set"
-                )
+    def action_status_accepted(self):
+        self.ensure_one()
+
+        if self.status in ("accepted", "refused"):
+            raise UserError("The state of an offer cannot be changed once it is set")
 
-            if (
-                float_compare(record.price, record.property_id.expected_price * 0.9, 2)
-                == -1
-            ):
-                raise ValidationError(
-                    "Offer has to be at least 90% of the expected price"
-                )
+        if float_compare(self.price, self.property_id.expected_price * 0.9, 2) == -1:
+            raise ValidationError("Offer has to be at least 90% of the expected price")
+
+        if any(
+            self.property_id.offer_ids.filtered(
+                lambda offer: offer.status == "accepted"
+            )
+        ):
+            raise UserError("Only one offer can be accepted")
 
-            for offer in self.property_id.offer_ids:
-                if offer.status == "accepted":
-                    raise UserError("Only one offer can be accepted")
+        self.property_id.state = "accepted"
+        self.status = "accepted"
 
-            record.status = "accepted"
+        for offer in self.property_id.offer_ids:
+            if offer.id == self.id:
+                continue
+
+            offer.status = "refused"
 
         return True
 
-    def set_status_refused(self):
-        for record in self:
-            if record.status == "accepted" or record.status == "refused":
-                raise UserError(
-                    "The state of an offer cannot be changed once it is set"
-                )
+    def action_status_refused(self):
+        self.ensure_one()
 
-            record.status = "refused"
+        if self.status in ["accepted", "refused"]:
+            raise UserError("The state of an offer cannot be changed once it is set")
 
+        self.status = "refused"
         return True
 
     # Overwrites
@@ -89,7 +92,10 @@ def set_status_refused(self):
     def create(self, vals_list):
         for vals in vals_list:
             property = self.env["estate.property"].browse(vals["property_id"])
+            if property.state == "sold":
+                raise UserError("No offer can be made for a sold property")
+
             if property.state == "new":
                 property.state = "received"
 
-        super().create(vals_list)
+        return super().create(vals_list)
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 6ed0991742a..1c3ca052c36 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -73,7 +73,7 @@ class EstateProperty(models.Model):
     )
 
     # Computations
-    @api.depends("offer_ids")
+    @api.depends("offer_ids.status")
     def _compute_selling_price(self):
         for record in self:
             record.selling_price = 0
@@ -82,7 +82,7 @@ def _compute_selling_price(self):
                     record.selling_price = offer.price
                     break
 
-    @api.depends("offer_ids")
+    @api.depends("offer_ids.status")
     def _compute_buyer_id(self):
         for record in self:
             record.buyer_id = None
@@ -100,16 +100,22 @@ def _compute_total_area(self):
     def _compute_best_price(self):
         for record in self:
             # Fallback to 0 if no records are present
-            record.best_price = max(record.offer_ids.mapped("price") + [0])
+            record.best_price = max(record.offer_ids.mapped("price") or [0])
 
     @api.onchange("garden")
     def _onchange_garden(self):
-        self.garden_area = 10 if self.garden else None
-        self.garden_orientation = "north" if self.garden else None
+        self.ensure_one()
+
+        if self.garden:
+            self.garden_area = 10
+            self.garden_orientation = "north"
+        else:
+            self.garden_area = 0
+            self.garden_orientation = False
 
     # Actions
 
-    def set_state_cancel(self):
+    def action_state_cancel(self):
         for record in self:
             if record.state == "sold":
                 raise UserError("A sold property cannot be cancelled")
@@ -117,18 +123,25 @@ def set_state_cancel(self):
                 raise UserError("A cancelled property cannot be cancelled again")
 
             record.state = "cancelled"
-
         return True
 
-    def set_state_sold(self):
+    def action_state_sold(self):
         for record in self:
+            accepted_offer = False
+            for offer in record.offer_ids:
+                if offer.status == "accepted":
+                    accepted_offer = True
+                    break
+
+            if not accepted_offer:
+                raise UserError("A property cannot be sold if no offer is accepted")
+
             if record.state == "cancelled":
                 raise UserError("A cancelled property cannot be sold")
             if record.state == "sold":
                 raise UserError("A sold property cannot be sold again")
 
             record.state = "sold"
-
         return True
 
     # Overwrites
diff --git a/estate/tests/__init__.py b/estate/tests/__init__.py
new file mode 100644
index 00000000000..a46b0574b87
--- /dev/null
+++ b/estate/tests/__init__.py
@@ -0,0 +1,2 @@
+from . import test_estate_offer
+from . import test_estate_property
diff --git a/estate/tests/common.py b/estate/tests/common.py
new file mode 100644
index 00000000000..0f6a21c4abb
--- /dev/null
+++ b/estate/tests/common.py
@@ -0,0 +1,17 @@
+from odoo.tests import TransactionCase
+
+
+class TestEstateCommon(TransactionCase):
+    def setUp(self):
+        super().setUp()
+
+        self.property = self.env["estate.property"].create(
+            {
+                "name": "House 1",
+                "expected_price": 1000,
+                "property_type_id": self.env["estate.type"]
+                .create({"name": "House"})
+                .id,
+                "seller_id": self.env.user.id,
+            }
+        )
diff --git a/estate/tests/test_estate_offer.py b/estate/tests/test_estate_offer.py
new file mode 100644
index 00000000000..700aa2ea2c9
--- /dev/null
+++ b/estate/tests/test_estate_offer.py
@@ -0,0 +1,15 @@
+from odoo.addons.estate.tests.common import TestEstateCommon
+from odoo.exceptions import UserError
+
+
+class TestEstateOffer(TestEstateCommon):
+    def test_create_offer_sold(self):
+        self.property.state = "sold"
+
+        with self.assertRaises(UserError):
+            self.env["estate.offer"].create(
+                {
+                    "partner_id": self.env.user.id,
+                    "property_id": self.property.id,
+                }
+            )
diff --git a/estate/tests/test_estate_property.py b/estate/tests/test_estate_property.py
new file mode 100644
index 00000000000..252bd3fcfcb
--- /dev/null
+++ b/estate/tests/test_estate_property.py
@@ -0,0 +1,35 @@
+from odoo.addons.estate.tests.test_estate_offer import TestEstateOffer
+from odoo.exceptions import UserError
+from odoo.tests import Form, HttpCase, tagged
+
+
+class TestEstateProperty(TestEstateOffer):
+    def test_sell_accepted_offers(self):
+        self.env["estate.offer"].create(
+            {
+                "partner_id": self.env.user.id,
+                "property_id": self.property.id,
+                "status": "accepted",
+            }
+        )
+
+        self.property.action_state_sold()
+        self.assertEqual(self.property.state, "sold")
+
+    def test_sell_no_accepted_offers(self):
+        with self.assertRaises(UserError):
+            self.property.action_state_sold()
+
+
+@tagged("post_install", "-at_install")
+class TestEstatePropertyHttp(HttpCase):
+    def test_garden_reset(self):
+        form = Form(self.env["estate.property"])
+
+        form.garden = True
+        self.assertEqual(form.garden_area, 10)
+        self.assertEqual(form.garden_orientation, "north")
+
+        form.garden = False
+        self.assertEqual(form.garden_area, 0)
+        self.assertEqual(form.garden_orientation, False)
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index b3f9748ea5b..9077b715738 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -1,12 +1,12 @@
 <?xml version="1.0"?>
 <odoo>
     <menuitem id="estate_menu_root" name="Real Estate">
-        <menuitem id="estate_first_menu_advertisements" name="Advertisements">
-            <menuitem id="estate_second_advertisement_advertisements" action="estate_property_action"/>
+        <menuitem id="estate_menu_advertisements" name="Advertisements">
+            <menuitem id="estate_menu_advertisement_advertisements" action="estate_property_action_view"/>
         </menuitem>
-        <menuitem id="estate_first_menu_settings" name="Settings">
-            <menuitem id="estate_second_settings_types" action="estate_type_action"/>
-            <menuitem id="estate_second_settings_tags" action="estate_tag_action"/>
+        <menuitem id="estate_menu_settings" name="Settings">
+            <menuitem id="estate_menu_settings_types" action="estate_type_action_view"/>
+            <menuitem id="estate_menu_settings_tags" action="estate_tag_action_view"/>
         </menuitem>
     </menuitem>
 </odoo>
diff --git a/estate/views/estate_offer_views.xml b/estate/views/estate_offer_views.xml
index 4fa3cabbe16..03a78bbbff1 100644
--- a/estate/views/estate_offer_views.xml
+++ b/estate/views/estate_offer_views.xml
@@ -1,14 +1,14 @@
 <?xml version="1.0"?>
 <odoo>
-    <record id="estate_offer_action" model="ir.actions.act_window">
+    <record id="estate_offer_action_view" model="ir.actions.act_window">
         <field name="name">Offers</field>
         <field name="res_model">estate.offer</field>
         <field name="view_mode">list</field>
         <field name="domain">[('property_type_id', '=', active_id)]</field>
     </record>
 
-    <record id="estate_offer_list" model="ir.ui.view">
-        <field name="name">Offers</field>
+    <record id="estate_offer_view_list" model="ir.ui.view">
+        <field name="name">estate.offer.view.list</field>
         <field name="model">estate.offer</field>
         <field name="arch" type="xml">
             <list editable="top" decoration-success="status == 'accepted'" decoration-danger="status == 'refused'">
@@ -16,8 +16,8 @@
                 <field name="partner_id" />
                 <field name="validity" />
                 <field name="date_deadline" />
-                <button type="object" string="Accept" name="set_status_accepted" icon="fa-check" invisible="status" />
-                <button type="object" string="Refuse" name="set_status_refused" icon="fa-close" invisible="status" />
+                <button type="object" string="Accept" name="action_status_accepted" icon="fa-check" invisible="status" />
+                <button type="object" string="Refuse" name="action_status_refused" icon="fa-close" invisible="status" />
                 <field name="status" />
             </list>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index c993d25c751..3b53f890df2 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -1,20 +1,20 @@
 <?xml version="1.0"?>
 <odoo>
-    <record id="estate_property_action" model="ir.actions.act_window">
+    <record id="estate_property_action_view" model="ir.actions.act_window">
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>
-        <field name="view_mode">list,form</field>
+        <field name="view_mode">list,form,kanban</field>
         <field name="context">{'search_default_available': True}</field>
     </record>
 
-    <record id="estate_property_search" model="ir.ui.view"> 
-        <field name="name">Search</field> 
+    <record id="estate_property_view_search" model="ir.ui.view"> 
+        <field name="name">estate.property.view.search</field> 
         <field name="model">estate.property</field> 
         <field name="arch" type="xml">
         <search>
             <field name="name" string="Estate" />
             <field name="selling_price" />
-            <field string="Living Area (sqm)" name="living_area" filter_domain="[('living_area', '>=', self)]" />
+            <field name="living_area" string="Living Area (sqm)" filter_domain="[('living_area', '%gt;=', self)]" />
             <separator />
             <filter string="Available" name="available" domain="[('state', 'not in', ('sold', 'cancelled'))]"/> 
             <group> 
@@ -24,8 +24,8 @@
         </field> 
     </record> 
 
-    <record id="estate_property_list" model="ir.ui.view">
-        <field name="name">Properties</field>
+    <record id="estate_property_view_list" model="ir.ui.view">
+        <field name="name">estate.property.view.list</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <list decoration-muted="state == 'sold'" decoration-success="state == 'received'" decoration-bf="state == 'accepted'">
@@ -36,20 +36,19 @@
                 <field name="expected_price" />
                 <field name="selling_price" />
                 <field name="date_availability" optional="True" />
-                <field name="state" column_invisible="True" />
             </list>
         </field>
     </record>
 
-    <record id="estate_property_form" model="ir.ui.view">
-        <field name="name">Properties</field>
+    <record id="estate_property_view_form" model="ir.ui.view">
+        <field name="name">estate.property.view.form</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form>
                 <header>
                     <group invisible="state == 'sold' or state == 'cancelled'">
-                    <button type="object" name="set_state_sold" string="Sold" />
-                    <button type="object" name="set_state_cancel" string="Cancel" />
+                    <button type="object" name="action_state_sold" string="Sold" />
+                    <button type="object" name="action_state_cancel" string="Cancel" />
                     </group>
                     <field name="state" string="" widget="statusbar" statusbar_visible="new,received,accepted,sold" />
                 </header>
@@ -61,7 +60,7 @@
                         </group>
                         <group>
                             <group>
-                                <field name="property_type_id" string="Property Type" options="{'no_quick_create': True, 'no_create_edit': True}" />
+                                <field name="property_type_id" options="{'no_quick_create': True, 'no_create_edit': True}" />
                                 <field name="postcode" />
                                 <field name="date_availability" string="Available From" />
                             </group>
@@ -101,4 +100,30 @@
         </field>
     </record>
 
+    <record id="estate_property_view_kanban" model="ir.ui.view">
+        <field name="name">estate.property.view.kanban</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <kanban default_group_by="property_type_id" records_draggable="False">
+                <templates>
+                    <t t-name="card" class="gap-1">
+                        <field name="name" class="fw-bolder text-dark" />
+                        <div>
+                            <span>Expected Price: </span>
+                            <field name="expected_price" />
+                        </div>
+                        <div t-if="record.best_price.raw_value">
+                            <span>Best Price: </span>
+                            <field name="best_price"/>
+                        </div>
+                        <div t-if="record.selling_price.raw_value">
+                            <span>Selling Price: </span>
+                            <field name="selling_price"/>
+                        </div>
+                        <field name="tag_ids" string="" widget="many2many_tags" options="{'color_field': 'color'}" />
+                    </t>
+                </templates>
+            </kanban>
+        </field>
+    </record>
 </odoo>
diff --git a/estate/views/estate_tag_views.xml b/estate/views/estate_tag_views.xml
index f2435d7b573..6a056e4b336 100644
--- a/estate/views/estate_tag_views.xml
+++ b/estate/views/estate_tag_views.xml
@@ -1,13 +1,13 @@
 <?xml version="1.0"?>
 <odoo>
-    <record id="estate_tag_action" model="ir.actions.act_window">
+    <record id="estate_tag_action_view" model="ir.actions.act_window">
         <field name="name">Tags</field>
         <field name="res_model">estate.tag</field>
         <field name="view_mode">list</field>
     </record>
 
-    <record id="estate_tag_list" model="ir.ui.view">
-        <field name="name">Type</field>
+    <record id="estate_tag_view_list" model="ir.ui.view">
+        <field name="name">estate.tag.view.list</field>
         <field name="model">estate.tag</field>
         <field name="arch" type="xml">
             <list editable="top">
diff --git a/estate/views/estate_type_views.xml b/estate/views/estate_type_views.xml
index f40c97d6660..db0fa859ae2 100644
--- a/estate/views/estate_type_views.xml
+++ b/estate/views/estate_type_views.xml
@@ -1,13 +1,13 @@
 <?xml version="1.0"?>
 <odoo>
-    <record id="estate_type_action" model="ir.actions.act_window">
+    <record id="estate_type_action_view" model="ir.actions.act_window">
         <field name="name">Types</field>
         <field name="res_model">estate.type</field>
         <field name="view_mode">list,form</field>
     </record>
 
-    <record id="estate_type_list" model="ir.ui.view">
-        <field name="name">Type</field>
+    <record id="estate_type_view_list" model="ir.ui.view">
+        <field name="name">estate.type.view.list</field>
         <field name="model">estate.type</field>
         <field name="arch" type="xml">
             <list>
@@ -17,14 +17,14 @@
         </field>
     </record>
 
-    <record id="estate_type_form" model="ir.ui.view">
-        <field name="name">Type</field>
+    <record id="estate_type_view_form" model="ir.ui.view">
+        <field name="name">estate.type.view.form</field>
         <field name="model">estate.type</field>
         <field name="arch" type="xml">
             <form>
                 <sheet>
                     <div class="oe_button_box" name="button_box">
-                        <button class="oe_stat_button" type="action" name="%(estate_offer_action)d" icon="fa-money">
+                        <button class="oe_stat_button" type="action" name="%(estate_offer_action_view)d" icon="fa-money">
                             <div class="o_form_field o_stat_info">
                                 <span class="o_stat_text">Offers</span>
                                 <span class="o_stat_value">
diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
index e5a733ca713..34a0e650c51 100644
--- a/estate_account/__manifest__.py
+++ b/estate_account/__manifest__.py
@@ -1,14 +1,12 @@
-# -*- coding: utf-8 -*-
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
-
 {
     "name": "estate_account",
-    "application": False,
     "depends": [
         "base",
         "account",
         "estate",
     ],
-    "data": [],
+    "author": "Odoo S.A.",
+    "license": "LGPL-3",
 }
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py
index 2b15afa78ed..995c2c1fbf9 100644
--- a/estate_account/models/estate_property.py
+++ b/estate_account/models/estate_property.py
@@ -4,11 +4,12 @@
 class EstateProperty(models.Model):
     _inherit = "estate.property"
 
-    def set_state_sold(self):
-        result = super().set_state_sold()
+    def action_state_sold(self):
+        result = super().action_state_sold()
+        invoices = []
 
         for record in self:
-            invoice = self.env["account.move"].create(
+            invoices.append(
                 {
                     "partner_id": record.buyer_id.id,
                     "move_type": "out_invoice",
@@ -30,6 +31,8 @@ def set_state_sold(self):
                     ],
                 }
             )
-            invoice.action_post()
+
+        invoice = self.env["account.move"].create(invoices)
+        invoice.action_post()
 
         return result