From e80c6c5d76cccc1597a3df8ff2c06463dcafb762 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Mon, 20 Oct 2025 13:48:50 +0200
Subject: [PATCH 01/12] [ADD] Estate: create the app

---
 estate/__init__.py     |  0
 estate/__manifest__.py | 18 ++++++++++++++++++
 2 files changed, 18 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..980588d9cb8
--- /dev/null
+++ b/estate/__manifest__.py
@@ -0,0 +1,18 @@
+{
+    'name': "estate",
+    'version': '1.0',
+    'depends': ['base'],
+    'author': "Victor Decleire",
+    'application': True,
+    # 'description': """
+    # Description text
+    # """,
+    # # data files always loaded at installation
+    # 'data': [
+    #     'views/mymodule_view.xml',
+    # ],
+    # # data files containing optionally loaded demonstration data
+    # 'demo': [
+    #     'demo/demo_data.xml',
+    # ],
+}
\ No newline at end of file

From fd2f7fe5fb315afa60552e16ad381d1658aacd13 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Mon, 20 Oct 2025 15:25:01 +0200
Subject: [PATCH 02/12] [IMP] Estate: Creating table fields

---
 estate/__init__.py               |  1 +
 estate/models/__init__.py        |  1 +
 estate/models/estate_property.py | 21 +++++++++++++++++++++
 3 files changed, 23 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..aaaa9c77c51
--- /dev/null
+++ b/estate/models/estate_property.py
@@ -0,0 +1,21 @@
+from odoo import fields, models
+
+class EstateProperty(models.Model):
+    _name = "estate_property"
+    _description = "Real estate propreties"
+
+    name = fields.Char(required=True)
+    description = fields.Text()
+    postcode = fields.Char()
+    date_availability = fields.Date()
+    expected_price = fields.Float()
+    selling_price = fields.Float(required=True)
+    bedrooms = fields.Float()
+    living_area = fields.Float()
+    facades = fields.Float()
+    garage = fields.Boolean()
+    garden = fields.Boolean()
+    garden_area = fields.Float()
+    garden_orientation = fields.Selection(
+        selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')]
+    )
\ No newline at end of file

From 7f5dc943466a06daa922bd68ac6c71e50ea8f974 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Mon, 20 Oct 2025 16:07:57 +0200
Subject: [PATCH 03/12] [IMP] Estate: Grant access group user

[FIX] Add licence and correct style
---
 estate/__init__.py                     |  2 +-
 estate/__manifest__.py                 | 12 ++++++++----
 estate/models/__init__.py              |  2 +-
 estate/models/estate_property.py       | 16 +++++++++-------
 estate/security/ir.model.access.csv    |  2 ++
 estate/views/estate_menus.xml          |  8 ++++++++
 estate/views/estate_property_views.xml |  8 ++++++++
 7 files changed, 37 insertions(+), 13 deletions(-)
 create mode 100644 estate/security/ir.model.access.csv
 create mode 100644 estate/views/estate_menus.xml
 create mode 100644 estate/views/estate_property_views.xml

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 980588d9cb8..04689bbfe2d 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -4,15 +4,19 @@
     'depends': ['base'],
     'author': "Victor Decleire",
     'application': True,
+    'license': "LGPL-3",
     # 'description': """
     # Description text
     # """,
     # # data files always loaded at installation
-    # 'data': [
-    #     'views/mymodule_view.xml',
-    # ],
+    'data': [
+        'security/ir.model.access.csv',
+
+        'views/estate_property_views.xml',
+        'views/estate_menus.xml',
+    ],
     # # data files containing optionally loaded demonstration data
     # 'demo': [
     #     'demo/demo_data.xml',
     # ],
-}
\ 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 aaaa9c77c51..6e6faa0e82f 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,21 +1,23 @@
 from odoo import fields, models
+from odoo.tools.date_utils import add
+
 
 class EstateProperty(models.Model):
-    _name = "estate_property"
+    _name = "estate.property"
     _description = "Real estate propreties"
 
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()
-    date_availability = fields.Date()
+    date_availability = fields.Date(copy=False, default=lambda x: add(fields.Date.today(), months=3))
     expected_price = fields.Float()
-    selling_price = fields.Float(required=True)
-    bedrooms = fields.Float()
-    living_area = fields.Float()
-    facades = fields.Float()
+    selling_price = fields.Float(required=True, readonly=True, copy=False)
+    bedrooms = fields.Integer(default=2)
+    living_area = fields.Integer()
+    facades = fields.Integer()
     garage = fields.Boolean()
     garden = fields.Boolean()
     garden_area = fields.Float()
     garden_orientation = fields.Selection(
         selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')]
-    )
\ No newline at end of file
+    )
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
new file mode 100644
index 00000000000..e0b8c48c63e
--- /dev/null
+++ b/estate/security/ir.model.access.csv
@@ -0,0 +1,2 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+access_estate_group_user,estate.group.user,model_estate_property,base.group_user,1,1,1,1
\ No newline at end of file
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
new file mode 100644
index 00000000000..53e18090998
--- /dev/null
+++ b/estate/views/estate_menus.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <menuitem id="estate_menu_root" name="Real Estate">
+        <menuitem id="estate_first_level_menu" name="Homes">
+            <menuitem id="estate_model_menu_action" action="estate_property_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..54a607e7e6a
--- /dev/null
+++ b/estate/views/estate_property_views.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <record id="estate_property_model_action" model="ir.actions.act_window">
+        <field name="name">Test action</field>
+        <field name="res_model">estate.property</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>
\ No newline at end of file

From b9fd3093bf6958887ea9e0a79f8e4e84086a93f9 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Tue, 21 Oct 2025 13:26:01 +0200
Subject: [PATCH 04/12] [IMP] Estate: Create views

---
 estate/models/estate_property.py       | 11 +++-
 estate/security/ir.model.access.csv    |  2 +-
 estate/views/estate_menus.xml          |  4 +-
 estate/views/estate_property_views.xml | 73 +++++++++++++++++++++++++-
 4 files changed, 84 insertions(+), 6 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 6e6faa0e82f..212ccc2b21d 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -10,8 +10,8 @@ class EstateProperty(models.Model):
     description = fields.Text()
     postcode = fields.Char()
     date_availability = fields.Date(copy=False, default=lambda x: add(fields.Date.today(), months=3))
-    expected_price = fields.Float()
-    selling_price = fields.Float(required=True, readonly=True, copy=False)
+    expected_price = fields.Float(required=True)
+    selling_price = fields.Float(readonly=True, copy=False)
     bedrooms = fields.Integer(default=2)
     living_area = fields.Integer()
     facades = fields.Integer()
@@ -21,3 +21,10 @@ class EstateProperty(models.Model):
     garden_orientation = fields.Selection(
         selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')]
     )
+    active = fields.Boolean(default=True)
+    state = fields.Selection(
+        selection=[('new', 'New'), ('received', 'Offer Received'), ('accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')],
+        default='new',
+        copy=False,
+        required=True
+    )
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index e0b8c48c63e..bc409891f2e 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -1,2 +1,2 @@
 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-access_estate_group_user,estate.group.user,model_estate_property,base.group_user,1,1,1,1
\ No newline at end of file
+access_estate_group_user,estate.group.user,model_estate_property,base.group_user,1,1,1,1
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index 53e18090998..5a0e2383ca2 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -1,8 +1,8 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
     <menuitem id="estate_menu_root" name="Real Estate">
-        <menuitem id="estate_first_level_menu" name="Homes">
+        <menuitem id="estate_first_level_menu" name="Advertisements">
             <menuitem id="estate_model_menu_action" action="estate_property_model_action"/>
         </menuitem>
     </menuitem>
-</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 54a607e7e6a..1e5e386b443 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -5,4 +5,75 @@
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
     </record>
-</odoo>
\ No newline at end of file
+
+    <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">
+                <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_property_view_form" model="ir.ui.view">
+        <field name="name">estate.property.form</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <form string="Propreties">
+                <sheet>
+                    <h1>
+                        <field name="name"/>
+                    </h1>
+                    <group>
+                        <group>
+                            <field name="postcode"/>
+                            <field name="date_availability"/>
+                        </group>
+                        <group>
+                            <field name="expected_price"/>
+                            <field name="selling_price"/>
+                        </group>
+                    </group>
+                    <notebook>
+                        <page string="Description">
+                            <group>
+                                <field name="description"/>
+                                <field name="bedrooms"/>
+                                <field name="living_area" string="Living Area (sqm)"/>
+                                <field name="facades"/>
+                                <field name="garage"/>
+                                <field name="garden"/>
+                                <field name="garden_area" string="Garden Area (sqm)"/>
+                                <field name="garden_orientation"/>
+                            </group>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+    <record id="estate_property_search" model="ir.ui.view">
+        <field name="name">estate.property.search</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <search string="Search">
+                <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"/>
+                <filter string="Available" name="available" domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]"/>
+                <filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>
+            </search>
+        </field>
+    </record>
+</odoo>

From 04c68d6200547a925900d8682af8a42a4c3168ba Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Tue, 21 Oct 2025 15:49:53 +0200
Subject: [PATCH 05/12] [IMP] Estate: Create tag, offer, type with according
 views

[FIX] Changing inverse one2many to property_id

[CORRECTION]

[CORRECTION COMMIT]

add line
---
 estate/__manifest__.py                       |  3 ++
 estate/models/__init__.py                    |  3 ++
 estate/models/estate_property.py             | 30 ++++++++++++++------
 estate/models/estate_property_offer.py       | 11 +++++++
 estate/models/estate_property_tag.py         |  8 ++++++
 estate/models/estate_property_type.py        |  8 ++++++
 estate/security/ir.model.access.csv          |  3 ++
 estate/views/estate_menus.xml                |  4 +++
 estate/views/estate_property_offer_views.xml | 14 +++++++++
 estate/views/estate_property_tag_views.xml   | 22 ++++++++++++++
 estate/views/estate_property_type_views.xml  | 22 ++++++++++++++
 estate/views/estate_property_views.xml       | 12 +++++++-
 12 files changed, 131 insertions(+), 9 deletions(-)
 create mode 100644 estate/models/estate_property_offer.py
 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_offer_views.xml
 create mode 100644 estate/views/estate_property_tag_views.xml
 create mode 100644 estate/views/estate_property_type_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 04689bbfe2d..9790db8b2cb 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -13,6 +13,9 @@
         'security/ir.model.access.csv',
 
         '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',
     ],
     # # data files containing optionally loaded demonstration data
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 5e1963c9d2f..2f1821a39c1 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1 +1,4 @@
 from . import estate_property
+from . import estate_property_type
+from . import estate_property_tag
+from . import estate_property_offer
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 212ccc2b21d..74744f0e4c5 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 fields, models, api
 from odoo.tools.date_utils import add
 
 
@@ -18,13 +18,27 @@ class EstateProperty(models.Model):
     garage = fields.Boolean()
     garden = fields.Boolean()
     garden_area = fields.Float()
-    garden_orientation = fields.Selection(
-        selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')]
-    )
+    garden_orientation = fields.Selection(selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')])
     active = fields.Boolean(default=True)
     state = fields.Selection(
         selection=[('new', 'New'), ('received', 'Offer Received'), ('accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')],
-        default='new',
-        copy=False,
-        required=True
-    )
+        default='new', copy=False, required=True)
+    seller_id = fields.Many2one('res.users', default=lambda self: self.env.user)
+    buyer_id = fields.Many2one('res.partner')
+    tag_ids = fields.Many2many('estate.property.tag', string="Tags")
+    offer_ids = fields.One2many('estate.property.offer', 'property_id')
+    total_area = fields.Float(compute="_compute_total_area", readonly=True, copy=False)
+    best_price = fields.Float(compute="_get_best_price", readonly=True, copy=False)
+
+    @api.depends('living_area', 'garden_area')
+    def _compute_total_area(self):
+        for record in self:
+            record.total_area = record.living_area + record.garden_area
+
+    @api.depends('offer_ids')
+    def _get_best_price(self):
+        for record in self:
+            if record.offer_ids:
+                record.best_price = max(record.offer_ids.mapped('price'))
+            else:
+                record.best_price = 0
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
new file mode 100644
index 00000000000..fae626171ca
--- /dev/null
+++ b/estate/models/estate_property_offer.py
@@ -0,0 +1,11 @@
+from odoo import fields, models
+
+
+class EstatePropertyOffer(models.Model):
+    _name = "estate.property.offer"
+    _description = "Offers for propreties"
+
+    price = fields.Float()
+    status = fields.Selection(selection=[('accepted', 'Accepted'), ('refused', 'Refused')], copy=False)
+    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
new file mode 100644
index 00000000000..d4ada2546c4
--- /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 = "Define the tags of the property"
+
+    name = fields.Char(required=True)
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
new file mode 100644
index 00000000000..18503b6b16f
--- /dev/null
+++ b/estate/models/estate_property_type.py
@@ -0,0 +1,8 @@
+from odoo import fields, models
+
+
+class EstatePropertyType(models.Model):
+    _name = "estate.property.type"
+    _description = "Define the type of the property"
+
+    name = fields.Char(required=True)
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index bc409891f2e..9eb29855a32 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -1,2 +1,5 @@
 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
 access_estate_group_user,estate.group.user,model_estate_property,base.group_user,1,1,1,1
+access_estate_type_group_user,estate.type.group.user,model_estate_property_type,base.group_user,1,1,1,1
+access_estate_tag_group_user,estate.tag.group.user,model_estate_property_tag,base.group_user,1,1,1,1
+access_estate_offer_group_user,estate.offer.group.user,model_estate_property_offer,base.group_user,1,1,1,1
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index 5a0e2383ca2..a4c9cd53fff 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -4,5 +4,9 @@
         <menuitem id="estate_first_level_menu" name="Advertisements">
             <menuitem id="estate_model_menu_action" action="estate_property_model_action"/>
         </menuitem>
+        <menuitem id="estate_sec_level_menu" name="Settings">
+            <menuitem id="estate_type_model_menu_action" action="estate_property_type_model_action"/>
+            <menuitem id="estate_tag_model_menu_action" action="estate_property_tag_model_action"/>
+        </menuitem>
     </menuitem>
 </odoo>
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
new file mode 100644
index 00000000000..0ae660f7a88
--- /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_view_list" 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="Channel">
+                <field name="price"/>
+                <field name="partner_id" string="Partner"/>
+                <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
new file mode 100644
index 00000000000..27f8fbaf7a4
--- /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_model_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,form</field>
+    </record>
+
+    <record id="estate_property_type_view_form" 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="Propreties Tags">
+                <sheet>
+                    <group>
+                        <field name="name"/>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+</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..9f839b6c58c
--- /dev/null
+++ b/estate/views/estate_property_type_views.xml
@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <record id="estate_property_type_model_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,form</field>
+    </record>
+
+    <record id="estate_property_type_view_form" model="ir.ui.view">
+        <field name="name">estate.property.type.form</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <form string="Propreties Types">
+                <sheet>
+                    <h1>
+                        <field name="name"/>
+                    </h1>
+                </sheet>
+            </form>
+        </field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 1e5e386b443..8d643b7bca8 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
     <record id="estate_property_model_action" model="ir.actions.act_window">
-        <field name="name">Test action</field>
+        <field name="name">Properties</field>
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
     </record>
@@ -52,6 +52,16 @@
                                 <field name="garden"/>
                                 <field name="garden_area" string="Garden Area (sqm)"/>
                                 <field name="garden_orientation"/>
+                                <field name="tag_ids" widget="many2many_tags"/>
+                            </group>
+                        </page>
+                        <page string="Offers">
+                            <field name="offer_ids"/>
+                        </page>
+                        <page string="More Info">
+                            <group>
+                                <field name="seller_id" string="Salesman"/>
+                                <field name="buyer_id" string="Buyer"/>
                             </group>
                         </page>
                     </notebook>

From b69ce9708df115ab45479804d0eb0410c62fa445 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Wed, 22 Oct 2025 09:02:18 +0200
Subject: [PATCH 06/12] [IMP] Estate: Adding computed fields

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

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 74744f0e4c5..949efed0ee0 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -42,3 +42,12 @@ def _get_best_price(self):
                 record.best_price = max(record.offer_ids.mapped('price'))
             else:
                 record.best_price = 0
+
+    @api.onchange('garden')
+    def _onchange_garden(self):
+        if self.garden:
+            self.garden_area = 10
+            self.garden_orientation = 'north'
+        else:
+            self.garden_area = 0
+            self.garden_orientation = None
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index fae626171ca..117e874f516 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,11 +1,24 @@
-from odoo import fields, models
+from odoo import fields, models, api
+from odoo.tools.date_utils import add
 
 
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Offers for propreties"
 
-    price = fields.Float()
+    price = fields.Float(required=True)
     status = fields.Selection(selection=[('accepted', 'Accepted'), ('refused', 'Refused')], copy=False)
     partner_id = fields.Many2one('res.partner', required=True)
     property_id = fields.Many2one('estate.property', required=True)
+    create_date = fields.Date(default=fields.Date.today(), readonly=True)
+    validity = fields.Integer(default=7)
+    date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
+
+    @api.depends('validity', 'create_date')
+    def _compute_date_deadline(self):
+        for record in self:
+            record.date_deadline = add(record.create_date, days=record.validity)
+
+    def _inverse_date_deadline(self):
+        for record in self:
+            record.validity = (record.date_deadline - record.create_date).days
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index 0ae660f7a88..bcdcb231d89 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -11,4 +11,23 @@
             </list>
         </field>
     </record>
+
+    <record id="estate_property_offer_view_form" model="ir.ui.view">
+        <field name="name">estate.property.offer.form</field>
+        <field name="model">estate.property.offer</field>
+        <field name="arch" type="xml">
+            <form string="Channel">
+                <group>
+                    <field name="price"/>
+                    <field name="partner_id" string="Partner"/>
+                    <field name="status"/>
+                </group>
+                <group>
+                    <field name="create_date"/>
+                    <field name="validity"/>
+                    <field name="date_deadline"/>
+                </group>
+            </form>
+        </field>
+    </record>
 </odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 8d643b7bca8..becc30c3030 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -38,6 +38,7 @@
                         </group>
                         <group>
                             <field name="expected_price"/>
+                            <field name="best_price" string="Best Offer"/>
                             <field name="selling_price"/>
                         </group>
                     </group>
@@ -52,6 +53,7 @@
                                 <field name="garden"/>
                                 <field name="garden_area" string="Garden Area (sqm)"/>
                                 <field name="garden_orientation"/>
+                                <field name="total_area" string="Total Area (sqm)"/>
                                 <field name="tag_ids" widget="many2many_tags"/>
                             </group>
                         </page>

From 3ab222443fbc5e576caec9cf42301b5664a9c762 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Wed, 22 Oct 2025 13:01:26 +0200
Subject: [PATCH 07/12] [IMP] Estate: Creating buttons

[FIX] delete whitespaces
---
 estate/models/estate_property.py             | 20 +++++++++++++++++++-
 estate/models/estate_property_offer.py       | 13 +++++++++++++
 estate/views/estate_property_offer_views.xml |  4 ++++
 estate/views/estate_property_views.xml       |  4 ++++
 4 files changed, 40 insertions(+), 1 deletion(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 949efed0ee0..3595ef14459 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,4 @@
-from odoo import fields, models, api
+from odoo import fields, models, api, exceptions
 from odoo.tools.date_utils import add
 
 
@@ -29,6 +29,7 @@ class EstateProperty(models.Model):
     offer_ids = fields.One2many('estate.property.offer', 'property_id')
     total_area = fields.Float(compute="_compute_total_area", readonly=True, copy=False)
     best_price = fields.Float(compute="_get_best_price", readonly=True, copy=False)
+    is_available = fields.Boolean(compute='_compute_is_available')
 
     @api.depends('living_area', 'garden_area')
     def _compute_total_area(self):
@@ -51,3 +52,20 @@ def _onchange_garden(self):
         else:
             self.garden_area = 0
             self.garden_orientation = None
+
+    def action_set_sold(self):
+        if self.state == 'cancelled':
+            raise exceptions.UserError("Cancelled property cannot be set as sold")
+        self.state = 'sold'
+        return 1
+
+    def action_set_cancelled(self):
+        if self.state == 'sold':
+            raise exceptions.UserError("Sold property cannot be cancelled")
+        self.state = 'cancelled'
+        return 1
+
+    @api.depends('offer_ids.status')
+    def _compute_is_available(self):
+        for record in self:
+            record.is_available = not any(s == 'accepted' for s in record.offer_ids.mapped('status'))
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 117e874f516..e7d7e887760 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -13,6 +13,7 @@ class EstatePropertyOffer(models.Model):
     create_date = fields.Date(default=fields.Date.today(), readonly=True)
     validity = fields.Integer(default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
+    is_available = fields.Boolean(related="property_id.is_available")
 
     @api.depends('validity', 'create_date')
     def _compute_date_deadline(self):
@@ -22,3 +23,15 @@ def _compute_date_deadline(self):
     def _inverse_date_deadline(self):
         for record in self:
             record.validity = (record.date_deadline - record.create_date).days
+
+    def action_accept_offer(self):
+        for record in self:
+            record.status = 'accepted'
+            record.property_id.buyer_id = record.partner_id
+            record.property_id.selling_price = record.price
+        return 1
+
+    def action_refuse_offer(self):
+        for record in self:
+            record.status = 'refused'
+        return 1
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index bcdcb231d89..2ef2ec07c3c 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -7,6 +7,10 @@
             <list string="Channel">
                 <field name="price"/>
                 <field name="partner_id" string="Partner"/>
+                <field name="validity" string="Validity (days)"/>
+                <field name="date_deadline" string="Deadline"/>
+                <button name="action_accept_offer" type="object" icon="fa-check" string="Accept" invisible="not is_available"/>
+                <button name="action_refuse_offer" type="object" icon="fa-times" string="Refuse"/>
                 <field name="status"/>
             </list>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index becc30c3030..0e5a282fa51 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -27,6 +27,10 @@
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form string="Propreties">
+                <header>
+                    <button name="action_set_sold" string="Sold" type="object"/>
+                    <button name="action_set_cancelled" string="Cancel" type="object"/>
+                </header>
                 <sheet>
                     <h1>
                         <field name="name"/>

From 0087bbb62871b2865fae6fd446f963a7addeb781 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Wed, 22 Oct 2025 14:14:56 +0200
Subject: [PATCH 08/12] [IMP] Estate: Adding constraints on fields

[FIX] Mofify usererror exception
---
 estate/models/estate_property.py       | 25 ++++++++++++++++++++++---
 estate/models/estate_property_offer.py |  4 ++++
 estate/models/estate_property_tag.py   |  4 ++++
 estate/models/estate_property_type.py  |  4 ++++
 4 files changed, 34 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 3595ef14459..1fd59fdaa71 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,11 +1,21 @@
-from odoo import fields, models, api, exceptions
+from odoo import fields, models, api
+from odoo.exceptions import UserError, ValidationError
 from odoo.tools.date_utils import add
+from odoo.tools.float_utils import float_compare
 
 
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Real estate propreties"
 
+    _check_expected_price = models.Constraint(
+        'CHECK(expected_price > 0)',
+        'Expected price must be strictly positive.',
+    )
+    _check_selling_price = models.Constraint(
+        'CHECK(selling_price >= 0)',
+        'Selling price must be positive.',
+    )
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()
@@ -46,6 +56,7 @@ def _get_best_price(self):
 
     @api.onchange('garden')
     def _onchange_garden(self):
+        self.ensure_one()
         if self.garden:
             self.garden_area = 10
             self.garden_orientation = 'north'
@@ -54,14 +65,16 @@ def _onchange_garden(self):
             self.garden_orientation = None
 
     def action_set_sold(self):
+        self.ensure_one()
         if self.state == 'cancelled':
-            raise exceptions.UserError("Cancelled property cannot be set as sold")
+            raise UserError("Cancelled property cannot be set as sold")
         self.state = 'sold'
         return 1
 
     def action_set_cancelled(self):
+        self.ensure_one()
         if self.state == 'sold':
-            raise exceptions.UserError("Sold property cannot be cancelled")
+            raise UserError("Sold property cannot be cancelled")
         self.state = 'cancelled'
         return 1
 
@@ -69,3 +82,9 @@ def action_set_cancelled(self):
     def _compute_is_available(self):
         for record in self:
             record.is_available = not any(s == 'accepted' for s in record.offer_ids.mapped('status'))
+
+    @api.constrains('selling_price', 'expected_price')
+    def _check_selling_price(self):
+        self.ensure_one()
+        if len(self.offer_ids) > 0 and float_compare(self.expected_price, 0.9 * self.selling_price, 0) > 0:
+            raise ValidationError("The selling price cannot be lower than 90% of the expected price.")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index e7d7e887760..00b68d56055 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -6,6 +6,10 @@ class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Offers for propreties"
 
+    _check_price = models.Constraint(
+        'CHECK(price > 0)',
+        'Offer price must be strictly positive.',
+    )
     price = fields.Float(required=True)
     status = fields.Selection(selection=[('accepted', 'Accepted'), ('refused', 'Refused')], copy=False)
     partner_id = fields.Many2one('res.partner', required=True)
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index d4ada2546c4..9734876466a 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -5,4 +5,8 @@ class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Define the tags of the property"
 
+    _check_unique_tag = models.Constraint(
+        'UNIQUE(name)',
+        'Tag already exists.',
+    )
     name = fields.Char(required=True)
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 18503b6b16f..d4dd0129c51 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -5,4 +5,8 @@ class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Define the type of the property"
 
+    _check_unique_type = models.Constraint(
+        'UNIQUE(name)',
+        'Type already exists.',
+    )
     name = fields.Char(required=True)

From 0e0fa916198529cfef9259405b86d54ac1813945 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Thu, 23 Oct 2025 13:15:52 +0200
Subject: [PATCH 09/12] [IMP] Estate: Add the sprinkles

---
 estate/__manifest__.py                       |  4 +--
 estate/models/estate_property.py             |  8 ++----
 estate/models/estate_property_offer.py       | 11 ++++----
 estate/models/estate_property_tag.py         |  2 ++
 estate/models/estate_property_type.py        | 12 +++++++-
 estate/views/estate_property_offer_views.xml | 14 +++++++---
 estate/views/estate_property_tag_views.xml   | 12 +++++++-
 estate/views/estate_property_type_views.xml  | 29 ++++++++++++++++++--
 estate/views/estate_property_views.xml       | 22 +++++++++------
 9 files changed, 84 insertions(+), 30 deletions(-)

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 9790db8b2cb..3ea5b488ebb 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -12,10 +12,10 @@
     'data': [
         'security/ir.model.access.csv',
 
+        'views/estate_property_offer_views.xml',
         '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_property_type_views.xml',
         'views/estate_menus.xml',
     ],
     # # data files containing optionally loaded demonstration data
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 1fd59fdaa71..5149023cea3 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -7,6 +7,7 @@
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Real estate propreties"
+    _order = "id desc"
 
     _check_expected_price = models.Constraint(
         'CHECK(expected_price > 0)',
@@ -36,10 +37,10 @@ class EstateProperty(models.Model):
     seller_id = fields.Many2one('res.users', default=lambda self: self.env.user)
     buyer_id = fields.Many2one('res.partner')
     tag_ids = fields.Many2many('estate.property.tag', string="Tags")
+    type_id = fields.Many2one('estate.property.type', string="Type")
     offer_ids = fields.One2many('estate.property.offer', 'property_id')
     total_area = fields.Float(compute="_compute_total_area", readonly=True, copy=False)
     best_price = fields.Float(compute="_get_best_price", readonly=True, copy=False)
-    is_available = fields.Boolean(compute='_compute_is_available')
 
     @api.depends('living_area', 'garden_area')
     def _compute_total_area(self):
@@ -78,11 +79,6 @@ def action_set_cancelled(self):
         self.state = 'cancelled'
         return 1
 
-    @api.depends('offer_ids.status')
-    def _compute_is_available(self):
-        for record in self:
-            record.is_available = not any(s == 'accepted' for s in record.offer_ids.mapped('status'))
-
     @api.constrains('selling_price', 'expected_price')
     def _check_selling_price(self):
         self.ensure_one()
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 00b68d56055..d6cee880d3b 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -5,6 +5,7 @@
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Offers for propreties"
+    _order = "price desc"
 
     _check_price = models.Constraint(
         'CHECK(price > 0)',
@@ -17,7 +18,7 @@ class EstatePropertyOffer(models.Model):
     create_date = fields.Date(default=fields.Date.today(), readonly=True)
     validity = fields.Integer(default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
-    is_available = fields.Boolean(related="property_id.is_available")
+    property_type_id = fields.Many2one(related="property_id.type_id")
 
     @api.depends('validity', 'create_date')
     def _compute_date_deadline(self):
@@ -29,10 +30,10 @@ def _inverse_date_deadline(self):
             record.validity = (record.date_deadline - record.create_date).days
 
     def action_accept_offer(self):
-        for record in self:
-            record.status = 'accepted'
-            record.property_id.buyer_id = record.partner_id
-            record.property_id.selling_price = record.price
+        self.ensure_one()
+        self.status = 'accepted'
+        self.property_id.buyer_id = self.partner_id
+        self.property_id.selling_price = self.price
         return 1
 
     def action_refuse_offer(self):
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 9734876466a..afc6f569d15 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -4,9 +4,11 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Define the tags of the property"
+    _order = "name"
 
     _check_unique_tag = models.Constraint(
         'UNIQUE(name)',
         'Tag already exists.',
     )
     name = fields.Char(required=True)
+    color = fields.Integer(default=1)
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index d4dd0129c51..928c1f2abc8 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -1,12 +1,22 @@
-from odoo import fields, models
+from odoo import fields, models, api
 
 
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Define the type of the property"
+    _order = "sequence, name"
 
     _check_unique_type = models.Constraint(
         'UNIQUE(name)',
         'Type already exists.',
     )
     name = fields.Char(required=True)
+    sequence = fields.Integer('Sequence', default=1, help="Used to order stages. Lower is better.")
+    property_ids = fields.One2many("estate.property", "type_id")
+    offer_ids = fields.One2many("estate.property.offer", "property_type_id")
+    offer_count = fields.Integer(compute="_compute_offer_count", readonly=True)
+
+    @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 2ef2ec07c3c..93b080ac2dc 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -1,17 +1,23 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
+    <record id="estate_property_offer_from_property_action" model="ir.actions.act_window">
+        <field name="name">Offers</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>
+
     <record id="estate_property_offer_view_list" 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="Channel">
+            <list string="Channel" editable="bottom" decoration-success="status == 'accepted'" decoration-danger="status == 'refused'">
                 <field name="price"/>
                 <field name="partner_id" string="Partner"/>
                 <field name="validity" string="Validity (days)"/>
                 <field name="date_deadline" string="Deadline"/>
-                <button name="action_accept_offer" type="object" icon="fa-check" string="Accept" invisible="not is_available"/>
-                <button name="action_refuse_offer" type="object" icon="fa-times" string="Refuse"/>
-                <field name="status"/>
+                <button name="action_accept_offer" type="object" icon="fa-check" string="Accept" invisible="status"/>
+                <button name="action_refuse_offer" type="object" icon="fa-times" string="Refuse" invisible="status"/>
             </list>
         </field>
     </record>
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
index 27f8fbaf7a4..19ed261b949 100644
--- a/estate/views/estate_property_tag_views.xml
+++ b/estate/views/estate_property_tag_views.xml
@@ -6,7 +6,17 @@
         <field name="view_mode">list,form</field>
     </record>
 
-    <record id="estate_property_type_view_form" model="ir.ui.view">
+    <record id="estate_property_tag_view_list" model="ir.ui.view">
+        <field name="name">estate.property.tag.list</field>
+        <field name="model">estate.property.tag</field>
+        <field name="arch" type="xml">
+            <list string="Channel" editable="bottom">
+                <field name="name" string="Tag"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_property_tag_view_form" model="ir.ui.view">
         <field name="name">estate.property.tag.form</field>
         <field name="model">estate.property.tag</field>
         <field name="arch" type="xml">
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index 9f839b6c58c..00b3b6df9b5 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -6,15 +6,40 @@
         <field name="view_mode">list,form</field>
     </record>
 
+    <record id="estate_property_type_view_list" 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 string="Channel">
+                <field name="sequence" widget="handle"/>
+                <field name="name" string="Type"/>
+            </list>
+        </field>
+    </record>
+
     <record id="estate_property_type_view_form" model="ir.ui.view">
         <field name="name">estate.property.type.form</field>
         <field name="model">estate.property.type</field>
         <field name="arch" type="xml">
-            <form string="Propreties Types">
+            <form string="Type">
+                <header>
+                    <button class="oe_stat_button" type="action" name="%(estate.estate_property_offer_from_property_action)d" string="Offers"/>
+                </header>
                 <sheet>
                     <h1>
-                        <field name="name"/>
+                        <field name="name" string="Type"/>
                     </h1>
+                    <notebook>
+                        <page string="Properties">
+                            <field name="property_ids">
+                                <list>
+                                    <field name="name" string="Title"/>
+                                    <field name="expected_price"/>
+                                    <field name="state" string="Status"/>
+                                </list>
+                            </field>
+                        </page>
+                    </notebook>
                 </sheet>
             </form>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 0e5a282fa51..bf051d24acc 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -4,20 +4,22 @@
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
+        <field name="context">{'search_default_available': True}</field>
     </record>
 
     <record id="estate_property_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="Channel" decoration-success="offer_ids or buyer_id" decoration-bf="buyer_id" decoration-muted="state == 'cancelled'">
                 <field name="name" string="Title"/>
                 <field name="postcode"/>
+                <field name="tag_ids" string="Tags" widget="many2many_tags" options="{'color_field': 'color'}"/>
                 <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="hide"/>
             </list>
         </field>
     </record>
@@ -28,8 +30,9 @@
         <field name="arch" type="xml">
             <form string="Propreties">
                 <header>
-                    <button name="action_set_sold" string="Sold" type="object"/>
-                    <button name="action_set_cancelled" string="Cancel" type="object"/>
+                    <button name="action_set_sold" string="Sold" type="object" invisible="state in ['sold', 'cancelled']"/>
+                    <button name="action_set_cancelled" string="Cancel" type="object" invisible="state in ['sold', 'cancelled']"/>
+                    <field name="state" widget="statusbar" statusbar_visible="new,received,accepted,sold"/>
                 </header>
                 <sheet>
                     <h1>
@@ -39,6 +42,7 @@
                         <group>
                             <field name="postcode"/>
                             <field name="date_availability"/>
+                            <field name="type_id" options="{'no_quick_create': true, 'no_create_edit': true}"/>
                         </group>
                         <group>
                             <field name="expected_price"/>
@@ -55,14 +59,14 @@
                                 <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" string="Total Area (sqm)"/>
-                                <field name="tag_ids" widget="many2many_tags"/>
+                                <field name="tag_ids" widget="many2many_tags" options="{'color_field': 'color', 'no_create_edit': true}"/>
                             </group>
                         </page>
                         <page string="Offers">
-                            <field name="offer_ids"/>
+                            <field name="offer_ids" readonly="state in ['sold', 'cancelled', 'accepted']"/>
                         </page>
                         <page string="More Info">
                             <group>
@@ -85,7 +89,7 @@
                 <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"/>
                 <filter string="Available" name="available" domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]"/>
                 <filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>

From 35d66ec5dc17815804a20d558ba240eec86c3c66 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Thu, 23 Oct 2025 16:49:45 +0200
Subject: [PATCH 10/12] [IMP] Estate: Adding inheritance

[FIX] super()
---
 estate/__manifest__.py                 |  9 +--------
 estate/models/__init__.py              |  1 +
 estate/models/estate_property.py       |  9 +++++++--
 estate/models/estate_property_offer.py | 12 ++++++++++++
 estate/models/inherited_model.py       |  7 +++++++
 estate/views/inherited_model_views.xml | 15 +++++++++++++++
 6 files changed, 43 insertions(+), 10 deletions(-)
 create mode 100644 estate/models/inherited_model.py
 create mode 100644 estate/views/inherited_model_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 3ea5b488ebb..0d718eeaef6 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -5,21 +5,14 @@
     'author': "Victor Decleire",
     'application': True,
     'license': "LGPL-3",
-    # 'description': """
-    # Description text
-    # """,
-    # # data files always loaded at installation
     'data': [
         'security/ir.model.access.csv',
 
         'views/estate_property_offer_views.xml',
+        'views/inherited_model_views.xml',
         'views/estate_property_views.xml',
         'views/estate_property_tag_views.xml',
         'views/estate_property_type_views.xml',
         'views/estate_menus.xml',
     ],
-    # # data files containing optionally loaded demonstration data
-    # 'demo': [
-    #     'demo/demo_data.xml',
-    # ],
 }
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 2f1821a39c1..c0917a3d550 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -2,3 +2,4 @@
 from . import estate_property_type
 from . import estate_property_tag
 from . import estate_property_offer
+from . import inherited_model
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 5149023cea3..0fb3bce52c4 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -35,7 +35,7 @@ class EstateProperty(models.Model):
         selection=[('new', 'New'), ('received', 'Offer Received'), ('accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')],
         default='new', copy=False, required=True)
     seller_id = fields.Many2one('res.users', default=lambda self: self.env.user)
-    buyer_id = fields.Many2one('res.partner')
+    buyer_id = fields.Many2one('res.partner', readonly=True)
     tag_ids = fields.Many2many('estate.property.tag', string="Tags")
     type_id = fields.Many2one('estate.property.type', string="Type")
     offer_ids = fields.One2many('estate.property.offer', 'property_id')
@@ -82,5 +82,10 @@ def action_set_cancelled(self):
     @api.constrains('selling_price', 'expected_price')
     def _check_selling_price(self):
         self.ensure_one()
-        if len(self.offer_ids) > 0 and float_compare(self.expected_price, 0.9 * self.selling_price, 0) > 0:
+        if self.buyer_id and float_compare(self.expected_price, 0.9 * self.selling_price, 0) > 0:
             raise ValidationError("The selling price cannot be lower than 90% of the expected price.")
+
+    @api.ondelete(at_uninstall=False)
+    def _unlink_if_not_new_or_cancelled(self):
+        if any(record.state in ['new', 'cancelled'] for record in self):
+            raise UserError("Can't delete a property which has a state of new or cancelled!")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index d6cee880d3b..958441b1d2b 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,5 +1,7 @@
 from odoo import fields, models, api
 from odoo.tools.date_utils import add
+from odoo.exceptions import ValidationError
+from odoo.tools.float_utils import float_compare
 
 
 class EstatePropertyOffer(models.Model):
@@ -40,3 +42,13 @@ def action_refuse_offer(self):
         for record in self:
             record.status = 'refused'
         return 1
+
+    @api.model
+    def create(self, vals):
+        for val in vals:
+            created_property = self.env['estate.property'].browse(val['property_id'])
+            if any(float_compare(val['price'], offer.price, 0) < 0 for offer in created_property.offer_ids):
+                raise ValidationError("A bigger offer already exists")
+            if created_property.state == 'new':
+                created_property.state = 'received'
+        return super().create(vals)
diff --git a/estate/models/inherited_model.py b/estate/models/inherited_model.py
new file mode 100644
index 00000000000..532979ad4e6
--- /dev/null
+++ b/estate/models/inherited_model.py
@@ -0,0 +1,7 @@
+from odoo import fields, models
+
+
+class InheritedModel(models.Model):
+    _inherit = "res.users"
+
+    property_ids = fields.One2many("estate.property", "seller_id", domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]")
diff --git a/estate/views/inherited_model_views.xml b/estate/views/inherited_model_views.xml
new file mode 100644
index 00000000000..0b004299611
--- /dev/null
+++ b/estate/views/inherited_model_views.xml
@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <record id="res_users_view_form" model="ir.ui.view">
+        <field name="name">res.users.view.form.inherit</field>
+        <field name="model">res.users</field>
+        <field name="inherit_id" ref="base.view_users_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//page[@name='page_security']" position="after">
+                <page string="Real Estate Properties">
+                    <field name="property_ids"/>
+                </page>
+            </xpath>
+        </field>
+    </record>
+</odoo>

From e4e8e7b1d08b55c171b2d075c045fdc53bf22acd Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Fri, 24 Oct 2025 10:47:47 +0200
Subject: [PATCH 11/12] [IMP] Estate: Create invoice when sold

[FIX] style
---
 estate/__manifest__.py                   |  2 +-
 estate_account/__init__.py               |  1 +
 estate_account/__manifest__.py           | 13 +++++++++++++
 estate_account/models/__init__.py        |  1 +
 estate_account/models/estate_property.py | 24 ++++++++++++++++++++++++
 5 files changed, 40 insertions(+), 1 deletion(-)
 create mode 100644 estate_account/__init__.py
 create mode 100644 estate_account/__manifest__.py
 create mode 100644 estate_account/models/__init__.py
 create mode 100644 estate_account/models/estate_property.py

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 0d718eeaef6..630f4458701 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -1,5 +1,5 @@
 {
-    'name': "estate",
+    'name': "Estate",
     'version': '1.0',
     'depends': ['base'],
     'author': "Victor Decleire",
diff --git a/estate_account/__init__.py b/estate_account/__init__.py
new file mode 100644
index 00000000000..0650744f6bc
--- /dev/null
+++ b/estate_account/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
new file mode 100644
index 00000000000..e3dc973fadd
--- /dev/null
+++ b/estate_account/__manifest__.py
@@ -0,0 +1,13 @@
+{
+    'name': "Estate Account",
+    'version': '1.0',
+    'depends': [
+        'estate',
+        'account'
+        ],
+    'author': "Victor Decleire",
+    'application': True,
+    'license': "LGPL-3",
+    'data': [
+    ],
+}
diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py
new file mode 100644
index 00000000000..5e1963c9d2f
--- /dev/null
+++ b/estate_account/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_property
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py
new file mode 100644
index 00000000000..ee7a4d51dae
--- /dev/null
+++ b/estate_account/models/estate_property.py
@@ -0,0 +1,24 @@
+from odoo import models, Command
+
+
+class EstatePropertyInherited(models.Model):
+    _inherit = "estate.property"
+
+    def action_set_sold(self):
+        super().action_set_sold()
+        self.env['account.move'].create({
+            'move_type': 'out_invoice',
+            'partner_id': self.buyer_id.id,
+            'invoice_line_ids': [
+                Command.create({
+                    'name': self.name,
+                    'quantity': 1,
+                    'price_unit': self.selling_price * 0.06
+                }),
+                Command.create({
+                    'name': 'Administrative fees',
+                    'quantity': 1,
+                    'price_unit': 100.00
+                })
+            ],
+        })

From 30a25369387c833c5daf692677f2c24155c91347 Mon Sep 17 00:00:00 2001
From: Victor Decleire <vdecleire@gmail.com>
Date: Fri, 24 Oct 2025 11:41:24 +0200
Subject: [PATCH 12/12] [IMP] Estate: Create kanban view for estate proprety
 model

[FIX] Refactor

[FIX] Correct style
---
 estate/models/estate_property.py         | 39 ++++++++++++------------
 estate/models/estate_property_offer.py   | 29 +++++++++---------
 estate/models/estate_property_tag.py     |  5 +--
 estate/models/estate_property_type.py    |  9 +++---
 estate/views/estate_property_views.xml   | 34 ++++++++++++++++++++-
 estate_account/models/estate_property.py |  1 +
 6 files changed, 77 insertions(+), 40 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 0fb3bce52c4..c52ce613fa8 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -9,14 +9,6 @@ class EstateProperty(models.Model):
     _description = "Real estate propreties"
     _order = "id desc"
 
-    _check_expected_price = models.Constraint(
-        'CHECK(expected_price > 0)',
-        'Expected price must be strictly positive.',
-    )
-    _check_selling_price = models.Constraint(
-        'CHECK(selling_price >= 0)',
-        'Selling price must be positive.',
-    )
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()
@@ -42,6 +34,15 @@ class EstateProperty(models.Model):
     total_area = fields.Float(compute="_compute_total_area", readonly=True, copy=False)
     best_price = fields.Float(compute="_get_best_price", readonly=True, copy=False)
 
+    _check_expected_price = models.Constraint(
+        'CHECK(expected_price > 0)',
+        'Expected price must be strictly positive.',
+    )
+    _check_selling_price = models.Constraint(
+        'CHECK(selling_price >= 0)',
+        'Selling price must be positive.',
+    )
+
     @api.depends('living_area', 'garden_area')
     def _compute_total_area(self):
         for record in self:
@@ -55,6 +56,12 @@ def _get_best_price(self):
             else:
                 record.best_price = 0
 
+    @api.constrains('selling_price', 'expected_price')
+    def _check_selling_price(self):
+        self.ensure_one()
+        if self.buyer_id and float_compare(self.expected_price, 0.9 * self.selling_price, 0) > 0:
+            raise ValidationError("The selling price cannot be lower than 90% of the expected price.")
+
     @api.onchange('garden')
     def _onchange_garden(self):
         self.ensure_one()
@@ -65,6 +72,11 @@ def _onchange_garden(self):
             self.garden_area = 0
             self.garden_orientation = None
 
+    @api.ondelete(at_uninstall=False)
+    def _unlink_if_not_new_or_cancelled(self):
+        if any(record.state in ['new', 'cancelled'] for record in self):
+            raise UserError("Can't delete a property which has a state of new or cancelled!")
+
     def action_set_sold(self):
         self.ensure_one()
         if self.state == 'cancelled':
@@ -78,14 +90,3 @@ def action_set_cancelled(self):
             raise UserError("Sold property cannot be cancelled")
         self.state = 'cancelled'
         return 1
-
-    @api.constrains('selling_price', 'expected_price')
-    def _check_selling_price(self):
-        self.ensure_one()
-        if self.buyer_id and float_compare(self.expected_price, 0.9 * self.selling_price, 0) > 0:
-            raise ValidationError("The selling price cannot be lower than 90% of the expected price.")
-
-    @api.ondelete(at_uninstall=False)
-    def _unlink_if_not_new_or_cancelled(self):
-        if any(record.state in ['new', 'cancelled'] for record in self):
-            raise UserError("Can't delete a property which has a state of new or cancelled!")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 958441b1d2b..bcec30e58fc 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -9,10 +9,6 @@ class EstatePropertyOffer(models.Model):
     _description = "Offers for propreties"
     _order = "price desc"
 
-    _check_price = models.Constraint(
-        'CHECK(price > 0)',
-        'Offer price must be strictly positive.',
-    )
     price = fields.Float(required=True)
     status = fields.Selection(selection=[('accepted', 'Accepted'), ('refused', 'Refused')], copy=False)
     partner_id = fields.Many2one('res.partner', required=True)
@@ -22,6 +18,11 @@ class EstatePropertyOffer(models.Model):
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
     property_type_id = fields.Many2one(related="property_id.type_id")
 
+    _check_price = models.Constraint(
+        'CHECK(price > 0)',
+        'Offer price must be strictly positive.',
+    )
+
     @api.depends('validity', 'create_date')
     def _compute_date_deadline(self):
         for record in self:
@@ -31,6 +32,16 @@ def _inverse_date_deadline(self):
         for record in self:
             record.validity = (record.date_deadline - record.create_date).days
 
+    @api.model
+    def create(self, vals):
+        for val in vals:
+            created_property = self.env['estate.property'].browse(val['property_id'])
+            if any(float_compare(val['price'], offer.price, 0) < 0 for offer in created_property.offer_ids):
+                raise ValidationError("A bigger offer already exists")
+            if created_property.state == 'new':
+                created_property.state = 'received'
+        return super().create(vals)
+
     def action_accept_offer(self):
         self.ensure_one()
         self.status = 'accepted'
@@ -42,13 +53,3 @@ def action_refuse_offer(self):
         for record in self:
             record.status = 'refused'
         return 1
-
-    @api.model
-    def create(self, vals):
-        for val in vals:
-            created_property = self.env['estate.property'].browse(val['property_id'])
-            if any(float_compare(val['price'], offer.price, 0) < 0 for offer in created_property.offer_ids):
-                raise ValidationError("A bigger offer already exists")
-            if created_property.state == 'new':
-                created_property.state = 'received'
-        return super().create(vals)
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index afc6f569d15..8948e299468 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -6,9 +6,10 @@ class EstatePropertyTag(models.Model):
     _description = "Define the tags of the property"
     _order = "name"
 
+    name = fields.Char(required=True)
+    color = fields.Integer(default=1)
+
     _check_unique_tag = models.Constraint(
         'UNIQUE(name)',
         'Tag already exists.',
     )
-    name = fields.Char(required=True)
-    color = fields.Integer(default=1)
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 928c1f2abc8..d4c8d014ea3 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -6,10 +6,6 @@ class EstatePropertyType(models.Model):
     _description = "Define the type of the property"
     _order = "sequence, name"
 
-    _check_unique_type = models.Constraint(
-        'UNIQUE(name)',
-        'Type already exists.',
-    )
     name = fields.Char(required=True)
     sequence = fields.Integer('Sequence', default=1, help="Used to order stages. Lower is better.")
     property_ids = fields.One2many("estate.property", "type_id")
@@ -20,3 +16,8 @@ class EstatePropertyType(models.Model):
     def _compute_offer_count(self):
         for record in self:
             record.offer_count = len(record.offer_ids)
+
+    _check_unique_type = models.Constraint(
+        'UNIQUE(name)',
+        'Type already exists.',
+    )
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index bf051d24acc..63bae20d086 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -3,7 +3,7 @@
     <record id="estate_property_model_action" model="ir.actions.act_window">
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>
-        <field name="view_mode">list,form</field>
+        <field name="view_mode">list,form,kanban</field>
         <field name="context">{'search_default_available': True}</field>
     </record>
 
@@ -80,6 +80,38 @@
         </field>
     </record>
 
+    <record id="estate_property_view_kanban" model="ir.ui.view">
+        <field name="name">estate.property.kanban</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <kanban default_group_by="type_id" records_draggable="false">
+                <field name="state"/>
+                <templates>
+                    <t t-name="card">
+                        <h2>
+                            <field name="name"/>
+                        </h2>
+                        <div>
+                            Expected price: 
+                            <field name="expected_price"/>
+                        </div>
+                        <div t-if="record.state.raw_value == 'received'">
+                            Offer received: 
+                            <field name="best_price"/>
+                        </div>
+                        <div t-if="record.state.raw_value == 'accepted'">
+                            Selled for:
+                            <field name="selling_price"/>
+                        </div>
+                        <div>
+                            <field name="tag_ids" widget="many2many_tags" options="{'color_field': 'color', 'no_create_edit': true}"/>
+                        </div>
+                    </t>
+                </templates>
+            </kanban>
+        </field>
+    </record>
+
     <record id="estate_property_search" model="ir.ui.view">
         <field name="name">estate.property.search</field>
         <field name="model">estate.property</field>
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py
index ee7a4d51dae..4594f877c2e 100644
--- a/estate_account/models/estate_property.py
+++ b/estate_account/models/estate_property.py
@@ -22,3 +22,4 @@ def action_set_sold(self):
                 })
             ],
         })
+        return 1