From 967fc30d13cfe7687cac5f4f858e28c1e80d3fb2 Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Tue, 18 Nov 2025 14:10:33 +0100
Subject: [PATCH 01/15] [ADD] estate module

---
 estate/__init__.py     |  0
 estate/__manifest__.py | 25 +++++++++++++++++++++++++
 2 files changed, 25 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..9d570a86a8b
--- /dev/null
+++ b/estate/__manifest__.py
@@ -0,0 +1,25 @@
+# -*- coding: utf-8 -*-
+{
+    'name': "Real Estate",
+
+    'summary': """
+        Cool Real Estate App 
+    """,
+
+    'description': """
+        Cool Real Estate App 
+    """,
+
+    'author': "kmhma",
+    'website': "https://www.odoo.com/",
+    'category': 'Tutorials',
+    'version': '0.1',
+    'application': True,
+    'installable': True,
+    'depends': ['base'],
+
+    'data': [],
+    'assets': {
+    },
+    'license': 'AGPL-3'
+}

From ba2124fbe14aaf7a2aaa72d4877f57e378ff8557 Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Tue, 18 Nov 2025 15:06:36 +0100
Subject: [PATCH 02/15] [ADD] estate: add estate_property fields

---
 estate/__init__.py               |  1 +
 estate/models/__init__.py        |  1 +
 estate/models/estate_property.py | 23 +++++++++++++++++++++++
 3 files changed, 25 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..5e1963c9d2f
--- /dev/null
+++ b/estate/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_property
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
new file mode 100644
index 00000000000..ae41f7e1c72
--- /dev/null
+++ b/estate/models/estate_property.py
@@ -0,0 +1,23 @@
+from odoo import models, fields
+
+class estateProperty(models.Model):
+    _name = "estate.property"
+    _description = "Real Estate model"
+
+    name = fields.Char(required=True)
+    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='Type',
+        selection=[('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')]
+        )
+
+

From f920c0a9c9c4e993b257777c4907971da0b8db4d Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Tue, 18 Nov 2025 15:51:52 +0100
Subject: [PATCH 03/15] [ADD] estate: add access rights

---
 estate/__init__.py                  |  2 +-
 estate/__manifest__.py              |  7 +++----
 estate/models/estate_property.py    | 16 ++++++++++------
 estate/security/ir.model.access.csv |  2 ++
 4 files changed, 16 insertions(+), 11 deletions(-)
 create mode 100644 estate/security/ir.model.access.csv

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 9d570a86a8b..1c248e12315 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -1,15 +1,12 @@
-# -*- coding: utf-8 -*-
 {
     'name': "Real Estate",
 
     'summary': """
         Cool Real Estate App 
     """,
-
     'description': """
         Cool Real Estate App 
     """,
-
     'author': "kmhma",
     'website': "https://www.odoo.com/",
     'category': 'Tutorials',
@@ -18,7 +15,9 @@
     'installable': True,
     'depends': ['base'],
 
-    'data': [],
+    'data': [
+        "security/ir.model.access.csv"
+    ],
     'assets': {
     },
     'license': 'AGPL-3'
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index ae41f7e1c72..ec365b17ee5 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,11 +1,12 @@
 from odoo import models, fields
 
+
 class estateProperty(models.Model):
     _name = "estate.property"
     _description = "Real Estate model"
 
     name = fields.Char(required=True)
-    postcode= fields.Char()
+    postcode = fields.Char()
     date_availability = fields.Date()
     expected_price = fields.Float(required=True)
     selling_price = fields.Float()
@@ -16,8 +17,11 @@ class estateProperty(models.Model):
     garden = fields.Boolean()
     garden_area = fields.Integer()
     garden_orientation = fields.Selection(
-        string='Type',
-        selection=[('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')]
-        )
-
-
+        string="Type",
+        selection=[
+            ("north", "North"),
+            ("south", "South"),
+            ("east", "East"),
+            ("west", "West"),
+        ],
+    )
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
new file mode 100644
index 00000000000..c6d251ec585
--- /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_property,access.estate.property,model_estate_property,base.group_user,1,1,1,1
\ No newline at end of file

From e722790a0e0c076c89476200abff12f5e22770cf Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Tue, 18 Nov 2025 17:14:24 +0100
Subject: [PATCH 04/15] [ADD] estate: add action, menu, and active and  state
 fields

---
 estate/__manifest__.py                 |  8 +++++---
 estate/models/estate_property.py       | 26 ++++++++++++++++++++++----
 estate/views/estate_menus.xml          |  8 ++++++++
 estate/views/estate_property_views.xml |  8 ++++++++
 4 files changed, 43 insertions(+), 7 deletions(-)
 create mode 100644 estate/views/estate_menus.xml
 create mode 100644 estate/views/estate_property_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 1c248e12315..a20084bae9e 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -2,10 +2,10 @@
     'name': "Real Estate",
 
     'summary': """
-        Cool Real Estate App 
+        Cool Real Estate App
     """,
     'description': """
-        Cool Real Estate App 
+        Cool Real Estate App
     """,
     'author': "kmhma",
     'website': "https://www.odoo.com/",
@@ -16,7 +16,9 @@
     'depends': ['base'],
 
     'data': [
-        "security/ir.model.access.csv"
+        "security/ir.model.access.csv",
+        "views/estate_property_views.xml",
+        "views/estate_menus.xml"
     ],
     'assets': {
     },
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index ec365b17ee5..5ada62a0c3d 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,5 @@
 from odoo import models, fields
+from dateutil.relativedelta import relativedelta
 
 
 class estateProperty(models.Model):
@@ -7,17 +8,20 @@ class estateProperty(models.Model):
 
     name = fields.Char(required=True)
     postcode = fields.Char()
-    date_availability = fields.Date()
+    date_availability = fields.Date(
+        copy=False, default=fields.Datetime.today() + relativedelta(months=3)
+    )
     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()
     garden = fields.Boolean()
+    active = fields.Boolean(default=True)
     garden_area = fields.Integer()
     garden_orientation = fields.Selection(
-        string="Type",
+        string="Orientation",
         selection=[
             ("north", "North"),
             ("south", "South"),
@@ -25,3 +29,17 @@ class estateProperty(models.Model):
             ("west", "West"),
         ],
     )
+    state = fields.Selection(
+        string="State",
+        selection=[
+            ("new", "New"),
+            ("offer", "Offer"),
+            ("received", "Received"),
+            ("offer accepted", "Offer Accepted"),
+            ("sold", "Sold"),
+            ("cancelled", "Cancelled"),
+        ],
+        required=True,
+        copy=False,
+        default="new",
+    )
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
new file mode 100644
index 00000000000..36f8848f8e4
--- /dev/null
+++ b/estate/views/estate_menus.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+<menuitem id="estate_menu_root" name="Estate Property">
+    <menuitem id="advertisement_menu" name="Advertisments">
+        <menuitem id="properties_action" action="estate_property_view"/>
+    </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..228c66c8f46
--- /dev/null
+++ b/estate/views/estate_property_views.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_property_view" model="ir.actions.act_window">
+        <field name="name">Estate Property</field>
+        <field name="res_model">estate.property</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>
\ No newline at end of file

From 815f4b0b36836b8de5363a159d5423bd9024db57 Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Wed, 19 Nov 2025 11:12:58 +0100
Subject: [PATCH 05/15] [ADD] estate: add attrs to list view, restructured form
 view, add search view

---
 estate/__manifest__.py                 |  5 ++-
 estate/models/estate_property.py       | 12 +++----
 estate/views/estate_form_view.xml      | 44 ++++++++++++++++++++++++++
 estate/views/estate_list_view.xml      | 18 +++++++++++
 estate/views/estate_property_views.xml |  2 +-
 estate/views/estate_search_view.xml    | 21 ++++++++++++
 6 files changed, 94 insertions(+), 8 deletions(-)
 create mode 100644 estate/views/estate_form_view.xml
 create mode 100644 estate/views/estate_list_view.xml
 create mode 100644 estate/views/estate_search_view.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index a20084bae9e..26d106e1454 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -18,7 +18,10 @@
     'data': [
         "security/ir.model.access.csv",
         "views/estate_property_views.xml",
-        "views/estate_menus.xml"
+        "views/estate_menus.xml",
+        "views/estate_list_view.xml",
+        "views/estate_form_view.xml",
+        "views/estate_search_view.xml",
     ],
     'assets': {
     },
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 5ada62a0c3d..a98c905d9b1 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -7,21 +7,22 @@ class estateProperty(models.Model):
     _description = "Real Estate model"
 
     name = fields.Char(required=True)
+    description = fields.Char()
     postcode = fields.Char()
-    date_availability = fields.Date(
+    available_from = fields.Date(
         copy=False, default=fields.Datetime.today() + relativedelta(months=3)
     )
     expected_price = fields.Float(required=True)
     selling_price = fields.Float(readonly=True, copy=False)
     bedrooms = fields.Integer(default=2)
-    living_area = fields.Integer()
+    living_area = fields.Integer(string="Living Area (sqm)")
     facades = fields.Integer()
     garage = fields.Boolean()
     garden = fields.Boolean()
     active = fields.Boolean(default=True)
-    garden_area = fields.Integer()
+    garden_area = fields.Integer(string="Garden Area (sqm)")
     garden_orientation = fields.Selection(
-        string="Orientation",
+        string="Garden Orientation",
         selection=[
             ("north", "North"),
             ("south", "South"),
@@ -33,8 +34,7 @@ class estateProperty(models.Model):
         string="State",
         selection=[
             ("new", "New"),
-            ("offer", "Offer"),
-            ("received", "Received"),
+            ("offer received", "Offer Received"),
             ("offer accepted", "Offer Accepted"),
             ("sold", "Sold"),
             ("cancelled", "Cancelled"),
diff --git a/estate/views/estate_form_view.xml b/estate/views/estate_form_view.xml
new file mode 100644
index 00000000000..dd592fef525
--- /dev/null
+++ b/estate/views/estate_form_view.xml
@@ -0,0 +1,44 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_form_view" model="ir.ui.view">
+        <field name="name">estate.property.form</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <form string="Properties">
+                <sheet>
+                    <group>
+                            <h1 class="mb32">
+                            <field name="name" class="mb16"/>
+                            </h1>
+                    </group>
+                    <group>
+                        <group>
+                            <field name="postcode"/>
+                            <field name="available_from"/>
+                            <field name="state"/>
+                        </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"/>
+                                <field name="facades"/>
+                                <field name="garage"/>
+                                <field name="garden"/>
+                                <field name="garden_area"/>
+                                <field name="garden_orientation"/>
+                            </group>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
+</odoo>
\ No newline at end of file
diff --git a/estate/views/estate_list_view.xml b/estate/views/estate_list_view.xml
new file mode 100644
index 00000000000..968d0fdcb17
--- /dev/null
+++ b/estate/views/estate_list_view.xml
@@ -0,0 +1,18 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_list_view" model="ir.ui.view">
+        <field name="name">estate.property.list</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <list string="Properties">
+                <field name="name"/>
+                <field name="postcode"/>
+                <field name="bedrooms"/>
+                <field name="living_area"/>
+                <field name="expected_price"/>
+                <field name="selling_price"/>
+                <field name="available_from"/>
+            </list>
+        </field>
+    </record>
+</odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 228c66c8f46..b4ecd555e96 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
 <odoo>
     <record id="estate_property_view" model="ir.actions.act_window">
-        <field name="name">Estate Property</field>
+        <field name="name">estate.property.view</field>
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
     </record>
diff --git a/estate/views/estate_search_view.xml b/estate/views/estate_search_view.xml
new file mode 100644
index 00000000000..7e7db70fea5
--- /dev/null
+++ b/estate/views/estate_search_view.xml
@@ -0,0 +1,21 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate_search_view" model="ir.ui.view">
+        <field name="name">estate.property.search</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <search string="Estate Property">
+                <field name="name"/>
+                <field name="postcode"/>
+                <field name="bedrooms"/>
+                <field name="living_area"/>
+                <field name="expected_price"/>
+                <field name="facades"/>
+
+                <filter string="Available" name="state" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer received')]"/>
+
+                <filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>
+            </search>
+        </field>
+    </record>
+</odoo>
\ No newline at end of file

From 783aac29c176ec47398cdf028cd68a37a4bcb03a Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Wed, 19 Nov 2025 11:26:25 +0100
Subject: [PATCH 06/15] [FIX] estate: add missing EOLs, trailing commas

---
 estate/__manifest__.py                 | 11 ++++-------
 estate/models/estate_property.py       |  5 ++---
 estate/security/ir.model.access.csv    |  2 +-
 estate/views/estate_form_view.xml      |  2 +-
 estate/views/estate_list_view.xml      |  2 +-
 estate/views/estate_menus.xml          |  2 +-
 estate/views/estate_property_views.xml |  2 +-
 estate/views/estate_search_view.xml    |  2 +-
 8 files changed, 12 insertions(+), 16 deletions(-)

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 26d106e1454..026c1bdf78d 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -1,11 +1,10 @@
 {
     'name': "Real Estate",
 
-    'summary': """
-        Cool Real Estate App
-    """,
+    'summary': "Cool Real Estate App",
     'description': """
-        Cool Real Estate App
+Cool Real Estate App
+Wow This is a description omgg
     """,
     'author': "kmhma",
     'website': "https://www.odoo.com/",
@@ -23,7 +22,5 @@
         "views/estate_form_view.xml",
         "views/estate_search_view.xml",
     ],
-    'assets': {
-    },
-    'license': 'AGPL-3'
+    'license': 'LGPL-3',
 }
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index a98c905d9b1..495afc5fd92 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,8 +1,7 @@
 from odoo import models, fields
-from dateutil.relativedelta import relativedelta
 
 
-class estateProperty(models.Model):
+class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Real Estate model"
 
@@ -10,7 +9,7 @@ class estateProperty(models.Model):
     description = fields.Char()
     postcode = fields.Char()
     available_from = fields.Date(
-        copy=False, default=fields.Datetime.today() + relativedelta(months=3)
+        copy=False, default=fields.Date.add(fields.Date.today(), months=3)
     )
     expected_price = fields.Float(required=True)
     selling_price = fields.Float(readonly=True, copy=False)
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index c6d251ec585..f0cdfdcac54 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_property,access.estate.property,model_estate_property,base.group_user,1,1,1,1
\ No newline at end of file
+access_estate_property,access.estate.property,model_estate_property,base.group_user,1,1,1,1
diff --git a/estate/views/estate_form_view.xml b/estate/views/estate_form_view.xml
index dd592fef525..3c30a636425 100644
--- a/estate/views/estate_form_view.xml
+++ b/estate/views/estate_form_view.xml
@@ -41,4 +41,4 @@
             </form>
         </field>
     </record>
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/views/estate_list_view.xml b/estate/views/estate_list_view.xml
index 968d0fdcb17..b07492282a6 100644
--- a/estate/views/estate_list_view.xml
+++ b/estate/views/estate_list_view.xml
@@ -15,4 +15,4 @@
             </list>
         </field>
     </record>
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index 36f8848f8e4..598ea192cb7 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -5,4 +5,4 @@
         <menuitem id="properties_action" action="estate_property_view"/>
     </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 b4ecd555e96..36f3cae2342 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -5,4 +5,4 @@
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
     </record>
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/views/estate_search_view.xml b/estate/views/estate_search_view.xml
index 7e7db70fea5..329f7767647 100644
--- a/estate/views/estate_search_view.xml
+++ b/estate/views/estate_search_view.xml
@@ -18,4 +18,4 @@
             </search>
         </field>
     </record>
-</odoo>
\ No newline at end of file
+</odoo>

From a64bdd2699a61211b4f842b286906b94ddddd326 Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Wed, 19 Nov 2025 15:52:20 +0100
Subject: [PATCH 07/15] [ADD] estate: add relations with salesman, buyer, tags,
 and offers

---
 estate/__manifest__.py                        | 10 ++++---
 estate/models/__init__.py                     |  7 ++++-
 estate/models/estate_property.py              |  9 ++++++
 estate/models/estate_property_offer.py        | 17 +++++++++++
 estate/models/estate_property_tag.py          |  8 +++++
 estate/models/estate_property_type.py         |  8 +++++
 estate/security/ir.model.access.csv           |  3 ++
 ...te_form_view.xml => estate_form_views.xml} | 30 ++++++++++++++++---
 ...te_list_view.xml => estate_list_views.xml} |  1 +
 ...state_menus.xml => estate_menus_views.xml} |  5 ++++
 estate/views/estate_property_views.xml        |  2 +-
 ...earch_view.xml => estate_search_views.xml} |  1 +
 estate/views/property_tag_views.xml           |  8 +++++
 estate/views/property_type_views.xml          |  8 +++++
 14 files changed, 107 insertions(+), 10 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
 rename estate/views/{estate_form_view.xml => estate_form_views.xml} (62%)
 rename estate/views/{estate_list_view.xml => estate_list_views.xml} (92%)
 rename estate/views/{estate_menus.xml => estate_menus_views.xml} (54%)
 rename estate/views/{estate_search_view.xml => estate_search_views.xml} (94%)
 create mode 100644 estate/views/property_tag_views.xml
 create mode 100644 estate/views/property_type_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 026c1bdf78d..f8d4fdfb4c2 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -17,10 +17,12 @@
     'data': [
         "security/ir.model.access.csv",
         "views/estate_property_views.xml",
-        "views/estate_menus.xml",
-        "views/estate_list_view.xml",
-        "views/estate_form_view.xml",
-        "views/estate_search_view.xml",
+        "views/property_type_views.xml",
+        "views/property_tag_views.xml",
+        "views/estate_menus_views.xml",
+        "views/estate_list_views.xml",
+        "views/estate_form_views.xml",
+        "views/estate_search_views.xml",
     ],
     'license': 'LGPL-3',
 }
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 5e1963c9d2f..a0a8b8c501c 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1 +1,6 @@
-from . import estate_property
+from . import (
+    estate_property,
+    estate_property_type,
+    estate_property_tag,
+    estate_property_offer,
+)
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 495afc5fd92..75e5c9a3368 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -42,3 +42,12 @@ class EstateProperty(models.Model):
         copy=False,
         default="new",
     )
+    property_type_id = fields.Many2one(
+        comodel_name="estate.property.type", string="Property Type"
+    )
+    buyer = fields.Many2one(comodel_name="res.partner", copy=False)
+    salesman = fields.Many2one(
+        comodel_name="res.users", default=lambda self: self.env.uid
+    )
+    tag_ids = fields.Many2many(comodel_name="estate.property.tag", string="Tags")
+    offer_ids = fields.One2many(comodel_name="estate.property.offer", inverse_name="property_id")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
new file mode 100644
index 00000000000..6e17fc2d869
--- /dev/null
+++ b/estate/models/estate_property_offer.py
@@ -0,0 +1,17 @@
+from odoo import models, fields
+
+
+class EstatePropertyOffer(models.Model):
+    _name = "estate.property.offer"
+    _description = "property offer"
+
+    price = fields.Float()
+    status = fields.Selection(
+        copy=False,
+        selection=[
+            ("accepted", "Accepted"),
+            ("refused", "Refused"),
+        ],
+    )
+    partner_id = fields.Many2one("res.partner", required=True, string="Partner")
+    property_id = fields.Many2one("estate.property", required=True, string="Property")
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
new file mode 100644
index 00000000000..6f4569f5954
--- /dev/null
+++ b/estate/models/estate_property_tag.py
@@ -0,0 +1,8 @@
+from odoo import models, fields
+
+
+class EstatePropertyTag(models.Model):
+    _name = "estate.property.tag"
+    _description = "property tags"
+
+    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..2e07a19bf05
--- /dev/null
+++ b/estate/models/estate_property_type.py
@@ -0,0 +1,8 @@
+from odoo import models, fields
+
+
+class EstatePropertyType(models.Model):
+    _name = "estate.property.type"
+    _description = "property types"
+
+    name = fields.Char(required=True)
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index f0cdfdcac54..e20ec4dd90b 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_property,access.estate.property,model_estate_property,base.group_user,1,1,1,1
+access_estate_property_type,access.estate.property.type,model_estate_property_type,base.group_user,1,1,1,1
+access_estate_property_tag,access.estate.property.tag,model_estate_property_tag,base.group_user,1,1,1,1
+access_estate_property_offer,access.estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1
diff --git a/estate/views/estate_form_view.xml b/estate/views/estate_form_views.xml
similarity index 62%
rename from estate/views/estate_form_view.xml
rename to estate/views/estate_form_views.xml
index 3c30a636425..ee6906ba221 100644
--- a/estate/views/estate_form_view.xml
+++ b/estate/views/estate_form_views.xml
@@ -7,22 +7,26 @@
             <form string="Properties">
                 <sheet>
                     <group>
-                            <h1 class="mb32">
+                        <h1 class="mb32">
                             <field name="name" class="mb16"/>
-                            </h1>
+                        </h1>
                     </group>
+
                     <group>
                         <group>
+                            <field name="tag_ids" widget="many2many_tags"/>
                             <field name="postcode"/>
                             <field name="available_from"/>
-                            <field name="state"/>
+                            <field name="property_type_id" string="Property Type"/>
                         </group>
 
-                        <group> 
+                        <group>
                             <field name="expected_price"/>
                             <field name="selling_price"/>
+                            <field name="state"/>
                         </group>
                     </group>
+
                     <notebook>
                         <page string="Description">
                             <group>
@@ -36,7 +40,25 @@
                                 <field name="garden_orientation"/>
                             </group>
                         </page>
+
+                        <page string="Offers">
+                            <field name="offer_ids">
+                                <list>
+                                    <field name="price"/>
+                                    <field name="partner_id"/>
+                                    <field name="status"/>
+                                </list>
+                            </field>
+                        </page>
+
+                        <page string="Other Info">
+                            <group>
+                                <field name="buyer"/>
+                                <field name="salesman"/>
+                            </group>
+                        </page>
                     </notebook>
+
                 </sheet>
             </form>
         </field>
diff --git a/estate/views/estate_list_view.xml b/estate/views/estate_list_views.xml
similarity index 92%
rename from estate/views/estate_list_view.xml
rename to estate/views/estate_list_views.xml
index b07492282a6..34a9d57d9b7 100644
--- a/estate/views/estate_list_view.xml
+++ b/estate/views/estate_list_views.xml
@@ -7,6 +7,7 @@
             <list string="Properties">
                 <field name="name"/>
                 <field name="postcode"/>
+                <field name="property_type_id"/>
                 <field name="bedrooms"/>
                 <field name="living_area"/>
                 <field name="expected_price"/>
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus_views.xml
similarity index 54%
rename from estate/views/estate_menus.xml
rename to estate/views/estate_menus_views.xml
index 598ea192cb7..1721ca49a12 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus_views.xml
@@ -4,5 +4,10 @@
     <menuitem id="advertisement_menu" name="Advertisments">
         <menuitem id="properties_action" action="estate_property_view"/>
     </menuitem>
+
+    <menuitem id="settings_menu" name="Settings">
+        <menuitem id="property_type_action" action="property_type_view"/>
+        <menuitem id="property_tag_action" action="property_tag_view"/>
+    </menuitem>
 </menuitem>
 </odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 36f3cae2342..c946bb3eeea 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
 <odoo>
     <record id="estate_property_view" model="ir.actions.act_window">
-        <field name="name">estate.property.view</field>
+        <field name="name">Estate Property</field>
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
     </record>
diff --git a/estate/views/estate_search_view.xml b/estate/views/estate_search_views.xml
similarity index 94%
rename from estate/views/estate_search_view.xml
rename to estate/views/estate_search_views.xml
index 329f7767647..778efe89f0c 100644
--- a/estate/views/estate_search_view.xml
+++ b/estate/views/estate_search_views.xml
@@ -11,6 +11,7 @@
                 <field name="living_area"/>
                 <field name="expected_price"/>
                 <field name="facades"/>
+                <field name="property_type_id"/>
 
                 <filter string="Available" name="state" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer received')]"/>
 
diff --git a/estate/views/property_tag_views.xml b/estate/views/property_tag_views.xml
new file mode 100644
index 00000000000..0c05d08f533
--- /dev/null
+++ b/estate/views/property_tag_views.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="property_tag_view" 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>
+</odoo>
diff --git a/estate/views/property_type_views.xml b/estate/views/property_type_views.xml
new file mode 100644
index 00000000000..ecc59967464
--- /dev/null
+++ b/estate/views/property_type_views.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="property_type_view" 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>
+</odoo>

From 58f471ae9db3537d024844bc424c412dcc2cac2a Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Wed, 19 Nov 2025 16:23:51 +0100
Subject: [PATCH 08/15] [MOV] estate: collect estate property views in a single
 file

---
 estate/__manifest__.py                 |   5 +-
 estate/views/estate_form_views.xml     |  66 ----------------
 estate/views/estate_list_views.xml     |  19 -----
 estate/views/estate_property_views.xml | 101 +++++++++++++++++++++++++
 estate/views/estate_search_views.xml   |  22 ------
 5 files changed, 102 insertions(+), 111 deletions(-)
 delete mode 100644 estate/views/estate_form_views.xml
 delete mode 100644 estate/views/estate_list_views.xml
 delete mode 100644 estate/views/estate_search_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index f8d4fdfb4c2..17c906b7f14 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -16,13 +16,10 @@
 
     'data': [
         "security/ir.model.access.csv",
-        "views/estate_property_views.xml",
         "views/property_type_views.xml",
         "views/property_tag_views.xml",
+        "views/estate_property_views.xml",
         "views/estate_menus_views.xml",
-        "views/estate_list_views.xml",
-        "views/estate_form_views.xml",
-        "views/estate_search_views.xml",
     ],
     'license': 'LGPL-3',
 }
diff --git a/estate/views/estate_form_views.xml b/estate/views/estate_form_views.xml
deleted file mode 100644
index ee6906ba221..00000000000
--- a/estate/views/estate_form_views.xml
+++ /dev/null
@@ -1,66 +0,0 @@
-<?xml version="1.0"?>
-<odoo>
-    <record id="estate_form_view" model="ir.ui.view">
-        <field name="name">estate.property.form</field>
-        <field name="model">estate.property</field>
-        <field name="arch" type="xml">
-            <form string="Properties">
-                <sheet>
-                    <group>
-                        <h1 class="mb32">
-                            <field name="name" class="mb16"/>
-                        </h1>
-                    </group>
-
-                    <group>
-                        <group>
-                            <field name="tag_ids" widget="many2many_tags"/>
-                            <field name="postcode"/>
-                            <field name="available_from"/>
-                            <field name="property_type_id" string="Property Type"/>
-                        </group>
-
-                        <group>
-                            <field name="expected_price"/>
-                            <field name="selling_price"/>
-                            <field name="state"/>
-                        </group>
-                    </group>
-
-                    <notebook>
-                        <page string="Description">
-                            <group>
-                                <field name="description"/>
-                                <field name="bedrooms"/>
-                                <field name="living_area"/>
-                                <field name="facades"/>
-                                <field name="garage"/>
-                                <field name="garden"/>
-                                <field name="garden_area"/>
-                                <field name="garden_orientation"/>
-                            </group>
-                        </page>
-
-                        <page string="Offers">
-                            <field name="offer_ids">
-                                <list>
-                                    <field name="price"/>
-                                    <field name="partner_id"/>
-                                    <field name="status"/>
-                                </list>
-                            </field>
-                        </page>
-
-                        <page string="Other Info">
-                            <group>
-                                <field name="buyer"/>
-                                <field name="salesman"/>
-                            </group>
-                        </page>
-                    </notebook>
-
-                </sheet>
-            </form>
-        </field>
-    </record>
-</odoo>
diff --git a/estate/views/estate_list_views.xml b/estate/views/estate_list_views.xml
deleted file mode 100644
index 34a9d57d9b7..00000000000
--- a/estate/views/estate_list_views.xml
+++ /dev/null
@@ -1,19 +0,0 @@
-<?xml version="1.0"?>
-<odoo>
-    <record id="estate_list_view" model="ir.ui.view">
-        <field name="name">estate.property.list</field>
-        <field name="model">estate.property</field>
-        <field name="arch" type="xml">
-            <list string="Properties">
-                <field name="name"/>
-                <field name="postcode"/>
-                <field name="property_type_id"/>
-                <field name="bedrooms"/>
-                <field name="living_area"/>
-                <field name="expected_price"/>
-                <field name="selling_price"/>
-                <field name="available_from"/>
-            </list>
-        </field>
-    </record>
-</odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index c946bb3eeea..993ccc8178a 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -5,4 +5,105 @@
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
     </record>
+
+    <record id="estate_list_view" model="ir.ui.view">
+        <field name="name">estate.property.list</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <list string="Properties">
+                <field name="name"/>
+                <field name="postcode"/>
+                <field name="property_type_id"/>
+                <field name="bedrooms"/>
+                <field name="living_area"/>
+                <field name="expected_price"/>
+                <field name="selling_price"/>
+                <field name="available_from"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_form_view" model="ir.ui.view">
+        <field name="name">estate.property.form</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <form string="Properties">
+                <sheet>
+                    <group>
+                        <h1 class="mb32">
+                            <field name="name" class="mb16"/>
+                        </h1>
+                    </group>
+
+                    <group>
+                        <group>
+                            <field name="tag_ids" widget="many2many_tags"/>
+                            <field name="postcode"/>
+                            <field name="available_from"/>
+                            <field name="property_type_id" string="Property Type"/>
+                        </group>
+
+                        <group>
+                            <field name="expected_price"/>
+                            <field name="selling_price"/>
+                            <field name="state"/>
+                        </group>
+                    </group>
+
+                    <notebook>
+                        <page string="Description">
+                            <group>
+                                <field name="description"/>
+                                <field name="bedrooms"/>
+                                <field name="living_area"/>
+                                <field name="facades"/>
+                                <field name="garage"/>
+                                <field name="garden"/>
+                                <field name="garden_area"/>
+                                <field name="garden_orientation"/>
+                            </group>
+                        </page>
+
+                        <page string="Offers">
+                            <field name="offer_ids">
+                                <list>
+                                    <field name="price"/>
+                                    <field name="partner_id"/>
+                                    <field name="status"/>
+                                </list>
+                            </field>
+                        </page>
+
+                        <page string="Other Info">
+                            <group>
+                                <field name="buyer"/>
+                                <field name="salesman"/>
+                            </group>
+                        </page>
+                    </notebook>
+
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+    <record id="estate_search_view" model="ir.ui.view">
+        <field name="name">estate.property.search</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <search string="Estate Property">
+                <field name="name"/>
+                <field name="postcode"/>
+                <field name="bedrooms"/>
+                <field name="living_area"/>
+                <field name="expected_price"/>
+                <field name="facades"/>
+                <field name="property_type_id"/>
+
+                <filter string="Available" name="state" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer received')]"/>
+
+                <filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>
+            </search>
+        </field>
+    </record>
 </odoo>
diff --git a/estate/views/estate_search_views.xml b/estate/views/estate_search_views.xml
deleted file mode 100644
index 778efe89f0c..00000000000
--- a/estate/views/estate_search_views.xml
+++ /dev/null
@@ -1,22 +0,0 @@
-<?xml version="1.0"?>
-<odoo>
-    <record id="estate_search_view" model="ir.ui.view">
-        <field name="name">estate.property.search</field>
-        <field name="model">estate.property</field>
-        <field name="arch" type="xml">
-            <search string="Estate Property">
-                <field name="name"/>
-                <field name="postcode"/>
-                <field name="bedrooms"/>
-                <field name="living_area"/>
-                <field name="expected_price"/>
-                <field name="facades"/>
-                <field name="property_type_id"/>
-
-                <filter string="Available" name="state" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer received')]"/>
-
-                <filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>
-            </search>
-        </field>
-    </record>
-</odoo>

From f962d4b210966d602f7ec20f0b49d75eaa8cfc20 Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Thu, 20 Nov 2025 09:47:09 +0100
Subject: [PATCH 09/15] [ADD] estate: compute total_area, best_price and
 validity; set default garden values

---
 estate/models/estate_property.py       | 28 ++++++++++++++++++++++++--
 estate/models/estate_property_offer.py | 14 ++++++++++++-
 estate/views/estate_property_views.xml |  4 ++++
 3 files changed, 43 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 75e5c9a3368..3b708a33823 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,4 @@
-from odoo import models, fields
+from odoo import models, fields, api
 
 
 class EstateProperty(models.Model):
@@ -50,4 +50,28 @@ class EstateProperty(models.Model):
         comodel_name="res.users", default=lambda self: self.env.uid
     )
     tag_ids = fields.Many2many(comodel_name="estate.property.tag", string="Tags")
-    offer_ids = fields.One2many(comodel_name="estate.property.offer", inverse_name="property_id")
+    offer_ids = fields.One2many(
+        comodel_name="estate.property.offer", inverse_name="property_id"
+    )
+    total_area = fields.Integer(compute="_compute_area", string="Total Area (sqm)")
+    best_price = fields.Float(compute="_compute_best_price")
+
+    @api.depends("living_area", "garden_area")
+    def _compute_area(self):
+        for record in self:
+            record.total_area = record.living_area + record.garden_area
+
+    @api.depends("offer_ids.price")
+    def _compute_best_price(self):
+        for record in self:
+            if record.best_price:
+                record.best_price = max(record.offer_ids.mapped('price'))
+
+    @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 = ''
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 6e17fc2d869..a5980989918 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,4 +1,4 @@
-from odoo import models, fields
+from odoo import models, fields, api
 
 
 class EstatePropertyOffer(models.Model):
@@ -15,3 +15,15 @@ class EstatePropertyOffer(models.Model):
     )
     partner_id = fields.Many2one("res.partner", required=True, string="Partner")
     property_id = fields.Many2one("estate.property", required=True, string="Property")
+    validity = fields.Integer(default=7)
+    deadline_date = fields.Date(compute="_compute_deadline_date", inverse="_inverse_deadline_date")
+
+    @api.depends("create_date", "validity")
+    def _compute_deadline_date(self):
+        for record in self:
+            record.deadline_date = fields.Date.add((record.create_date or fields.Date.today()), days=record.validity)
+    
+    def _inverse_deadline_date(self):
+        for record in self:
+            record.validity = (record.deadline_date - fields.Date.today()).days
+    
\ No newline at end of file
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 993ccc8178a..4f04ee1d670 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -45,6 +45,7 @@
 
                         <group>
                             <field name="expected_price"/>
+                            <field name="best_price"/>
                             <field name="selling_price"/>
                             <field name="state"/>
                         </group>
@@ -61,6 +62,7 @@
                                 <field name="garden"/>
                                 <field name="garden_area"/>
                                 <field name="garden_orientation"/>
+                                <field name="total_area"/>
                             </group>
                         </page>
 
@@ -70,6 +72,8 @@
                                     <field name="price"/>
                                     <field name="partner_id"/>
                                     <field name="status"/>
+                                    <field name="validity"/>
+                                    <field name="deadline_date"/>
                                 </list>
                             </field>
                         </page>

From 3ffa891d53f9021427a303a8de73e5441eb17db9 Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Thu, 20 Nov 2025 11:48:34 +0100
Subject: [PATCH 10/15] [ADD] estate: add sold/cancel buttons on property; add
 accept/refuse buttons on offers

---
 estate/models/estate_property.py       | 21 ++++++++++++--
 estate/models/estate_property_offer.py | 39 ++++++++++++++++++++++----
 estate/views/estate_property_views.xml | 12 ++++++--
 3 files changed, 60 insertions(+), 12 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 3b708a33823..6754ab3da84 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,4 @@
-from odoo import models, fields, api
+from odoo import models, fields, api, exceptions
 
 
 class EstateProperty(models.Model):
@@ -64,8 +64,7 @@ def _compute_area(self):
     @api.depends("offer_ids.price")
     def _compute_best_price(self):
         for record in self:
-            if record.best_price:
-                record.best_price = max(record.offer_ids.mapped('price'))
+            record.best_price = max(record.offer_ids.mapped('price'), default=0.0)
 
     @api.onchange("garden")
     def _onchange_garden(self):
@@ -75,3 +74,19 @@ def _onchange_garden(self):
         else:
             self.garden_area = 0
             self.garden_orientation = ''
+
+    def sell_property(self):
+        for record in self:
+            if record.state == "cancelled":
+                raise exceptions.UserError("You cannot sell a cancelled property")
+            else:
+                record.state = "sold"
+        return True
+
+    def cancel_property(self):
+        for record in self:
+            if record.state == "sold":
+                raise exceptions.UserError("You cannot cancel a sold property")
+            else:
+                record.state = "cancelled"
+        return True
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index a5980989918..05867c7021a 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,4 +1,4 @@
-from odoo import models, fields, api
+from odoo import models, fields, api, exceptions
 
 
 class EstatePropertyOffer(models.Model):
@@ -16,14 +16,41 @@ class EstatePropertyOffer(models.Model):
     partner_id = fields.Many2one("res.partner", required=True, string="Partner")
     property_id = fields.Many2one("estate.property", required=True, string="Property")
     validity = fields.Integer(default=7)
-    deadline_date = fields.Date(compute="_compute_deadline_date", inverse="_inverse_deadline_date")
+    deadline_date = fields.Date(
+        compute="_compute_deadline_date", inverse="_inverse_deadline_date"
+    )
 
     @api.depends("create_date", "validity")
     def _compute_deadline_date(self):
         for record in self:
-            record.deadline_date = fields.Date.add((record.create_date or fields.Date.today()), days=record.validity)
-    
+            create_date = record.create_date or fields.Date.today()
+            record.deadline_date = fields.Date.add(create_date, days=record.validity)
+
     def _inverse_deadline_date(self):
         for record in self:
-            record.validity = (record.deadline_date - fields.Date.today()).days
-    
\ No newline at end of file
+            create_date = record.create_date or fields.Date.today()
+            record.validity = (record.deadline_date - (create_date.date())).days
+
+    def accept_offer(self):
+        for offer in self:
+            if offer.property_id.state == "sold":
+                raise exceptions.UserError(
+                    f"Property '{offer.property_id.name}' is already sold"
+                )
+            offer.property_id.write(
+                {
+                    "buyer": offer.partner_id,
+                    "selling_price": offer.price,
+                }
+            )
+            offer.status = "accepted"
+        return True
+
+    def refuse_offer(self):
+        for offer in self:
+            if offer.property_id.state == "sold" and offer.status == "accepted":
+                raise exceptions.UserError(
+                    f"Property '{offer.property_id.name}' is already sold to '{offer.partner_id.name}'"
+                )
+            offer.status = "refused"
+        return True
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 4f04ee1d670..721c65388e3 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -28,6 +28,10 @@
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form string="Properties">
+            <header>
+                <button name="sell_property" type="object" string="SOLD"/>
+                <button name="cancel_property" type="object" string="CANCEL"/>
+            </header>
                 <sheet>
                     <group>
                         <h1 class="mb32">
@@ -38,16 +42,16 @@
                     <group>
                         <group>
                             <field name="tag_ids" widget="many2many_tags"/>
+                            <field name="state"/>
                             <field name="postcode"/>
-                            <field name="available_from"/>
                             <field name="property_type_id" string="Property Type"/>
                         </group>
 
                         <group>
+                            <field name="available_from"/>
                             <field name="expected_price"/>
                             <field name="best_price"/>
                             <field name="selling_price"/>
-                            <field name="state"/>
                         </group>
                     </group>
 
@@ -71,9 +75,11 @@
                                 <list>
                                     <field name="price"/>
                                     <field name="partner_id"/>
-                                    <field name="status"/>
                                     <field name="validity"/>
                                     <field name="deadline_date"/>
+                                    <button name="accept_offer" string="Accept" type="object" icon="fa-check"/>
+                                    <button name="refuse_offer" string="Refuse" type="object" icon="fa-times"/>
+                                    <field name="status"/>
                                 </list>
                             </field>
                         </page>

From 39831c5879e29f8a788062cd7d14dfce04ac8ff4 Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Thu, 20 Nov 2025 14:34:13 +0100
Subject: [PATCH 11/15] [ADD] estate: add constraints on prices, and unique
 types and tags

---
 .gitignore                             |  2 ++
 estate/models/estate_property.py       | 17 +++++++++++++++++
 estate/models/estate_property_offer.py |  9 ++++++++-
 estate/models/estate_property_tag.py   |  6 ++++++
 estate/models/estate_property_type.py  |  5 +++++
 5 files changed, 38 insertions(+), 1 deletion(-)

diff --git a/.gitignore b/.gitignore
index b6e47617de1..eded9e6c769 100644
--- a/.gitignore
+++ b/.gitignore
@@ -127,3 +127,5 @@ dmypy.json
 
 # Pyre type checker
 .pyre/
+
+.vscode/*
\ No newline at end of file
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 6754ab3da84..10143b00546 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,5 @@
 from odoo import models, fields, api, exceptions
+from odoo.tools import float_compare
 
 
 class EstateProperty(models.Model):
@@ -56,6 +57,16 @@ class EstateProperty(models.Model):
     total_area = fields.Integer(compute="_compute_area", string="Total Area (sqm)")
     best_price = fields.Float(compute="_compute_best_price")
 
+    _postive_expected_price = models.Constraint(
+        "CHECK (expected_price > 0)",
+        "The expected price must be strictly positive"
+    )
+
+    _postive_selling_price = models.Constraint(
+        "CHECK (selling_price > 0)",
+        "The selling price must be strictly positive"
+    )
+
     @api.depends("living_area", "garden_area")
     def _compute_area(self):
         for record in self:
@@ -90,3 +101,9 @@ def cancel_property(self):
             else:
                 record.state = "cancelled"
         return True
+    
+    @api.constrains("selling_price")
+    def _check_selling_price(self):
+        for record in self:
+            if float_compare(record.selling_price, 0.9 * record.expected_price, 2) == -1:
+                raise exceptions.ValidationError("The selling price must be at least 90% of the expected price.")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 05867c7021a..624c9687d04 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -14,12 +14,19 @@ class EstatePropertyOffer(models.Model):
         ],
     )
     partner_id = fields.Many2one("res.partner", required=True, string="Partner")
-    property_id = fields.Many2one("estate.property", required=True, string="Property")
+    property_id = fields.Many2one(
+        "estate.property", required=True, string="Property", ondelete="cascade"
+    )
     validity = fields.Integer(default=7)
     deadline_date = fields.Date(
         compute="_compute_deadline_date", inverse="_inverse_deadline_date"
     )
 
+    _postive_price = models.Constraint(
+        "CHECK (price > 0)",
+        "The expected price must be strictly positive"
+    )
+
     @api.depends("create_date", "validity")
     def _compute_deadline_date(self):
         for record in self:
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 6f4569f5954..3c46662b91b 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -6,3 +6,9 @@ class EstatePropertyTag(models.Model):
     _description = "property tags"
 
     name = fields.Char(required=True)
+
+    _unique_tag = models.Constraint(
+        "unique(name)",
+        "This tag already exists"
+    )
+    
\ No newline at end of file
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 2e07a19bf05..289f4335a94 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -6,3 +6,8 @@ class EstatePropertyType(models.Model):
     _description = "property types"
 
     name = fields.Char(required=True)
+
+    _unique_type = models.Constraint(
+        "unique(name)",
+        "This type already exists"
+    )

From 53c7607c7ae9467a4192a9bc09ed82bd00d7da56 Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Mon, 24 Nov 2025 10:46:35 +0100
Subject: [PATCH 12/15] [ADD] estate: added the sprinkles

---
 estate/__manifest__.py                 |  1 +
 estate/models/__init__.py              | 10 ++--
 estate/models/estate_property.py       | 41 +++++++++--------
 estate/models/estate_property_offer.py | 17 +++----
 estate/models/estate_property_tag.py   |  3 +-
 estate/models/estate_property_type.py  | 18 ++++++--
 estate/views/estate_property_views.xml | 63 +++++++++++++++-----------
 estate/views/property_offer_views.xml  | 22 +++++++++
 estate/views/property_type_views.xml   | 48 ++++++++++++++++++++
 9 files changed, 160 insertions(+), 63 deletions(-)
 create mode 100644 estate/views/property_offer_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 17c906b7f14..f855c7f9635 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -18,6 +18,7 @@
         "security/ir.model.access.csv",
         "views/property_type_views.xml",
         "views/property_tag_views.xml",
+        "views/property_offer_views.xml",
         "views/estate_property_views.xml",
         "views/estate_menus_views.xml",
     ],
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index a0a8b8c501c..09b2099fe84 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1,6 +1,4 @@
-from . import (
-    estate_property,
-    estate_property_type,
-    estate_property_tag,
-    estate_property_offer,
-)
+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
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 10143b00546..c1e8b09c528 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -5,6 +5,7 @@
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Real Estate model"
+    _order = "id desc"
 
     name = fields.Char(required=True)
     description = fields.Char()
@@ -34,8 +35,8 @@ class EstateProperty(models.Model):
         string="State",
         selection=[
             ("new", "New"),
-            ("offer received", "Offer Received"),
-            ("offer accepted", "Offer Accepted"),
+            ("offer_received", "Offer Received"),
+            ("offer_accepted", "Offer Accepted"),
             ("sold", "Sold"),
             ("cancelled", "Cancelled"),
         ],
@@ -46,8 +47,8 @@ class EstateProperty(models.Model):
     property_type_id = fields.Many2one(
         comodel_name="estate.property.type", string="Property Type"
     )
-    buyer = fields.Many2one(comodel_name="res.partner", copy=False)
-    salesman = fields.Many2one(
+    buyer_id = fields.Many2one(comodel_name="res.partner", copy=False)
+    salesman_id = fields.Many2one(
         comodel_name="res.users", default=lambda self: self.env.uid
     )
     tag_ids = fields.Many2many(comodel_name="estate.property.tag", string="Tags")
@@ -69,13 +70,18 @@ class EstateProperty(models.Model):
 
     @api.depends("living_area", "garden_area")
     def _compute_area(self):
-        for record in self:
-            record.total_area = record.living_area + record.garden_area
+        for property in self:
+            property.total_area = property.living_area + property.garden_area
 
     @api.depends("offer_ids.price")
     def _compute_best_price(self):
-        for record in self:
-            record.best_price = max(record.offer_ids.mapped('price'), default=0.0)
+        for property in self:
+            property.best_price = max(property.offer_ids.mapped('price'), default=0.0)
+
+    @api.onchange("offer_ids")
+    def _onchange_offer_receieved(self):
+        if len(self.offer_ids):
+            self.state = "offer_received"
 
     @api.onchange("garden")
     def _onchange_garden(self):
@@ -87,23 +93,22 @@ def _onchange_garden(self):
             self.garden_orientation = ''
 
     def sell_property(self):
-        for record in self:
-            if record.state == "cancelled":
+        for property in self:
+            if property.state == "cancelled":
                 raise exceptions.UserError("You cannot sell a cancelled property")
-            else:
-                record.state = "sold"
+            property.state = "sold"
         return True
 
     def cancel_property(self):
-        for record in self:
-            if record.state == "sold":
+        for property in self:
+            if property.state == "sold":
                 raise exceptions.UserError("You cannot cancel a sold property")
             else:
-                record.state = "cancelled"
+                property.state = "cancelled"
         return True
-    
+ 
     @api.constrains("selling_price")
     def _check_selling_price(self):
-        for record in self:
-            if float_compare(record.selling_price, 0.9 * record.expected_price, 2) == -1:
+        for property in self:
+            if float_compare(property.selling_price, 0.9 * property.expected_price, 2) == -1:
                 raise exceptions.ValidationError("The selling price must be at least 90% of the expected price.")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 624c9687d04..4e213a81b9d 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 = "property offer"
+    _order = "price desc"
 
     price = fields.Float()
     status = fields.Selection(
@@ -21,6 +22,7 @@ class EstatePropertyOffer(models.Model):
     deadline_date = fields.Date(
         compute="_compute_deadline_date", inverse="_inverse_deadline_date"
     )
+    property_type_id = fields.Many2one(related="property_id.property_type_id", store=True)
 
     _postive_price = models.Constraint(
         "CHECK (price > 0)",
@@ -36,24 +38,23 @@ def _compute_deadline_date(self):
     def _inverse_deadline_date(self):
         for record in self:
             create_date = record.create_date or fields.Date.today()
-            record.validity = (record.deadline_date - (create_date.date())).days
+            record.validity = (record.deadline_date - create_date.date()).days
 
-    def accept_offer(self):
+    def button_accept_offer(self):
         for offer in self:
             if offer.property_id.state == "sold":
                 raise exceptions.UserError(
                     f"Property '{offer.property_id.name}' is already sold"
                 )
-            offer.property_id.write(
-                {
-                    "buyer": offer.partner_id,
+            offer.property_id.write({
+                    "buyer_id": offer.partner_id,
                     "selling_price": offer.price,
-                }
-            )
+                    "state": "offer_accepted"
+                })
             offer.status = "accepted"
         return True
 
-    def refuse_offer(self):
+    def button_refuse_offer(self):
         for offer in self:
             if offer.property_id.state == "sold" and offer.status == "accepted":
                 raise exceptions.UserError(
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 3c46662b91b..502ace7b04b 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -4,11 +4,12 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "property tags"
+    _order = "name asc"
 
     name = fields.Char(required=True)
+    color = fields.Integer()
 
     _unique_tag = models.Constraint(
         "unique(name)",
         "This tag already exists"
     )
-    
\ No newline at end of file
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 289f4335a94..ef9d173ef7b 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -1,13 +1,23 @@
-from odoo import models, fields
+from odoo import models, fields, api
 
 
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "property types"
+    _order = "sequence, name asc"
 
     name = fields.Char(required=True)
+    sequence = fields.Integer('Sequence', default=1)
 
-    _unique_type = models.Constraint(
-        "unique(name)",
-        "This type already exists"
+    property_ids = fields.One2many(
+        comodel_name="estate.property", inverse_name="property_type_id"
     )
+    offer_ids = fields.One2many(comodel_name="estate.property.offer", inverse_name="property_type_id")
+    offer_count = fields.Integer(compute="_compute_offer_count", default=0, string="Offers")
+
+    _unique_type = models.Constraint("unique(name)", "This type already exists")
+
+    @api.depends("offer_ids")
+    def _compute_offer_count(self):
+        for type in self:
+            type.offer_count = len(type.offer_ids)
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 721c65388e3..b919a2e8455 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -1,16 +1,13 @@
 <?xml version="1.0"?>
 <odoo>
-    <record id="estate_property_view" model="ir.actions.act_window">
-        <field name="name">Estate Property</field>
-        <field name="res_model">estate.property</field>
-        <field name="view_mode">list,form</field>
-    </record>
-
     <record id="estate_list_view" model="ir.ui.view">
         <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-bf="state=='offer_accepted'" 
+                decoration-success="state in ['offer_received', 'offer_accepted']"
+                decoration-muted="state=='sold'">
                 <field name="name"/>
                 <field name="postcode"/>
                 <field name="property_type_id"/>
@@ -18,7 +15,7 @@
                 <field name="living_area"/>
                 <field name="expected_price"/>
                 <field name="selling_price"/>
-                <field name="available_from"/>
+                <field name="available_from" optional="hide"/>
             </list>
         </field>
     </record>
@@ -29,8 +26,9 @@
         <field name="arch" type="xml">
             <form string="Properties">
             <header>
-                <button name="sell_property" type="object" string="SOLD"/>
-                <button name="cancel_property" type="object" string="CANCEL"/>
+                <button name="sell_property" type="object" string="SOLD" invisible="state == 'sold' or state == 'cancelled'"/>
+                <button name="cancel_property" 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>
                     <group>
@@ -41,10 +39,9 @@
 
                     <group>
                         <group>
-                            <field name="tag_ids" widget="many2many_tags"/>
-                            <field name="state"/>
+                            <field name="tag_ids" widget="many2many_tags" options="{'color_field': 'color'}"/>
                             <field name="postcode"/>
-                            <field name="property_type_id" string="Property Type"/>
+                            <field name="property_type_id" string="Property Type" options="{'no_create':True}"/>
                         </group>
 
                         <group>
@@ -64,30 +61,38 @@
                                 <field name="facades"/>
                                 <field name="garage"/>
                                 <field name="garden"/>
-                                <field name="garden_area"/>
-                                <field name="garden_orientation"/>
+                                <field name="garden_area" invisible="not garden"/>
+                                <field name="garden_orientation" invisible="not garden"/>
                                 <field name="total_area"/>
                             </group>
                         </page>
 
                         <page string="Offers">
-                            <field name="offer_ids">
-                                <list>
+                            <field name="offer_ids" readonly="state in ['offer_accepted', 'sold', 'cancelled']">
+                                <list editable="bottom" 
+                                    decoration-success="status=='accepted'" 
+                                    decoration-danger="status=='refused'">
                                     <field name="price"/>
                                     <field name="partner_id"/>
                                     <field name="validity"/>
                                     <field name="deadline_date"/>
-                                    <button name="accept_offer" string="Accept" type="object" icon="fa-check"/>
-                                    <button name="refuse_offer" string="Refuse" type="object" icon="fa-times"/>
-                                    <field name="status"/>
+                                    <button name="button_accept_offer"
+                                        string="Accept"
+                                        type="object"
+                                        icon="fa-check"
+                                        invisible="status in ['accepted', 'refused']"/>
+                                    <button name="button_refuse_offer"
+                                        string="Refuse" type="object"
+                                        icon="fa-times"
+                                        invisible="status in ['accepted', 'refused']"/>
                                 </list>
                             </field>
                         </page>
 
                         <page string="Other Info">
                             <group>
-                                <field name="buyer"/>
-                                <field name="salesman"/>
+                                <field name="buyer_id"/>
+                                <field name="salesman_id"/>
                             </group>
                         </page>
                     </notebook>
@@ -105,15 +110,21 @@
                 <field name="name"/>
                 <field name="postcode"/>
                 <field name="bedrooms"/>
-                <field name="living_area"/>
+                <field name="living_area" filter_domain="[('living_area', '>=', self)]"/>
                 <field name="expected_price"/>
                 <field name="facades"/>
                 <field name="property_type_id"/>
 
-                <filter string="Available" name="state" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer received')]"/>
-
-                <filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>
+                <filter name="available" string="Available" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer received')]"/>
+                <filter name="postcode" string="Postcode" context="{'group_by':'postcode'}"/>
             </search>
         </field>
     </record>
+
+    <record id="estate_property_view" model="ir.actions.act_window">
+        <field name="name">Estate Property</field>
+        <field name="res_model">estate.property</field>
+        <field name="view_mode">list,form</field>
+        <field name="context">{'search_default_available': True}</field>
+    </record>
 </odoo>
diff --git a/estate/views/property_offer_views.xml b/estate/views/property_offer_views.xml
new file mode 100644
index 00000000000..ea63a0c1942
--- /dev/null
+++ b/estate/views/property_offer_views.xml
@@ -0,0 +1,22 @@
+<?xml version="1.0"?>
+<odoo>    
+    <record id="property_offer_view" model="ir.actions.act_window">
+        <field name="name">Property Offer</field>
+        <field name="res_model">estate.property.offer</field>
+        <field name="view_mode">list,form</field>
+        <field name="domain">[('property_type_id', '=', active_id)]</field>
+    </record>
+
+        <record id="estate_offer_list_view" model="ir.ui.view">
+        <field name="name">estate.property.offer</field>
+        <field name="model">estate.property.offer</field>
+        <field name="arch" type="xml">
+            <list string="Offers">
+                <field name="price"/>
+                <field name="partner_id"/>
+                <field name="validity"/>
+                <field name="deadline_date"/>
+            </list>
+        </field>
+    </record>
+</odoo>
diff --git a/estate/views/property_type_views.xml b/estate/views/property_type_views.xml
index ecc59967464..f45debb4f22 100644
--- a/estate/views/property_type_views.xml
+++ b/estate/views/property_type_views.xml
@@ -5,4 +5,52 @@
         <field name="res_model">estate.property.type</field>
         <field name="view_mode">list,form</field>
     </record>
+
+    <record id="estate_type_list_view" model="ir.ui.view">
+        <field name="name">estate.property.list</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <list string="Property Types">
+                <field name="sequence" widget="handle"/>
+                <field name="name"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_type_form_view" model="ir.ui.view">
+        <field name="name">estate.property.type</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <form string="Types">
+                <group>
+                    <h1 class="mb32">
+                        <field name="name" class="mb16"/>
+                    </h1>
+                    <button class="oe_stat_button" type="action"
+                                name="%(estate.property_offer_view)d"
+                                icon="fa-money">
+                            <div class="o_stat_info">
+                                <span class="o_stat_value">
+                                    <field name="offer_count"/>
+                                </span>
+                                <span class="o_stat_text">Offers</span>
+                            </div>
+                    </button>
+                </group>
+                <sheet>
+                    <notebook>
+                        <page string="Properties">
+                            <field name="property_ids">
+                                <list>
+                                    <field name="name"/>
+                                    <field name="expected_price"/>
+                                    <field name="state"/>
+                                </list>
+                            </field>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
 </odoo>

From a68f98c4fec420c91f88242c89a0c3095bd83035 Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Mon, 24 Nov 2025 15:13:03 +0100
Subject: [PATCH 13/15] [ADD] estate: add properties to user, and properites in
 user profile

---
 estate/__manifest__.py                 |  1 +
 estate/models/__init__.py              |  3 ++-
 estate/models/estate_property.py       |  6 +++++
 estate/models/estate_property_offer.py |  9 +++++++
 estate/models/res_users.py             |  7 +++++
 estate/security/ir.model.access.csv    |  1 +
 estate/views/estate_menus_views.xml    |  1 +
 estate/views/res_users_views.xml       | 36 ++++++++++++++++++++++++++
 8 files changed, 63 insertions(+), 1 deletion(-)
 create mode 100644 estate/models/res_users.py
 create mode 100644 estate/views/res_users_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index f855c7f9635..e4efa83f70b 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -20,6 +20,7 @@
         "views/property_tag_views.xml",
         "views/property_offer_views.xml",
         "views/estate_property_views.xml",
+        "views/res_users_views.xml",
         "views/estate_menus_views.xml",
     ],
     'license': 'LGPL-3',
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 09b2099fe84..9a2189b6382 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1,4 +1,5 @@
 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
+from . import res_users
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index c1e8b09c528..de46d1861cc 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -112,3 +112,9 @@ def _check_selling_price(self):
         for property in self:
             if float_compare(property.selling_price, 0.9 * property.expected_price, 2) == -1:
                 raise exceptions.ValidationError("The selling price must be at least 90% of the expected price.")
+            
+    @api.ondelete(at_uninstall=False)
+    def _unlink_property_with_offer(self):
+        for property in self:
+            if property.state not in ["new", "cancelled"]:
+                raise exceptions.UserError("You cannot delete a property with existing offers!")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 4e213a81b9d..a7ea9d5e10c 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -62,3 +62,12 @@ def button_refuse_offer(self):
                 )
             offer.status = "refused"
         return True
+    
+    @api.model_create_multi
+    def create(self, vals_list):
+        for val in vals_list:
+            property_id = self.env['estate.property'].browse(val['property_id'])
+            if property_id.offer_ids and max(offer.price for offer in property_id.offer_ids) > val['price']:
+                raise exceptions.UserError("An offer with a higher price exists")
+            property_id.state = "offer_received"
+        return super().create(vals_list)
diff --git a/estate/models/res_users.py b/estate/models/res_users.py
new file mode 100644
index 00000000000..61e580dc963
--- /dev/null
+++ b/estate/models/res_users.py
@@ -0,0 +1,7 @@
+from odoo import models, fields
+
+
+class Users(models.Model):
+    _inherit="res.users"
+    property_ids = fields.One2many(comodel_name="estate.property", inverse_name="salesman_id",
+        string="Properties")
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index e20ec4dd90b..be6afd74af1 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -3,3 +3,4 @@ access_estate_property,access.estate.property,model_estate_property,base.group_u
 access_estate_property_type,access.estate.property.type,model_estate_property_type,base.group_user,1,1,1,1
 access_estate_property_tag,access.estate.property.tag,model_estate_property_tag,base.group_user,1,1,1,1
 access_estate_property_offer,access.estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1
+access_res_users,access.res.users,model_res_users,base.group_user,1,1,1,1
diff --git a/estate/views/estate_menus_views.xml b/estate/views/estate_menus_views.xml
index 1721ca49a12..412a4657d0b 100644
--- a/estate/views/estate_menus_views.xml
+++ b/estate/views/estate_menus_views.xml
@@ -8,6 +8,7 @@
     <menuitem id="settings_menu" name="Settings">
         <menuitem id="property_type_action" action="property_type_view"/>
         <menuitem id="property_tag_action" action="property_tag_view"/>
+        <menuitem id="users_and_companies" action="user_and_companies_view"/>
     </menuitem>
 </menuitem>
 </odoo>
diff --git a/estate/views/res_users_views.xml b/estate/views/res_users_views.xml
new file mode 100644
index 00000000000..ea14ea580f4
--- /dev/null
+++ b/estate/views/res_users_views.xml
@@ -0,0 +1,36 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+<data>
+    <record id="res_users_view_form" model="ir.ui.view">
+        <field name="name">res.users.view.form.inherit</field>
+        <field name="model">res.users</field>
+        <field name="inherit_id" ref="base.view_users_simple_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//group[@name='access_groups']" position="after">
+                <notebook>
+                    <page string="Real Estate Properties">
+                        <field name="property_ids">
+                            <list>
+                                <field name="name"/>
+                                <field name="property_type_id"/>
+                                <field name="postcode"/>
+                                <field name="tag_ids"/>
+                                <field name="bedrooms"/>
+                                <field name="living_area"/>
+                                <field name="expected_price"/>
+                                <field name="selling_price"/>
+                            </list>
+                        </field>
+                    </page>
+                </notebook>
+            </xpath>    
+        </field>
+    </record>
+
+    <record id="user_and_companies_view" model="ir.actions.act_window">
+        <field name="name"> Users &amp; Companies </field>
+        <field name="res_model">res.users</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</data>
+</odoo>

From bc04c280cd2c3105919a32a2609b01686ee8c04f Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Mon, 24 Nov 2025 17:12:16 +0100
Subject: [PATCH 14/15] [ADD] estate: create estate_account module

---
 estate/models/estate_property.py         | 29 ++++++++++++---------
 estate/models/estate_property_tag.py     |  5 +---
 estate_account/__init__.py               |  1 +
 estate_account/__manifest__.py           | 20 +++++++++++++++
 estate_account/models/__init__.py        |  1 +
 estate_account/models/estate_property.py | 32 ++++++++++++++++++++++++
 6 files changed, 72 insertions(+), 16 deletions(-)
 create mode 100644 estate_account/__init__.py
 create mode 100644 estate_account/__manifest__.py
 create mode 100644 estate_account/models/__init__.py
 create mode 100644 estate_account/models/estate_property.py

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index de46d1861cc..5b4d1145d77 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -59,13 +59,11 @@ class EstateProperty(models.Model):
     best_price = fields.Float(compute="_compute_best_price")
 
     _postive_expected_price = models.Constraint(
-        "CHECK (expected_price > 0)",
-        "The expected price must be strictly positive"
+        "CHECK (expected_price > 0)", "The expected price must be strictly positive"
     )
 
     _postive_selling_price = models.Constraint(
-        "CHECK (selling_price > 0)",
-        "The selling price must be strictly positive"
+        "CHECK (selling_price > 0)", "The selling price must be strictly positive"
     )
 
     @api.depends("living_area", "garden_area")
@@ -76,7 +74,7 @@ def _compute_area(self):
     @api.depends("offer_ids.price")
     def _compute_best_price(self):
         for property in self:
-            property.best_price = max(property.offer_ids.mapped('price'), default=0.0)
+            property.best_price = max(property.offer_ids.mapped("price"), default=0.0)
 
     @api.onchange("offer_ids")
     def _onchange_offer_receieved(self):
@@ -87,10 +85,10 @@ def _onchange_offer_receieved(self):
     def _onchange_garden(self):
         if self.garden:
             self.garden_area = 10
-            self.garden_orientation = 'north'
+            self.garden_orientation = "north"
         else:
             self.garden_area = 0
-            self.garden_orientation = ''
+            self.garden_orientation = ""
 
     def sell_property(self):
         for property in self:
@@ -106,15 +104,22 @@ def cancel_property(self):
             else:
                 property.state = "cancelled"
         return True
- 
+
     @api.constrains("selling_price")
     def _check_selling_price(self):
         for property in self:
-            if float_compare(property.selling_price, 0.9 * property.expected_price, 2) == -1:
-                raise exceptions.ValidationError("The selling price must be at least 90% of the expected price.")
-            
+            if (
+                float_compare(property.selling_price, 0.9 * property.expected_price, 2)
+                == -1
+            ):
+                raise exceptions.ValidationError(
+                    "The selling price must be at least 90% of the expected price."
+                )
+
     @api.ondelete(at_uninstall=False)
     def _unlink_property_with_offer(self):
         for property in self:
             if property.state not in ["new", "cancelled"]:
-                raise exceptions.UserError("You cannot delete a property with existing offers!")
+                raise exceptions.UserError(
+                    "You cannot delete a property with existing offers!"
+                )
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 502ace7b04b..5d62ae9ceb8 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -9,7 +9,4 @@ class EstatePropertyTag(models.Model):
     name = fields.Char(required=True)
     color = fields.Integer()
 
-    _unique_tag = models.Constraint(
-        "unique(name)",
-        "This tag already exists"
-    )
+    _unique_tag = models.Constraint("unique(name)", "This tag already exists")
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..8806a6ccce8
--- /dev/null
+++ b/estate_account/__manifest__.py
@@ -0,0 +1,20 @@
+{
+    'name': "Real Estate Account",
+
+    'summary': "Cool Real Estate For Invoicing  ",
+    'description': """
+Cool Real Estate App
+Wow This is a description omgg
+    """,
+    'author': "kmhma",
+    'website': "https://www.odoo.com/",
+    'category': 'Tutorials',
+    'version': '0.1',
+    'application': True,
+    'installable': True,
+    'depends': ['base', 'estate', 'account'],
+
+    'data': [
+    ],
+    'license': 'LGPL-3',
+}
diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py
new file mode 100644
index 00000000000..f4c8fd6db6d
--- /dev/null
+++ b/estate_account/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_property
\ No newline at end of file
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py
new file mode 100644
index 00000000000..1c6611b61e2
--- /dev/null
+++ b/estate_account/models/estate_property.py
@@ -0,0 +1,32 @@
+from odoo import models, Command
+
+
+class EstateProperty(models.Model):
+    _inherit = "estate.property"
+
+    def sell_property(self):
+        partner_id = self.buyer_id.id
+        move_type = "out_invoice"
+        self.env["account.move"].create(
+            {
+                "partner_id": partner_id,
+                "move_type": move_type,
+                "invoice_line_ids": [
+                    Command.create(
+                        {
+                            "name": "Selling Price",
+                            "quantity": 1,
+                            "price_unit": 0.06 * self.selling_price,
+                        }
+                    ),
+                    Command.create(
+                        {
+                            "name": "Administrative Fees",
+                            "quantity": 1,
+                            "price_unit": 100,
+                        }
+                    ),
+                ],
+            }
+        )
+        return super().sell_property()

From 0959dc371ca796c95e990013126a0ad8f54d240f Mon Sep 17 00:00:00 2001
From: KarimH537 <karimhassib537@gmail.com>
Date: Tue, 25 Nov 2025 09:36:35 +0100
Subject: [PATCH 15/15] [ADD] estate: add properties kanban view

---
 estate/models/estate_property_offer.py |  2 +-
 estate/models/res_users.py             |  2 +-
 estate/views/estate_property_views.xml | 27 +++++++++++++++++++++++++-
 estate_account/models/__init__.py      |  2 +-
 4 files changed, 29 insertions(+), 4 deletions(-)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index a7ea9d5e10c..b25f99ba7fa 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -62,7 +62,7 @@ def button_refuse_offer(self):
                 )
             offer.status = "refused"
         return True
-    
+
     @api.model_create_multi
     def create(self, vals_list):
         for val in vals_list:
diff --git a/estate/models/res_users.py b/estate/models/res_users.py
index 61e580dc963..9912be284b6 100644
--- a/estate/models/res_users.py
+++ b/estate/models/res_users.py
@@ -2,6 +2,6 @@
 
 
 class Users(models.Model):
-    _inherit="res.users"
+    _inherit = "res.users"
     property_ids = fields.One2many(comodel_name="estate.property", inverse_name="salesman_id",
         string="Properties")
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index b919a2e8455..79ce14c26a9 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -121,10 +121,35 @@
         </field>
     </record>
 
+    <record id="estate_kanban_view" 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="property_type_id" groups_draggable="0" records_draggable="0">
+                <field name="state"/>
+                <templates>
+                    <t t-name="card">
+                            <h4><field name="name"/></h4>
+                            <div>
+                                Expected Price: <field name="expected_price"/>
+                            </div>
+                            <div t-if="record.state.raw_value == 'offer_received'">
+                                Best Offer: <field name="best_price"/>
+                            </div>
+                            <div t-if="record.state.raw_value == 'offer_accepted'">
+                                Selling Price: <field name="selling_price"/>
+                            </div>
+                            <field name="tag_ids"/>
+                    </t>
+                </templates>
+            </kanban>            
+        </field>
+    </record>
+
     <record id="estate_property_view" model="ir.actions.act_window">
         <field name="name">Estate Property</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>
 </odoo>
diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py
index f4c8fd6db6d..5e1963c9d2f 100644
--- a/estate_account/models/__init__.py
+++ b/estate_account/models/__init__.py
@@ -1 +1 @@
-from . import estate_property
\ No newline at end of file
+from . import estate_property