From c408da572cd2328531ed38c9f2fa264bca0c89c6 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Tue, 18 Nov 2025 13:36:55 +0100
Subject: [PATCH 01/34] [ADD] Estate: init

---
 estate/__init__.py     |  0
 estate/__manifest__.py | 10 ++++++++++
 2 files changed, 10 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..8f41337d3bb
--- /dev/null
+++ b/estate/__manifest__.py
@@ -0,0 +1,10 @@
+{
+    'name': 'Estate',
+    'version': '0.0',
+    'depends': ['base'],
+    'author': 'haall-odoo',
+    'application': True,
+    'installable': True,
+    'category': '',
+    'description': ''
+}
\ No newline at end of file

From a946449d02821a2881021ef40d481b692becb475 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Tue, 18 Nov 2025 14:14:15 +0100
Subject: [PATCH 02/34] [IMP] estate: chapter 3

---
 estate/__init__.py               |  1 +
 estate/models/__init__.py        |  1 +
 estate/models/estate_property.py | 24 ++++++++++++++++++++++++
 3 files changed, 26 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..9a7e03eded3 100644
--- a/estate/__init__.py
+++ b/estate/__init__.py
@@ -0,0 +1 @@
+from . import models
\ No newline at end of file
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
new file mode 100644
index 00000000000..f4c8fd6db6d
--- /dev/null
+++ b/estate/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_property
\ No newline at end of file
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
new file mode 100644
index 00000000000..739352bd4c1
--- /dev/null
+++ b/estate/models/estate_property.py
@@ -0,0 +1,24 @@
+from odoo import fields, models
+
+
+class EstateProperty(models.Model):
+    _name = "estate.property"
+    _description = "An awesome estate module"
+    
+    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.Integer()
+    garden_orientation = fields.Selection(
+        string='Orientation',
+        selection=[('north', 'North'), ('west', 'West'), ('south', 'South'), ('east', 'East')],
+        help="Choose the appropriate orientation of the garden"
+    )
\ No newline at end of file

From 6deb7a6ce6e20e1b6523d1ce67484ab4ba142825 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Tue, 18 Nov 2025 14:21:07 +0100
Subject: [PATCH 03/34] [FIX] estate: CI PEP8 conformity for EoF

---
 estate/__init__.py               | 2 +-
 estate/__manifest__.py           | 2 +-
 estate/models/__init__.py        | 2 +-
 estate/models/estate_property.py | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/estate/__init__.py b/estate/__init__.py
index 9a7e03eded3..0650744f6bc 100644
--- a/estate/__init__.py
+++ b/estate/__init__.py
@@ -1 +1 @@
-from . import models
\ No newline at end of file
+from . import models
diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 8f41337d3bb..c6e8a1dcedb 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -7,4 +7,4 @@
     'installable': True,
     'category': '',
     'description': ''
-}
\ No newline at end of file
+}
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index f4c8fd6db6d..5e1963c9d2f 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1 +1 @@
-from . import estate_property
\ No newline at end of file
+from . import estate_property
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 739352bd4c1..df04e12c61c 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -21,4 +21,4 @@ class EstateProperty(models.Model):
         string='Orientation',
         selection=[('north', 'North'), ('west', 'West'), ('south', 'South'), ('east', 'East')],
         help="Choose the appropriate orientation of the garden"
-    )
\ No newline at end of file
+    )

From e429205f6296f88961c1d86dd0ce112c27790655 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Tue, 18 Nov 2025 14:57:52 +0100
Subject: [PATCH 04/34] [FIX] estate: PEP8 conformity\nestate_property.py
 whitespace in blankline (L7) deletation

---
 estate/models/estate_property.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index df04e12c61c..8dc7d80f736 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -4,7 +4,7 @@
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "An awesome estate module"
-    
+   
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()

From ffb822c3986555f9d2b2cd7d8ea91f958906ec4f Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Tue, 18 Nov 2025 15:35:24 +0100
Subject: [PATCH 05/34] [IMP] estate: Chapter 4

---
 estate/__manifest__.py              | 5 ++++-
 estate/data/.gitkeep                | 0
 estate/security/ir.model.access.csv | 3 +++
 3 files changed, 7 insertions(+), 1 deletion(-)
 create mode 100644 estate/data/.gitkeep
 create mode 100644 estate/security/ir.model.access.csv

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index c6e8a1dcedb..b81ffed90c7 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -6,5 +6,8 @@
     'application': True,
     'installable': True,
     'category': '',
-    'description': ''
+    'description': '',
+    'data': [
+        'security/ir.model.access.csv'
+    ]
 }
diff --git a/estate/data/.gitkeep b/estate/data/.gitkeep
new file mode 100644
index 00000000000..e69de29bb2d
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
new file mode 100644
index 00000000000..29e50389adb
--- /dev/null
+++ b/estate/security/ir.model.access.csv
@@ -0,0 +1,3 @@
+id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
+access_estate_model,estate_property,model_estate_property,base.group_user,1,1,1,1
+portal_access_estate_model,estate_property,model_estate_property,base.group_portal,1,0,0,0
\ No newline at end of file

From c71e8806b413b776d70c48a52f3d0616e970e302 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Tue, 18 Nov 2025 15:36:34 +0100
Subject: [PATCH 06/34] [FIX] estate: fix whitespace in blank line

---
 estate/models/estate_property.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 8dc7d80f736..9891c284435 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -4,7 +4,7 @@
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "An awesome estate module"
-   
+
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()

From e01993fe68bc7b256b7e8500634aa6883b6ad793 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Tue, 18 Nov 2025 16:12:59 +0100
Subject: [PATCH 07/34] [FIX] estate: license fix

---
 estate/__manifest__.py                 | 5 ++++-
 estate/views/estate_menus.xml          | 8 ++++++++
 estate/views/estate_property_views.xml | 8 ++++++++
 3 files changed, 20 insertions(+), 1 deletion(-)
 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 b81ffed90c7..5b11e6d0deb 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -7,7 +7,10 @@
     'installable': True,
     'category': '',
     'description': '',
+    'license': 'GPL-3',
     'data': [
-        'security/ir.model.access.csv'
+        'security/ir.model.access.csv',
+        'views/estate_property_views.xml',
+        'views/estate_menus.xml'
     ]
 }
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
new file mode 100644
index 00000000000..d10bdc9c22e
--- /dev/null
+++ b/estate/views/estate_menus.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+    <menuitem id="estate_menu_root" name="Estate">
+        <menuitem id="estate_first_level_menu" name="Advertisements">
+            <menuitem id="estate_model_menu_action" action="estate_model_action"/>
+        </menuitem>
+    </menuitem>
+</odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
new file mode 100644
index 00000000000..4696efa2404
--- /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</field>
+    </record>
+</odoo>

From de4a3154eb070288ea84ace4b508b1eb90423151 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Tue, 18 Nov 2025 17:03:55 +0100
Subject: [PATCH 08/34] [IMP] estate: Chapter 5

---
 estate/models/estate_property.py | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 9891c284435..15e7875f041 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,3 +1,5 @@
+import datetime as dt
+
 from odoo import fields, models
 
 
@@ -8,10 +10,10 @@ 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=dt.datetime.today() + dt.timedelta(days=90))
     expected_price = fields.Float(required=True)
-    selling_price = fields.Float()
-    bedrooms = fields.Integer()
+    selling_price = fields.Float(readonly=True, copy=False)
+    bedrooms = fields.Integer(default=2)
     living_area = fields.Integer()
     facades = fields.Integer()
     garage = fields.Boolean()
@@ -22,3 +24,12 @@ class EstateProperty(models.Model):
         selection=[('north', 'North'), ('west', 'West'), ('south', 'South'), ('east', 'East')],
         help="Choose the appropriate orientation of the garden"
     )
+    active = fields.Boolean(default=True)
+    state = fields.Selection(
+        string="Estate status",
+        selection=[('new', 'New'), ('offer received', 'Offer Received'), ('offer accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')],
+        help='This field explain the estate status.',
+        required=True,
+        copy=False,
+        default='new'
+    )

From de78e31a020c382442910f764f2c1b5368f818c5 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Wed, 19 Nov 2025 10:04:19 +0100
Subject: [PATCH 09/34] [IMP] Estate: Chapter 5 Sec 1

---
 estate/views/estate_property_views.xml | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 4696efa2404..a6f6926ba0d 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -1,5 +1,21 @@
 <?xml version="1.0"?>
 <odoo>
+    <record id="estate_property_view_tree" model="ir.ui.view">
+        <field name="name">estate.property.name</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <list string="Channel">
+                <field name="name" string="Title"/>
+                <field name="postcode"/>
+                <field name="bedrooms"/>
+                <field name="living_area" string="Living Area (sqm)"/>
+                <field name="expected_price"/>
+                <field name="selling_price"/>
+                <field name="date_availability" string="Available From"/>
+            </list>
+        </field>
+    </record>
+
     <record id="estate_model_action" model="ir.actions.act_window">
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>

From adb33160a95a1ce24403a36fd009ad6013b1b396 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Wed, 19 Nov 2025 11:08:26 +0100
Subject: [PATCH 10/34] [IMP] Estate: Chapter 6

---
 estate/views/estate_property_views.xml | 56 ++++++++++++++++++++++++++
 1 file changed, 56 insertions(+)

diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index a6f6926ba0d..ec9ead5d5cb 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -16,6 +16,62 @@
         </field>
     </record>
 
+    <record id="estate_property_form_view" model="ir.ui.view">
+        <field name="name">estate.property.name</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <form string="My new house">
+                <sheet>
+                    <group>
+                        <h1><field name="name" string="" placeholder="My new house"/></h1>
+                    </group>
+                    <group>
+                        <group>
+                            <field name="postcode" default="1000"/>
+                            <field name="date_availability" string="Available From"/>
+                        </group>
+                        <group>
+                            <field name="expected_price" default="150000"/>
+                            <field name="selling_price"/>
+                        </group>
+                    </group>
+                    <notebook>
+                        <page string="Description">
+                            <group>
+                                <field name="description" help="When duplicated, status and date are not copied"/>
+                                <field name="bedrooms" default="4"/>
+                                <field name="living_area" string="Living Area (sqm)"/>
+                                <field name="facades" default="4"/>
+                                <field name="garage"/>
+                                <field name="garden" default="True"/>
+                                <field name="garden_area" string="Garden Area (sqm)"/>
+                                <field name="garden_orientation"/>
+                            </group>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+    <record id="estate_property_search_view" model="ir.ui.view">
+        <field name="name">estate.property.name</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <search string="TestSearch">
+                <field name="name" string="Title"/>
+                <field name="postcode"/>
+                <field name="expected_price"/>
+                <field name="bedrooms"/>
+                <field name="living_area" string="Living Area (sqm)"/>
+                <field name="facades"/>
+                <separator/>
+                <filter name="Available" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer received')]"/>
+                <filter name="Postcode" context="{'group_by': 'postcode'}"/>
+            </search>
+        </field>
+    </record>
+
     <record id="estate_model_action" model="ir.actions.act_window">
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>

From e66dd36760a40db0c3f5b66cdb9cb7a8fea1c2f8 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Wed, 19 Nov 2025 13:21:07 +0100
Subject: [PATCH 11/34] [FIX] estate: review 1

---
 estate/__manifest__.py                      |  7 ++--
 estate/data/.gitkeep                        |  0
 estate/models/estate_property.py            | 16 ++++++---
 estate/security/ir.model.access.csv         |  5 +--
 estate/views/estate_menus.xml               |  7 ++--
 estate/views/estate_property_type_views.xml | 22 ++++++++++++
 estate/views/estate_property_views.xml      | 37 +++++++++++++--------
 7 files changed, 69 insertions(+), 25 deletions(-)
 delete mode 100644 estate/data/.gitkeep
 create mode 100644 estate/views/estate_property_type_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 5b11e6d0deb..0f6f9233932 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -1,16 +1,17 @@
 {
     'name': 'Estate',
-    'version': '0.0',
+    'version': '1.0',
     'depends': ['base'],
-    'author': 'haall-odoo',
+    'author': 'Odoo S.A.',
     'application': True,
     'installable': True,
     'category': '',
     'description': '',
-    'license': 'GPL-3',
+    'license': 'LGPL-3',
     'data': [
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
+        'views/estate_property_type_views.xml',
         'views/estate_menus.xml'
     ]
 }
diff --git a/estate/data/.gitkeep b/estate/data/.gitkeep
deleted file mode 100644
index e69de29bb2d..00000000000
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 15e7875f041..411e100dcf6 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -5,12 +5,12 @@
 
 class EstateProperty(models.Model):
     _name = "estate.property"
-    _description = "An awesome estate module"
+    _description = "Estate Property"
 
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()
-    date_availability = fields.Date(copy=False, default=dt.datetime.today() + dt.timedelta(days=90))
+    date_availability = fields.Date(copy=False, default=fields.Date.add(fields.Date.today(), months=3), string="Available From")
     expected_price = fields.Float(required=True)
     selling_price = fields.Float(readonly=True, copy=False)
     bedrooms = fields.Integer(default=2)
@@ -21,15 +21,23 @@ class EstateProperty(models.Model):
     garden_area = fields.Integer()
     garden_orientation = fields.Selection(
         string='Orientation',
-        selection=[('north', 'North'), ('west', 'West'), ('south', 'South'), ('east', 'East')],
+        selection=[
+                ('north', 'North'),
+                ('west', 'West'),
+                ('south', 'South'),
+                ('east', 'East')
+            ],
         help="Choose the appropriate orientation of the garden"
     )
     active = fields.Boolean(default=True)
     state = fields.Selection(
         string="Estate status",
-        selection=[('new', 'New'), ('offer received', 'Offer Received'), ('offer accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')],
+        selection=[('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')],
         help='This field explain the estate status.',
         required=True,
         copy=False,
         default='new'
     )
+    property_type_id = fields.Many2one("estate.property.type", string="Type")
+    seller_id = fields.Many2one("res.users", string="Salesman", default=lambda self: self.env.user,  domain="[('type', '=', 'internal')]")
+    buyer_id = fields.Many2one("res.partner", string="Buyer", domain="[('type', '=', 'portal')]")
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index 29e50389adb..d0eb72f0fc1 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -1,3 +1,4 @@
 id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
-access_estate_model,estate_property,model_estate_property,base.group_user,1,1,1,1
-portal_access_estate_model,estate_property,model_estate_property,base.group_portal,1,0,0,0
\ No newline at end of file
+access_estate_model,estate_property_users,model_estate_property,base.group_user,1,1,1,1
+access_estate_model_type,estate_property_type_users,model_estate_property_type,base.group_user,1,1,1,1
+portal_access_estate_model,estate_property_portal,model_estate_property,base.group_portal,1,0,0,0
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index d10bdc9c22e..1b04e166d8f 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -2,7 +2,10 @@
 <odoo>
     <menuitem id="estate_menu_root" name="Estate">
         <menuitem id="estate_first_level_menu" name="Advertisements">
-            <menuitem id="estate_model_menu_action" action="estate_model_action"/>
+            <menuitem id="estate_property_menu_action" action="estate_property_action"/>
+        </menuitem>
+        <menuitem id="estate_preoperty_type_settings_menu" name="Settings">
+            <menuitem id="estate_property_type_menu" action="estate_property_type_action"/>
         </menuitem>
     </menuitem>
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
new file mode 100644
index 00000000000..dd393e37095
--- /dev/null
+++ b/estate/views/estate_property_type_views.xml
@@ -0,0 +1,22 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_property_type_form_view" model="ir.ui.view">
+        <field name="name">estate.property.type.name</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <form string="New Type">
+                <sheet>
+                    <group>
+                        <h1><field name="name" string="" placeholder="New property type"/></h1>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+    <record id="estate_model_type_action" model="ir.actions.act_window">
+        <field name="name">Property Types</field>
+        <field name="res_model">estate.property.type</field>
+        <field name="view_mode">list</field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index ec9ead5d5cb..5223ef32829 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -1,10 +1,10 @@
 <?xml version="1.0"?>
 <odoo>
-    <record id="estate_property_view_tree" model="ir.ui.view">
-        <field name="name">estate.property.name</field>
+    <record id="estate_property_view_list" model="ir.ui.view">
+        <field name="name">estate.property.list</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
-            <list string="Channel">
+            <list string="Properties">
                 <field name="name" string="Title"/>
                 <field name="postcode"/>
                 <field name="bedrooms"/>
@@ -17,21 +17,22 @@
     </record>
 
     <record id="estate_property_form_view" model="ir.ui.view">
-        <field name="name">estate.property.name</field>
+        <field name="name">estate.property.form</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form string="My new house">
                 <sheet>
                     <group>
-                        <h1><field name="name" string="" placeholder="My new house"/></h1>
+                        <h1><field name="name" nolabel="1" placeholder="My new house"/></h1>
                     </group>
                     <group>
                         <group>
-                            <field name="postcode" default="1000"/>
+                            <field name="property_type_id" string="Property Type"/>
+                            <field name="postcode"/>
                             <field name="date_availability" string="Available From"/>
                         </group>
                         <group>
-                            <field name="expected_price" default="150000"/>
+                            <field name="expected_price"/>
                             <field name="selling_price"/>
                         </group>
                     </group>
@@ -39,15 +40,21 @@
                         <page string="Description">
                             <group>
                                 <field name="description" help="When duplicated, status and date are not copied"/>
-                                <field name="bedrooms" default="4"/>
+                                <field name="bedrooms"/>
                                 <field name="living_area" string="Living Area (sqm)"/>
-                                <field name="facades" default="4"/>
+                                <field name="facades"/>
                                 <field name="garage"/>
-                                <field name="garden" default="True"/>
+                                <field name="garden"/>
                                 <field name="garden_area" string="Garden Area (sqm)"/>
                                 <field name="garden_orientation"/>
                             </group>
                         </page>
+                        <page string="Other Info">
+                            <group>
+                                <field name="seller_id" string="Salesman"/>
+                                <field name="buyer_id" string="Buyer"/>
+                            </group>
+                        </page>
                     </notebook>
                 </sheet>
             </form>
@@ -55,19 +62,21 @@
     </record>
 
     <record id="estate_property_search_view" model="ir.ui.view">
-        <field name="name">estate.property.name</field>
+        <field name="name">estate.property.search</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
-            <search string="TestSearch">
+            <search string="Search Properties">
                 <field name="name" string="Title"/>
                 <field name="postcode"/>
                 <field name="expected_price"/>
                 <field name="bedrooms"/>
                 <field name="living_area" string="Living Area (sqm)"/>
                 <field name="facades"/>
+                <field name="property_type_id" string="Type"/>
                 <separator/>
-                <filter name="Available" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer received')]"/>
+                <filter name="Available" domain="[('state', 'in', ('new', 'offer_received'))]"/>
                 <filter name="Postcode" context="{'group_by': 'postcode'}"/>
+                <filter name="Type" context="{'group_by': 'property_type_id'}"/>
             </search>
         </field>
     </record>
@@ -75,6 +84,6 @@
     <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</field>
+        <field name="view_mode">list, form</field>
     </record>
 </odoo>

From 55064b2cb762242dd5c012daabba3d6ad7aee298 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Wed, 19 Nov 2025 13:40:57 +0100
Subject: [PATCH 12/34] [FIX] estate: review fix 2 + chapter 7 part 1

---
 estate/__manifest__.py                      |  1 +
 estate/models/__init__.py                   |  2 ++
 estate/models/estate_property.py            |  4 +---
 estate/models/estate_property_tag.py        |  8 ++++++++
 estate/models/estate_property_type.py       |  7 +++++++
 estate/security/ir.model.access.csv         |  1 +
 estate/views/estate_menus.xml               |  4 +++-
 estate/views/estate_property_tag_views.xml  | 22 +++++++++++++++++++++
 estate/views/estate_property_type_views.xml |  6 +++---
 estate/views/estate_property_views.xml      |  4 ++--
 10 files changed, 50 insertions(+), 9 deletions(-)
 create mode 100644 estate/models/estate_property_tag.py
 create mode 100644 estate/models/estate_property_type.py
 create mode 100644 estate/views/estate_property_tag_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 0f6f9233932..27020c34467 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -12,6 +12,7 @@
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
         'views/estate_property_type_views.xml',
+        'views/estate_property_tag_views.xml',
         'views/estate_menus.xml'
     ]
 }
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 5e1963c9d2f..fdab6f99948 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1 +1,3 @@
 from . import estate_property
+from . import estate_property_type
+from . import estate_property_tag
\ No newline at end of file
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 411e100dcf6..f820cbdcb7d 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,5 +1,3 @@
-import datetime as dt
-
 from odoo import fields, models
 
 
@@ -39,5 +37,5 @@ class EstateProperty(models.Model):
         default='new'
     )
     property_type_id = fields.Many2one("estate.property.type", string="Type")
-    seller_id = fields.Many2one("res.users", string="Salesman", default=lambda self: self.env.user,  domain="[('type', '=', 'internal')]")
+    seller_id = fields.Many2one("res.users", string="Salesman", default=lambda self: self.env.user, domain="[('type', '=', 'internal')]")
     buyer_id = fields.Many2one("res.partner", string="Buyer", domain="[('type', '=', 'portal')]")
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
new file mode 100644
index 00000000000..c7e1caaf155
--- /dev/null
+++ b/estate/models/estate_property_tag.py
@@ -0,0 +1,8 @@
+from odoo import fields, models
+
+
+class EstatePropertyTag(models.Model):
+    _name = "estate.property.tag"
+    _description = "Estate Property Tag"
+
+    name = fields.Char(required=True)
\ No newline at end of file
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
new file mode 100644
index 00000000000..b6306e34aa1
--- /dev/null
+++ b/estate/models/estate_property_type.py
@@ -0,0 +1,7 @@
+from odoo import fields, models
+
+class EstatePropertyType(models.Model):
+    _name = "estate.property.type"
+    _description = "Estate Property Types"
+
+    name = fields.Char(required=True)
\ No newline at end of file
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index d0eb72f0fc1..193b5d0e052 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -2,3 +2,4 @@ id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
 access_estate_model,estate_property_users,model_estate_property,base.group_user,1,1,1,1
 access_estate_model_type,estate_property_type_users,model_estate_property_type,base.group_user,1,1,1,1
 portal_access_estate_model,estate_property_portal,model_estate_property,base.group_portal,1,0,0,0
+access_estate_model_tag,estate_property_tag_users,model_estate_property_tag,base.group_user,1,1,1,1
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index 1b04e166d8f..ba6a2f8f00a 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -4,8 +4,10 @@
         <menuitem id="estate_first_level_menu" name="Advertisements">
             <menuitem id="estate_property_menu_action" action="estate_property_action"/>
         </menuitem>
-        <menuitem id="estate_preoperty_type_settings_menu" name="Settings">
+
+        <menuitem id="estate_property_type_settings_menu" name="Settings">
             <menuitem id="estate_property_type_menu" action="estate_property_type_action"/>
+            <menuitem id="estate_property_tag_menu" action="estate_property_tag_action"/>
         </menuitem>
     </menuitem>
 </odoo>
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
new file mode 100644
index 00000000000..b8d02cd0cc9
--- /dev/null
+++ b/estate/views/estate_property_tag_views.xml
@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+    <record id="estate_property_tag_form_view" model="ir.ui.view">
+        <field name="name">estate.property.tag.form</field>
+        <field name="model">estate.property.tag</field>
+        <field name="arch" type="xml">
+            <form string="New Tag">
+                <sheet>
+                    <group>
+                        <h1><field name="name" placeholder="New property tag"/></h1>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+    <record id="estate_property_tag_action" model="ir.actions.act_window">
+        <field name="name">Property Tags</field>
+        <field name="res_model">estate.property.tag</field>
+        <field name="view_mode">list</field>
+    </record>
+</odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index dd393e37095..d124ae8d555 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -1,20 +1,20 @@
 <?xml version="1.0"?>
 <odoo>
     <record id="estate_property_type_form_view" model="ir.ui.view">
-        <field name="name">estate.property.type.name</field>
+        <field name="name">estate.property.type.form</field>
         <field name="model">estate.property.type</field>
         <field name="arch" type="xml">
             <form string="New Type">
                 <sheet>
                     <group>
-                        <h1><field name="name" string="" placeholder="New property type"/></h1>
+                        <h1><field name="name" nolabel="1" placeholder="New property type"/></h1>
                     </group>
                 </sheet>
             </form>
         </field>
     </record>
 
-    <record id="estate_model_type_action" model="ir.actions.act_window">
+    <record id="estate_property_type_action" model="ir.actions.act_window">
         <field name="name">Property Types</field>
         <field name="res_model">estate.property.type</field>
         <field name="view_mode">list</field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 5223ef32829..fc7d8030783 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -81,9 +81,9 @@
         </field>
     </record>
 
-    <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>
+        <field name="view_mode">list,form</field>
     </record>
 </odoo>

From 218b8f0c75d60e811a9ccb6e17801a00895bdd37 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Wed, 19 Nov 2025 14:17:09 +0100
Subject: [PATCH 13/34] [IMP] estate: chapter 7

---
 estate/__manifest__.py                       |  1 +
 estate/models/__init__.py                    |  3 ++-
 estate/models/estate_property.py             |  2 ++
 estate/models/estate_property_offer.py       | 18 ++++++++++++++++++
 estate/models/estate_property_tag.py         |  2 +-
 estate/models/estate_property_type.py        |  3 ++-
 estate/security/ir.model.access.csv          |  1 +
 estate/views/estate_property_offer_views.xml | 14 ++++++++++++++
 estate/views/estate_property_tag_views.xml   |  2 +-
 estate/views/estate_property_views.xml       |  6 ++++++
 10 files changed, 48 insertions(+), 4 deletions(-)
 create mode 100644 estate/models/estate_property_offer.py
 create mode 100644 estate/views/estate_property_offer_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 27020c34467..93b455da68a 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -13,6 +13,7 @@
         'views/estate_property_views.xml',
         'views/estate_property_type_views.xml',
         'views/estate_property_tag_views.xml',
+        'views/estate_property_offer_views.xml',
         'views/estate_menus.xml'
     ]
 }
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index fdab6f99948..09b2099fe84 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1,3 +1,4 @@
 from . import estate_property
 from . import estate_property_type
-from . import estate_property_tag
\ No newline at end of file
+from . import estate_property_tag
+from . import estate_property_offer
\ No newline at end of file
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index f820cbdcb7d..57beaa5d9f7 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -39,3 +39,5 @@ class EstateProperty(models.Model):
     property_type_id = fields.Many2one("estate.property.type", string="Type")
     seller_id = fields.Many2one("res.users", string="Salesman", default=lambda self: self.env.user, domain="[('type', '=', 'internal')]")
     buyer_id = fields.Many2one("res.partner", string="Buyer", domain="[('type', '=', 'portal')]")
+    tag_ids = fields.Many2many("estate.property.tag")
+    offer_ids = fields.One2many("estate.property.offer", "property_id")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
new file mode 100644
index 00000000000..5238615fe1e
--- /dev/null
+++ b/estate/models/estate_property_offer.py
@@ -0,0 +1,18 @@
+from odoo import fields, models
+
+
+class EstatePropertyOffer(models.Model):
+    _name = "estate.property.offer"
+    _description = "Estate Property Offer"
+
+    price = fields.Float()
+    status = fields.Selection(
+        string="Status",
+        copy=False,
+        selection = [
+            ('accepted', 'Accepted'),
+            ('refused', 'Refused')
+        ]
+    )
+    partner_id = fields.Many2one("res.partner", required=True)
+    property_id = fields.Many2one("estate.property", required=True)
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index c7e1caaf155..165132d19de 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -5,4 +5,4 @@ class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Estate Property Tag"
 
-    name = fields.Char(required=True)
\ No newline at end of file
+    name = fields.Char(required=True)
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index b6306e34aa1..d7e69e557ef 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -1,7 +1,8 @@
 from odoo import fields, models
 
+
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Estate Property Types"
 
-    name = fields.Char(required=True)
\ No newline at end of file
+    name = fields.Char(required=True)
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index 193b5d0e052..d375e4a2ac5 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -3,3 +3,4 @@ access_estate_model,estate_property_users,model_estate_property,base.group_user,
 access_estate_model_type,estate_property_type_users,model_estate_property_type,base.group_user,1,1,1,1
 portal_access_estate_model,estate_property_portal,model_estate_property,base.group_portal,1,0,0,0
 access_estate_model_tag,estate_property_tag_users,model_estate_property_tag,base.group_user,1,1,1,1
+access_estate_model_offer,estate_property_offer_users,model_estate_property_offer,base.group_user,1,1,1,1
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
new file mode 100644
index 00000000000..e551a5243a9
--- /dev/null
+++ b/estate/views/estate_property_offer_views.xml
@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+    <record id="estate_property_offer_list_view" model="ir.ui.view">
+        <field name="name">estate.property.offer.list</field>
+        <field name="model">estate.property.offer</field>
+        <field name="arch" type="xml">
+            <list string="Properties">
+                <field name="price" />
+                <field name="partner_id"/>
+                <field name="status"/>
+            </list>
+        </field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
index b8d02cd0cc9..8fb328889f9 100644
--- a/estate/views/estate_property_tag_views.xml
+++ b/estate/views/estate_property_tag_views.xml
@@ -19,4 +19,4 @@
         <field name="res_model">estate.property.tag</field>
         <field name="view_mode">list</field>
     </record>
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index fc7d8030783..e042199fd17 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -26,6 +26,9 @@
                         <h1><field name="name" nolabel="1" placeholder="My new house"/></h1>
                     </group>
                     <group>
+                        <field name="tag_ids" nolabel="1" widget="many2many_tags"/>
+                        <br/>
+                        <br/>
                         <group>
                             <field name="property_type_id" string="Property Type"/>
                             <field name="postcode"/>
@@ -49,6 +52,9 @@
                                 <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"/>

From ff73e1baa819d60b32c41070e5113d704d28cab3 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Wed, 19 Nov 2025 14:45:44 +0100
Subject: [PATCH 14/34] [FIX] estate: PEP8 fix

---
 estate/models/__init__.py              | 2 +-
 estate/models/estate_property_offer.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 09b2099fe84..2f1821a39c1 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1,4 +1,4 @@
 from . import estate_property
 from . import estate_property_type
 from . import estate_property_tag
-from . import estate_property_offer
\ No newline at end of file
+from . import estate_property_offer
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 5238615fe1e..5ff9ea236ff 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -9,7 +9,7 @@ class EstatePropertyOffer(models.Model):
     status = fields.Selection(
         string="Status",
         copy=False,
-        selection = [
+        selection=[
             ('accepted', 'Accepted'),
             ('refused', 'Refused')
         ]

From 871dc8d6dbd6eb0e5dfd7c3d80a9898e1c3a1217 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Wed, 19 Nov 2025 15:33:45 +0100
Subject: [PATCH 15/34] [IMP] estate: add computed field for total_area and
 best_price

---
 estate/models/estate_property.py             | 14 +++++++++++++-
 estate/views/estate_property_offer_views.xml |  2 +-
 estate/views/estate_property_views.xml       |  2 ++
 3 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 57beaa5d9f7..27d0a2b93be 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class EstateProperty(models.Model):
@@ -41,3 +41,15 @@ class EstateProperty(models.Model):
     buyer_id = fields.Many2one("res.partner", string="Buyer", domain="[('type', '=', 'portal')]")
     tag_ids = fields.Many2many("estate.property.tag")
     offer_ids = fields.One2many("estate.property.offer", "property_id")
+    total_area = fields.Float(compute="_compute_total_area", string="Total Area (sqm)")
+    best_price = fields.Float(compute="_compute_best_offer", string="Best Offer")
+
+    @api.depends("garden_area", "living_area")
+    def _compute_total_area(self):
+        for record in self:
+            record.total_area = record.garden_area + record.living_area
+
+    @api.depends("offer_ids")
+    def _compute_best_offer(self):
+        for record in self:
+            record.best_price = max(record.offer_ids.mapped("price"))    
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index e551a5243a9..e3e1106bb6b 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -4,7 +4,7 @@
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
-            <list string="Properties">
+            <list string="Offers">
                 <field name="price" />
                 <field name="partner_id"/>
                 <field name="status"/>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index e042199fd17..aad30042a6b 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -36,6 +36,7 @@
                         </group>
                         <group>
                             <field name="expected_price"/>
+                            <field name="best_price" string="Best Offer"/>
                             <field name="selling_price"/>
                         </group>
                     </group>
@@ -50,6 +51,7 @@
                                 <field name="garden"/>
                                 <field name="garden_area" string="Garden Area (sqm)"/>
                                 <field name="garden_orientation"/>
+                                <field name="total_area"/>
                             </group>
                         </page>
                         <page string="Offers">

From 640793564044cafbf84a7e2873e043523f6095b7 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Wed, 19 Nov 2025 16:15:59 +0100
Subject: [PATCH 16/34] [IMP] estate: chapter 8 reversed field

---
 estate/models/estate_property.py       |  2 +-
 estate/models/estate_property_offer.py | 14 +++++++++++++-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 27d0a2b93be..77c047a0e91 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -52,4 +52,4 @@ def _compute_total_area(self):
     @api.depends("offer_ids")
     def _compute_best_offer(self):
         for record in self:
-            record.best_price = max(record.offer_ids.mapped("price"))    
+            record.best_price = max(record.offer_ids.mapped("price")) if len(record.offer_ids) > 0 else 0.0
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 5ff9ea236ff..a7352849bba 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class EstatePropertyOffer(models.Model):
@@ -16,3 +16,15 @@ class EstatePropertyOffer(models.Model):
     )
     partner_id = fields.Many2one("res.partner", required=True)
     property_id = fields.Many2one("estate.property", required=True)
+    create_date = fields.Date(readonly=True, default=fields.Date.today())
+    validity = fields.Integer(default=7)
+    date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_reverse_date_deadline")
+
+    @api.depends("validity")
+    def _compute_date_deadline(self):
+        for record in self:
+            record.date_deadline = fields.Date.add(record.create_date if record.create_date is not None else fields.Date.today(), days=record.validity)
+
+    def _reverse_date_deadline(self):
+        for record in self:
+            record.validity = fields.Date.substract(record.date_deadline, )

From 0b791abb168ce5bd0efedf771f992866620e380a Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 09:28:27 +0100
Subject: [PATCH 17/34] [FIX] gitignore: ignore vscode config

---
 .gitignore | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/.gitignore b/.gitignore
index b6e47617de1..61a0ca00289 100644
--- a/.gitignore
+++ b/.gitignore
@@ -127,3 +127,6 @@ dmypy.json
 
 # Pyre type checker
 .pyre/
+
+
+.vscode/

From b5fc120fa81ae82e612e274fc08d3c61445c1eae Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 09:30:49 +0100
Subject: [PATCH 18/34] [IMP] estate: chapter 8

---
 estate/models/estate_property.py       | 10 ++++++++++
 estate/models/estate_property_offer.py | 10 +++++-----
 2 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 77c047a0e91..a1139d4ea75 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -53,3 +53,13 @@ def _compute_total_area(self):
     def _compute_best_offer(self):
         for record in self:
             record.best_price = max(record.offer_ids.mapped("price")) if len(record.offer_ids) > 0 else 0.0
+
+    @api.onchange("garden")
+    def _onchange_garden(self):
+        for record in self:
+            if not record.garden:
+                record.garden_area = 0
+                record.garden_orientation = ''
+            else:
+                record.garden_area = 10
+                record.garden_orientation = 'north'
\ No newline at end of file
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index a7352849bba..efd66819837 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -16,15 +16,15 @@ class EstatePropertyOffer(models.Model):
     )
     partner_id = fields.Many2one("res.partner", required=True)
     property_id = fields.Many2one("estate.property", required=True)
-    create_date = fields.Date(readonly=True, default=fields.Date.today())
     validity = fields.Integer(default=7)
-    date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_reverse_date_deadline")
+    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 = fields.Date.add(record.create_date if record.create_date is not None else fields.Date.today(), days=record.validity)
+            record.date_deadline = fields.Date.add(fields.Date.today(), days=record.validity)
 
-    def _reverse_date_deadline(self):
+    @api.onchange("date_deadline")
+    def _inverse_date_deadline(self):
         for record in self:
-            record.validity = fields.Date.substract(record.date_deadline, )
+            record.validity = (record.date_deadline - fields.Date.today()).days

From ac39bedc01f688d366e8bf5ea87906a6c2909587 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 09:35:43 +0100
Subject: [PATCH 19/34] [FIX] estate: estate_property.py - EOL

---
 estate/models/estate_property.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index a1139d4ea75..327dafea56c 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -62,4 +62,4 @@ def _onchange_garden(self):
                 record.garden_orientation = ''
             else:
                 record.garden_area = 10
-                record.garden_orientation = 'north'
\ No newline at end of file
+                record.garden_orientation = 'north'

From ee8a1bee732a55a74f506812fc883fa22f464e12 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 11:22:47 +0100
Subject: [PATCH 20/34] [IMP] estate: chapter 9

---
 estate/models/estate_property.py             | 19 +++++++++++++++++++
 estate/models/estate_property_offer.py       | 17 +++++++++++++++++
 estate/views/estate_property_offer_views.xml |  2 ++
 estate/views/estate_property_views.xml       |  4 ++++
 4 files changed, 42 insertions(+)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 327dafea56c..61f6a1b04d0 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,5 @@
 from odoo import api, fields, models
+from odoo.exceptions import UserError
 
 
 class EstateProperty(models.Model):
@@ -63,3 +64,21 @@ def _onchange_garden(self):
             else:
                 record.garden_area = 10
                 record.garden_orientation = 'north'
+
+    def estate_property_action_sold(self):
+        self.__estate_property_action_sold_cancel('sold', "A cancelled property cannot be sold!", "This property is already sold!")
+
+    def estate_property_action_cancel(self):
+        self.__estate_property_action_sold_cancel('cancelled', "A sold property cannot be cancelled!", "This property is already cancelled!")
+
+    def __estate_property_action_sold_cancel(self, target, error_message, error_same_target_message):
+        for record in self:
+            # exclude target from next condition
+            if record.state == target:
+                raise UserError(error_same_target_message)
+            # easiest way to exclude the other state
+            elif record.state in ('sold', 'cancelled'):
+                raise UserError(error_message)
+            # the property can be sold/cancelled
+            else:
+                record.state = target
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index efd66819837..9e4b2194530 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -28,3 +28,20 @@ def _compute_date_deadline(self):
     def _inverse_date_deadline(self):
         for record in self:
             record.validity = (record.date_deadline - fields.Date.today()).days
+
+    def action_accept_offer(self):
+        for record in self:
+            record.property_id.selling_price = record.price
+            record.property_id.buyer_id = record.partner_id
+            for offer in record.property_id.offer_ids:
+                offer.status = "refused"
+            record.status = 'accepted'
+        return True
+
+    def action_refuse_offer(self):
+        for record in self:
+            if record.status == 'accepted':
+                record.property_id.selling_price = 0.0
+                record.property_id.buyer_id = None
+            record.status = 'refused'
+        return True
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index e3e1106bb6b..e76bab386dd 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -7,6 +7,8 @@
             <list string="Offers">
                 <field name="price" />
                 <field name="partner_id"/>
+                <button type="object" name="action_accept_offer" icon="fa-check"/>
+                <button type="object" name="action_refuse_offer" icon="fa-times"/>
                 <field name="status"/>
             </list>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index aad30042a6b..15187148a75 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -21,6 +21,10 @@
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form string="My new house">
+                <header>
+                    <button name="estate_property_action_sold" type="object" string="Sold"/>
+                    <button name="estate_property_action_cancel" type="object" string="Cancel"/>
+                </header>
                 <sheet>
                     <group>
                         <h1><field name="name" nolabel="1" placeholder="My new house"/></h1>

From 0faad122b6df4f817faf0c9366c086903330ff05 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 13:15:07 +0100
Subject: [PATCH 21/34] [FIX] estate: chapter 9

---
 estate/models/estate_property.py             | 8 ++++++++
 estate/models/estate_property_offer.py       | 4 ++++
 estate/models/estate_property_tag.py         | 4 ++++
 estate/views/estate_property_offer_views.xml | 4 ++--
 4 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 61f6a1b04d0..f6d529a37a0 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -5,6 +5,14 @@
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Estate Property"
+    _check_expected_price = models.Constraint(
+        'CHECK(expected_price > 0)',
+        'The expected price should be stricly positive'
+    )
+    _check_selling_price = models.Constraint(
+        'CHECK(selling_price >= 0)',
+        'The selling price should be positive'
+    )
 
     name = fields.Char(required=True)
     description = fields.Text()
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 9e4b2194530..0e91cbeffe9 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -4,6 +4,10 @@
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Estate Property Offer"
+    _check_offer_price = models.Constraint(
+        'CHECK(price > 0)',
+        'The offer price should be stricly positive'
+    )
 
     price = fields.Float()
     status = fields.Selection(
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 165132d19de..56efc9417a0 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -4,5 +4,9 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Estate Property Tag"
+    _check_name = models.Constraint(
+        'UNIQUE(name)',
+        'The tag name should be unique!'
+    )
 
     name = fields.Char(required=True)
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index e76bab386dd..b7ba2a6c118 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -7,8 +7,8 @@
             <list string="Offers">
                 <field name="price" />
                 <field name="partner_id"/>
-                <button type="object" name="action_accept_offer" icon="fa-check"/>
-                <button type="object" name="action_refuse_offer" icon="fa-times"/>
+                <button type="object" name="action_accept_offer" title="accept" icon="fa-check"/>
+                <button type="object" name="action_refuse_offer" title="refuse" icon="fa-times"/>
                 <field name="status"/>
             </list>
         </field>

From be0cfdf7a8a4f01a92b7eb99e2ecd9a7e1976b85 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 13:56:37 +0100
Subject: [PATCH 22/34] [IMP] estate: chapter 10

---
 estate/models/estate_property.py             | 9 ++++++++-
 estate/models/estate_property_offer.py       | 3 +++
 estate/models/estate_property_tag.py         | 6 ++----
 estate/views/estate_property_offer_views.xml | 2 ++
 4 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index f6d529a37a0..3454ef9111f 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,5 +1,6 @@
 from odoo import api, fields, models
-from odoo.exceptions import UserError
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools.float_utils import float_compare
 
 
 class EstateProperty(models.Model):
@@ -90,3 +91,9 @@ def __estate_property_action_sold_cancel(self, target, error_message, error_same
             # the property can be sold/cancelled
             else:
                 record.state = target
+
+    @api.constrains('selling_price')
+    def _check_price_constraint(self):
+        for record in self:
+            if float_compare(record.selling_price, record.expected_price * 0.9, precision_digits=4) < 0:
+                raise ValidationError("The price cannot be les than 90% of the expected price")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 0e91cbeffe9..48c5919cb4f 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -45,7 +45,10 @@ def action_accept_offer(self):
     def action_refuse_offer(self):
         for record in self:
             if record.status == 'accepted':
+                val = record.property_id.expected_price # bypass 90% constraint L1
+                record.property_id.expected_price = 0.0 # bypass 90% constraint L2
                 record.property_id.selling_price = 0.0
+                record.property_id.expected_price = val # bypass 90% constraint L3
                 record.property_id.buyer_id = None
             record.status = 'refused'
         return True
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 56efc9417a0..20ab0fcffa4 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -4,9 +4,7 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Estate Property Tag"
-    _check_name = models.Constraint(
-        'UNIQUE(name)',
-        'The tag name should be unique!'
-    )
+    
+
 
     name = fields.Char(required=True)
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index b7ba2a6c118..6b70dfd2605 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -7,6 +7,8 @@
             <list string="Offers">
                 <field name="price" />
                 <field name="partner_id"/>
+                <field name="validity" string="Validity (days)"/>
+                <field name="date_deadline" string="Deadline"/>
                 <button type="object" name="action_accept_offer" title="accept" icon="fa-check"/>
                 <button type="object" name="action_refuse_offer" title="refuse" icon="fa-times"/>
                 <field name="status"/>

From 75b455808cd4806f7a69fd22268de0eee2da577f Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 14:04:33 +0100
Subject: [PATCH 23/34] [FIX] estate: inline comments spacing in
 estate_property_odder.py

---
 estate/models/estate_property_offer.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 48c5919cb4f..5f1a81890e2 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -45,10 +45,10 @@ def action_accept_offer(self):
     def action_refuse_offer(self):
         for record in self:
             if record.status == 'accepted':
-                val = record.property_id.expected_price # bypass 90% constraint L1
-                record.property_id.expected_price = 0.0 # bypass 90% constraint L2
+                val = record.property_id.expected_price  # bypass 90% constraint L1
+                record.property_id.expected_price = 0.0  # bypass 90% constraint L2
                 record.property_id.selling_price = 0.0
-                record.property_id.expected_price = val # bypass 90% constraint L3
+                record.property_id.expected_price = val  # bypass 90% constraint L3
                 record.property_id.buyer_id = None
             record.status = 'refused'
         return True

From d76827d9ad91a31798ca73a8444090861517c7c0 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 14:07:40 +0100
Subject: [PATCH 24/34] [FIX] estate: PEP8 correction estate_property_tag.py

---
 estate/models/estate_property_tag.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 20ab0fcffa4..165132d19de 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -4,7 +4,5 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Estate Property Tag"
-    
-
 
     name = fields.Char(required=True)

From 88cd366570a892787927219a2c45764e3be85560 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 14:33:26 +0100
Subject: [PATCH 25/34] [FIX] estate: bypass constraint modification

---
 estate/models/estate_property.py       | 2 +-
 estate/models/estate_property_offer.py | 3 ---
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 3454ef9111f..ac7c7098114 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -95,5 +95,5 @@ def __estate_property_action_sold_cancel(self, target, error_message, error_same
     @api.constrains('selling_price')
     def _check_price_constraint(self):
         for record in self:
-            if float_compare(record.selling_price, record.expected_price * 0.9, precision_digits=4) < 0:
+            if record.selling_price and float_compare(record.selling_price, record.expected_price * 0.9, precision_digits=4) < 0:
                 raise ValidationError("The price cannot be les than 90% of the expected price")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 5f1a81890e2..0e91cbeffe9 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -45,10 +45,7 @@ def action_accept_offer(self):
     def action_refuse_offer(self):
         for record in self:
             if record.status == 'accepted':
-                val = record.property_id.expected_price  # bypass 90% constraint L1
-                record.property_id.expected_price = 0.0  # bypass 90% constraint L2
                 record.property_id.selling_price = 0.0
-                record.property_id.expected_price = val  # bypass 90% constraint L3
                 record.property_id.buyer_id = None
             record.status = 'refused'
         return True

From 70eb50e8b43ea2595936a99b224d8ebb90aec949 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 14:43:11 +0100
Subject: [PATCH 26/34] [IMP] estate: add a ondelete decorator to reset the
 selling price

---
 estate/models/estate_property_offer.py | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 0e91cbeffe9..839aedd9cb2 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -49,3 +49,10 @@ def action_refuse_offer(self):
                 record.property_id.buyer_id = None
             record.status = 'refused'
         return True
+    
+    @api.ondelete(at_uninstall=False)
+    def _unlink_if_deleted(self):
+        print("#"*100)
+        for record in self:
+            print(record)
+            record.property_id.selling_price = 0.0

From bcbe15da0bdb4fa407e34acc5aa7441f15ae71ea Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 15:25:08 +0100
Subject: [PATCH 27/34] [IMP] estate: chapter 11 part 1

---
 estate/models/estate_property_offer.py      |  4 +---
 estate/models/estate_property_type.py       |  1 +
 estate/views/estate_property_type_views.xml | 13 ++++++++++++-
 3 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 839aedd9cb2..1ad109b5b9f 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -49,10 +49,8 @@ def action_refuse_offer(self):
                 record.property_id.buyer_id = None
             record.status = 'refused'
         return True
-    
+
     @api.ondelete(at_uninstall=False)
     def _unlink_if_deleted(self):
-        print("#"*100)
         for record in self:
-            print(record)
             record.property_id.selling_price = 0.0
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index d7e69e557ef..531aef24458 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -6,3 +6,4 @@ class EstatePropertyType(models.Model):
     _description = "Estate Property Types"
 
     name = fields.Char(required=True)
+    property_ids = fields.One2many("estate.property", "property_type_id")
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index d124ae8d555..d5649e62812 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -9,6 +9,17 @@
                     <group>
                         <h1><field name="name" nolabel="1" placeholder="New property type"/></h1>
                     </group>
+                    <notebook>
+                        <page string="Properties">
+                            <field name="property_ids">
+                                <list>
+                                    <field name="name"/>
+                                    <field name="expected_price"/>
+                                    <field name="state" string="Status"/>
+                                </list>
+                            </field>
+                        </page>
+                    </notebook>
                 </sheet>
             </form>
         </field>
@@ -17,6 +28,6 @@
     <record id="estate_property_type_action" model="ir.actions.act_window">
         <field name="name">Property Types</field>
         <field name="res_model">estate.property.type</field>
-        <field name="view_mode">list</field>
+        <field name="view_mode">list,form</field>
     </record>
 </odoo>

From 73aacdca67bfd02375415120c24f85c68619cd1b Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 16:01:49 +0100
Subject: [PATCH 28/34] [IMP] estate: chapter 11 part 2

---
 estate/models/estate_property.py       | 9 +++++++++
 estate/models/estate_property_offer.py | 3 +++
 estate/models/estate_property_tag.py   | 1 +
 estate/models/estate_property_type.py  | 1 +
 estate/views/estate_property_views.xml | 1 +
 5 files changed, 15 insertions(+)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index ac7c7098114..a3061e9d9be 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -6,6 +6,7 @@
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Estate Property"
+    _order = "id desc"
     _check_expected_price = models.Constraint(
         'CHECK(expected_price > 0)',
         'The expected price should be stricly positive'
@@ -97,3 +98,11 @@ def _check_price_constraint(self):
         for record in self:
             if record.selling_price and float_compare(record.selling_price, record.expected_price * 0.9, precision_digits=4) < 0:
                 raise ValidationError("The price cannot be les than 90% of the expected price")
+            
+    @api.onchange("offer_ids")
+    def _onchange_offer_ids(self):
+        for record in self:
+            if len(record.offer_ids) > 0:
+                record.state = "offer_received"
+            else:
+                record.state = "new"
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 1ad109b5b9f..e6da4a3fd70 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -4,6 +4,7 @@
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Estate Property Offer"
+    _order = "price desc"
     _check_offer_price = models.Constraint(
         'CHECK(price > 0)',
         'The offer price should be stricly positive'
@@ -40,6 +41,7 @@ def action_accept_offer(self):
             for offer in record.property_id.offer_ids:
                 offer.status = "refused"
             record.status = 'accepted'
+            record.property_id.state = 'offer_accepted'
         return True
 
     def action_refuse_offer(self):
@@ -47,6 +49,7 @@ def action_refuse_offer(self):
             if record.status == 'accepted':
                 record.property_id.selling_price = 0.0
                 record.property_id.buyer_id = None
+                record.property_id.state = 'offer_received'
             record.status = 'refused'
         return True
 
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 165132d19de..c78f9d22962 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -4,5 +4,6 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Estate Property Tag"
+    _order = "name"
 
     name = fields.Char(required=True)
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 531aef24458..2fb70e797dc 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -4,6 +4,7 @@
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Estate Property Types"
+    _order = "name"
 
     name = fields.Char(required=True)
     property_ids = fields.One2many("estate.property", "property_type_id")
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 15187148a75..a17b6b5602b 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -24,6 +24,7 @@
                 <header>
                     <button name="estate_property_action_sold" type="object" string="Sold"/>
                     <button name="estate_property_action_cancel" type="object" string="Cancel"/>
+                    <field name="state" widget="statusbar" statusbar_visible="new,offer_received,offer_accepted,sold"/>
                 </header>
                 <sheet>
                     <group>

From 5f60173cc14b4fd1a72b82151574a46c90ff382a Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 16:24:34 +0100
Subject: [PATCH 29/34] [FIX] estate: whitespace in blank line in
 estate_property.py

---
 estate/models/estate_property.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index a3061e9d9be..673cc3c764d 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -98,7 +98,7 @@ def _check_price_constraint(self):
         for record in self:
             if record.selling_price and float_compare(record.selling_price, record.expected_price * 0.9, precision_digits=4) < 0:
                 raise ValidationError("The price cannot be les than 90% of the expected price")
-            
+
     @api.onchange("offer_ids")
     def _onchange_offer_ids(self):
         for record in self:

From cd5b754b92ccb13481cd49d34109c2a1c6e93b80 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Thu, 20 Nov 2025 17:05:10 +0100
Subject: [PATCH 30/34] [IMP] estate: chapter 11 color picker part

---
 estate/models/estate_property_tag.py        |  1 +
 estate/models/estate_property_type.py       |  3 ++-
 estate/views/estate_property_type_views.xml | 11 +++++++++++
 estate/views/estate_property_views.xml      |  4 ++--
 4 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index c78f9d22962..59cbe8b8f99 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -7,3 +7,4 @@ class EstatePropertyTag(models.Model):
     _order = "name"
 
     name = fields.Char(required=True)
+    color = fields.Integer()
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 2fb70e797dc..424b38a4594 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -4,7 +4,8 @@
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Estate Property Types"
-    _order = "name"
+    _order = "sequence, name"
 
     name = fields.Char(required=True)
     property_ids = fields.One2many("estate.property", "property_type_id")
+    sequence = fields.Integer('Sequence', default=1, help="Used to order estate property types.")
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index d5649e62812..c37f255b85e 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -25,6 +25,17 @@
         </field>
     </record>
 
+    <record id="estate_property_type_list_view" model="ir.ui.view">
+        <field name="name">estate.property.type.list</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <list>
+                <field name="sequence" widget="handle"/>
+                <field name="name" string="Type"/>
+            </list>
+        </field>
+    </record>
+
     <record id="estate_property_type_action" model="ir.actions.act_window">
         <field name="name">Property Types</field>
         <field name="res_model">estate.property.type</field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index a17b6b5602b..ccd711d32e4 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -31,11 +31,11 @@
                         <h1><field name="name" nolabel="1" placeholder="My new house"/></h1>
                     </group>
                     <group>
-                        <field name="tag_ids" nolabel="1" widget="many2many_tags"/>
+                        <field name="tag_ids" nolabel="1" widget="many2many_tags" options="{'color_field': 'color'}"/>
                         <br/>
                         <br/>
                         <group>
-                            <field name="property_type_id" string="Property Type"/>
+                            <field name="property_type_id" string="Property Type" options="{'no_create': true, 'no_open': true}"/>
                             <field name="postcode"/>
                             <field name="date_availability" string="Available From"/>
                         </group>

From c8cb661bdbeb3d02fb9068641e774a8c31978150 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Fri, 21 Nov 2025 10:36:49 +0100
Subject: [PATCH 31/34] [IMP] estate: chapter 11 part 3

---
 estate/models/estate_property_offer.py       | 10 +++++-----
 estate/views/estate_property_offer_views.xml |  9 ++++-----
 estate/views/estate_property_views.xml       | 18 +++++++++++-------
 3 files changed, 20 insertions(+), 17 deletions(-)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index e6da4a3fd70..25db005f6b8 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -36,6 +36,8 @@ def _inverse_date_deadline(self):
 
     def action_accept_offer(self):
         for record in self:
+            if record.status == 'accepted':
+                return False
             record.property_id.selling_price = record.price
             record.property_id.buyer_id = record.partner_id
             for offer in record.property_id.offer_ids:
@@ -46,12 +48,10 @@ def action_accept_offer(self):
 
     def action_refuse_offer(self):
         for record in self:
-            if record.status == 'accepted':
-                record.property_id.selling_price = 0.0
-                record.property_id.buyer_id = None
-                record.property_id.state = 'offer_received'
-            record.status = 'refused'
+            record.status = "refused"
         return True
+    
+
 
     @api.ondelete(at_uninstall=False)
     def _unlink_if_deleted(self):
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index 6b70dfd2605..7b024d26d9d 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -4,14 +4,13 @@
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
-            <list string="Offers">
-                <field name="price" />
+            <list string="Offers" editable="bottom" decoration-success="status == 'accepted'" decoration-danger="status == 'refused'">
+                <field name="price"/>
                 <field name="partner_id"/>
                 <field name="validity" string="Validity (days)"/>
                 <field name="date_deadline" string="Deadline"/>
-                <button type="object" name="action_accept_offer" title="accept" icon="fa-check"/>
-                <button type="object" name="action_refuse_offer" title="refuse" icon="fa-times"/>
-                <field name="status"/>
+                <button type="object" name="action_accept_offer" title="accept" icon="fa-check" invisible="status in ('accepted', 'refused')"/>
+                <button type="object" name="action_refuse_offer" title="refuse" icon="fa-times" invisible="status in ('accepted', 'refused')"/>
             </list>
         </field>
     </record>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index ccd711d32e4..c1b43ec18ec 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -4,14 +4,18 @@
         <field name="name">estate.property.list</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
-            <list string="Properties">
+            <list string="Properties"
+                decoration-success="state in ('offer_received', 'offer_accepted')" 
+                decoration-bf="state == 'offer_accepted'"
+                decoration-muted="state in ('sold', 'cancelled')"
+            >
                 <field name="name" string="Title"/>
                 <field name="postcode"/>
                 <field name="bedrooms"/>
                 <field name="living_area" string="Living Area (sqm)"/>
                 <field name="expected_price"/>
                 <field name="selling_price"/>
-                <field name="date_availability" string="Available From"/>
+                <field name="date_availability" string="Available From" optional="True"/>
             </list>
         </field>
     </record>
@@ -22,8 +26,8 @@
         <field name="arch" type="xml">
             <form string="My new house">
                 <header>
-                    <button name="estate_property_action_sold" type="object" string="Sold"/>
-                    <button name="estate_property_action_cancel" type="object" string="Cancel"/>
+                    <button name="estate_property_action_sold" type="object" string="Sold" invisible="state == 'sold' or state == 'cancelled'"/>
+                    <button name="estate_property_action_cancel" type="object" string="Cancel" invisible="state == 'sold' or state == 'cancelled'"/>
                     <field name="state" widget="statusbar" statusbar_visible="new,offer_received,offer_accepted,sold"/>
                 </header>
                 <sheet>
@@ -54,13 +58,13 @@
                                 <field name="facades"/>
                                 <field name="garage"/>
                                 <field name="garden"/>
-                                <field name="garden_area" string="Garden Area (sqm)"/>
-                                <field name="garden_orientation"/>
+                                <field name="garden_area" string="Garden Area (sqm)" invisible="not garden"/>
+                                <field name="garden_orientation" invisible="not garden"/>
                                 <field name="total_area"/>
                             </group>
                         </page>
                         <page string="Offers">
-                            <field name="offer_ids"/>
+                            <field name="offer_ids" readonly="state in ('offer_accepted', 'sold', 'cancelled')"/>
                         </page>
                         <page string="Other Info">
                             <group>

From 6f106c1a4e7be7f94b213c2ed073edebb661352c Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Fri, 21 Nov 2025 10:37:26 +0100
Subject: [PATCH 32/34] [FIX] estate: PEP8 fix in estate_property_offer.py

---
 estate/models/estate_property_offer.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 25db005f6b8..7c56adf61f4 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -50,8 +50,6 @@ def action_refuse_offer(self):
         for record in self:
             record.status = "refused"
         return True
-    
-
 
     @api.ondelete(at_uninstall=False)
     def _unlink_if_deleted(self):

From 3607166f750cee32ee91a8594cb8eb808af73a0e Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Fri, 21 Nov 2025 12:04:57 +0100
Subject: [PATCH 33/34] [IMP] estate: chapter 11 end

---
 estate/__manifest__.py                       |  2 +-
 estate/models/estate_property_offer.py       |  1 +
 estate/models/estate_property_type.py        |  9 ++++++++-
 estate/views/estate_property_offer_views.xml |  7 +++++++
 estate/views/estate_property_type_views.xml  | 13 +++++++++++--
 estate/views/estate_property_views.xml       |  9 +++++----
 6 files changed, 33 insertions(+), 8 deletions(-)

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 93b455da68a..2bc44e1e3ff 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_property_offer_views.xml',
         'views/estate_property_type_views.xml',
         'views/estate_property_tag_views.xml',
-        'views/estate_property_offer_views.xml',
         'views/estate_menus.xml'
     ]
 }
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 7c56adf61f4..2f0a1285ed0 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -21,6 +21,7 @@ class EstatePropertyOffer(models.Model):
     )
     partner_id = fields.Many2one("res.partner", required=True)
     property_id = fields.Many2one("estate.property", required=True)
+    property_type_id = fields.Many2one(related="property_id.property_type_id", stored=True)
     validity = fields.Integer(default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
 
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 424b38a4594..977fba2d1be 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class EstatePropertyType(models.Model):
@@ -9,3 +9,10 @@ class EstatePropertyType(models.Model):
     name = fields.Char(required=True)
     property_ids = fields.One2many("estate.property", "property_type_id")
     sequence = fields.Integer('Sequence', default=1, help="Used to order estate property types.")
+    offer_ids = fields.One2many("estate.property.offer", "property_type_id")
+    offer_count = fields.Integer(compute="_compute_offer_count")
+
+    @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_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index 7b024d26d9d..cef3c4baa73 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -14,4 +14,11 @@
             </list>
         </field>
     </record>
+
+    <record id="estate_property_offer_action" model="ir.actions.act_window">
+        <field name="name">Offer</field>
+        <field name="res_model">estate.property.offer</field>
+        <field name="view_mode">list</field>
+        <field name="domain">[('property_type_id', '=', active_id)]</field>
+    </record>
 </odoo>
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index c37f255b85e..96ee410abd8 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -6,8 +6,17 @@
         <field name="arch" type="xml">
             <form string="New Type">
                 <sheet>
-                    <group>
-                        <h1><field name="name" nolabel="1" placeholder="New property type"/></h1>
+                    <group class="border-bottom">
+                        <group>
+                            <h1><field name="name" nolabel="1" placeholder="New property type"/></h1>
+                        </group>
+                        <group>
+                            <div class="d-flex flex-row-reverse">
+                                <button class="oe_stat_button border" name="%(estate.estate_property_offer_action)d" type="action" string="Stat button" icon="fa-money">
+                                    <field name="offer_count"/>
+                                </button>
+                            </div>
+                        </group>
                     </group>
                     <notebook>
                         <page string="Properties">
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index c1b43ec18ec..6036828434b 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -87,13 +87,13 @@
                 <field name="postcode"/>
                 <field name="expected_price"/>
                 <field name="bedrooms"/>
-                <field name="living_area" string="Living Area (sqm)"/>
+                <field name="living_area" string="Living Area (sqm)" filter_domain="[('living_area', '>=', self)]"/>
                 <field name="facades"/>
                 <field name="property_type_id" string="Type"/>
                 <separator/>
-                <filter name="Available" domain="[('state', 'in', ('new', 'offer_received'))]"/>
-                <filter name="Postcode" context="{'group_by': 'postcode'}"/>
-                <filter name="Type" context="{'group_by': 'property_type_id'}"/>
+                <filter name="available" string="Available" domain="[('state', 'in', ('new', 'offer_received'))]"/>
+                <filter name="postcode" string="Postcode" context="{'group_by': 'postcode'}"/>
+                <filter name="type" string="Type" context="{'group_by': 'property_type_id'}"/>
             </search>
         </field>
     </record>
@@ -102,5 +102,6 @@
         <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':1}</field>
     </record>
 </odoo>

From 09fdf455e4bb079117d31ca3174381237131bbf3 Mon Sep 17 00:00:00 2001
From: haall-odoo <haall@odoo.com>
Date: Fri, 21 Nov 2025 12:14:29 +0100
Subject: [PATCH 34/34] [FIX] estate: estate.property.offer stored -> store

---
 estate/models/estate_property_offer.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 2f0a1285ed0..c5a65b2fdfe 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -21,7 +21,7 @@ class EstatePropertyOffer(models.Model):
     )
     partner_id = fields.Many2one("res.partner", required=True)
     property_id = fields.Many2one("estate.property", required=True)
-    property_type_id = fields.Many2one(related="property_id.property_type_id", stored=True)
+    property_type_id = fields.Many2one(related="property_id.property_type_id", store=True)
     validity = fields.Integer(default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")