[ADD] estate: create initial module structure and manifest by mawat-odoo · Pull Request #1416 · odoo/tutorials

mawat-odoo · 2026-09-15T11:23:15Z

Initialize the estate module with its basic directory structure and manifest file as required for the technical onboarding training.

task-6573564

robodoo · 2026-09-15T11:23:23Z

Initialize the estate module with its basic directory structure and manifest file as required for the technical onboarding training. task-6573564

msho-odoo

nice job on the commit message format
I left you a comment, feel free to ping me here when yo push for the next improvement in the chapters 🔥

msho-odoo · 2026-09-15T13:36:11Z

@@ -1,3 +1,9 @@
+# Created by https://www.toptal.com/developers/gitignore/api/python,odoo


ops, I shouldn't see that file :)
Environmental files (usually starts with .) shouldn't be pushed or seen here, you might have a quick search on how to avoid this :)

- Create models package and __init__.py files - Define estate.property model with basic fields: * name, description, postcode * date_availability, expected_price, selling_price * bedrooms, living_area, facades, garage * garden, garden_area, garden_orientation task-6573564

- Add security/ir.model.access.csv with full permissions for internal users - Declare security file in __manifest__.py task-6573564

…elds - Create estate_property_views.xml with act_window for estate.property model - Create estate_menus.xml with 3-level menu hierarchy - Register XML files sequentially in __manifest__.py - Set selling_price as readonly and non-copyable - Prevent copy on date_availability and set default to 3 months from today - Set default bedrooms to 2 - Add reserved active field (Boolean) with default=True - Add required state field (Selection: New, Offer Received, Offer Accepted, Sold, Cancelled) with default='new' and copy=False task-6573564

Task ID: 6573564

Add list, form, and search views for estate.property model. Search view includes custom filter for available properties and grouping by postcode. - Add list view for property model - Add form view with sheet, group, and notebook tags - Add search view with domain filter and group_by context Task ID: 6573564

mawat-odoo · 2026-09-17T07:14:01Z

@msho-odoo some stuffs to review when you have time:) thank you!

msho-odoo

Thank you for the good work (and the green runbot :))

Regarding the commit message, we only use [ADD] tag when adding a brand new module but adding fields, views, logic, etc is considered an improvement so we use [IMP] tag.

I also left you a couple of comments to address,
please ping me when you finish the next chapter 😉
Thanks!

msho-odoo · 2026-09-17T09:15:27Z

@@ -0,0 +1,34 @@
+from odoo.tools import date_utils
+


nitpick: you can delete that empty line

msho-odoo · 2026-09-17T09:18:23Z

+    name = fields.Char(required=True)
+    description = fields.Text()
+    postcode = fields.Char()
+    date_availability = fields.Date(copy=False, default=lambda self: date_utils.add(fields.Date.today(), months=3))


you can import the add directly from odoo.tools.date_utils and use it instead of importing the whole thing

msho-odoo · 2026-09-17T09:19:30Z

+    garage = fields.Boolean()
+    garden = fields.Boolean()


it's better to have boolean fields named is_something or has_something so may call these has_garden and has_garage and then you can give them UI names using the string attribute or just put a string as the first parameter in Boolean()

msho-odoo · 2026-09-17T09:23:43Z

+    garden_orientation = fields.Selection(
+        string='Type',
+        selection=[('North', 'North'), ('South', 'South'), ('East', 'East'), ('West', 'West')]
+        )


Having the selection list in one line is okay but when we have such multiple values we style it like this.
Also it's better to have the keys of the selection tuples to be all lower case letters to avoid confusion when used later in the code.

Suggested change

garden_orientation = fields.Selection(

string='Type',

selection=[('North', 'North'), ('South', 'South'), ('East', 'East'), ('West', 'West')]

)

garden_orientation = fields.Selection(

string='Garden Orientation',

selection=[

('north', 'North'),

('south', 'South'),

('east', 'East'),

('west', 'West')

]

)

msho-odoo · 2026-09-17T09:24:18Z

+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <menuitem id="estate_menu_root" name="Real Estate">
+        <menuitem id="estatre_first_level_menu" name="Advertisements">


ids better be descriptive so other coders know what it mostly does by name only

msho-odoo · 2026-09-17T09:25:50Z

+                <filter name="state" string="Available Properties" domain="['|',
+                                                                ('state', '=', 'new'),
+                                                                ('state', '=', 'offer_received')]"/>


correct but this is more readable :)

Suggested change

<filter name="state" string="Available Properties" domain="['|',

('state', '=', 'new'),

('state', '=', 'offer_received')]"/>

<filter name="state" string="Available Properties" domain="[('state', 'in', ('new', 'offer_received'))]"/>

msho-odoo · 2026-09-17T09:27:11Z

+                            <group>
+                                <field name="description"/>
+                                <field name="bedrooms"/>
+                                <field name="living_area" string="Living Area (sqm)"/>


You can add the string attribute to the field definition in the python model, that way you wouldn't need to repeat it every time you use the field in a view (unless you want it different in a certain view)

msho-odoo · 2026-09-17T09:27:49Z

@@ -0,0 +1,13 @@
+# __manifest__.py
+{  # noqa: B018
+    "author": "mawat",


author is usually Odoo S.A. :)

msho-odoo · 2026-09-17T09:40:49Z

@@ -1,129 +0,0 @@
-# Byte-compiled / optimized / DLL files


hmmm I still believe you should make this whole file disappear from the diff history.
Maybe get this file in a separate commit and then drop it using git rebase -i 🤔
Consider it a git challenge on how to remove a file from your git commit diff :)

Add estate.property.type, estate.property.tag, and estate.property.offer models along with their actions and views to handle property classification and offers. - Create estate.property.type model with menu, action, and views - Create estate.property.tag model with action and list/form views - Create estate.property.offer model with list and form views - Add Many2one links (property_type_id, buyer_id, user_id) to estate.property - Add Many2many link (tag_ids) to estate.property - Add One2many link (offer_ids) to estate.property Task ID: 6573564

Address code review comments by improving field definitions, using lowercase selection keys, simplifying domains, and fixing PEP 8 blank line rules (E302). - Fix E302 missing blank lines before class declarations - Remove unused empty line and refine date_utils import in estate_property.py - Convert selection keys to lowercase for garden_orientation - Simplify domain filter syntax using in operator in estate_property_views.xml - Move field string labels to Python definitions - Fix typo in menu identifier estate_first_level_menu - Change manifest author to Odoo S.A. Task ID: 6573564

Restore .gitignore to its initial branch state to eliminate unneeded changes from the PR diff history. Task ID: 6573564

[ADD] estate: create initial module structure and manifest

55b2599

Initialize the estate module with its basic directory structure and manifest file as required for the technical onboarding training. task-6573564

mawat-odoo force-pushed the 19.0-Technical-Training-mawat branch from 2e4eb5e to 55b2599 Compare September 15, 2026 11:36

msho-odoo reviewed Sep 15, 2026

View reviewed changes

mawat-odoo force-pushed the 19.0-Technical-Training-mawat branch from c8728d7 to f34bcf9 Compare September 16, 2026 08:29

mawat-odoo added 2 commits September 16, 2026 10:38

[FIX] remove environmental gitignore file

8e7aed2

mawat-odoo force-pushed the 19.0-Technical-Training-mawat branch 2 times, most recently from cd8d2d9 to 06d81bc Compare September 16, 2026 11:11

[ADD] estate: add access rights for estate.property model

9f2dbc6

- Add security/ir.model.access.csv with full permissions for internal users - Declare security file in __manifest__.py task-6573564

mawat-odoo force-pushed the 19.0-Technical-Training-mawat branch 2 times, most recently from 669949c to 97a5918 Compare September 16, 2026 22:46

mawat-odoo force-pushed the 19.0-Technical-Training-mawat branch from 97a5918 to 32a2e25 Compare September 16, 2026 22:51

mawat-odoo added 2 commits September 17, 2026 03:06

[FIX] estate: fix state selection key formatting

af5fe3c

Task ID: 6573564

mawat-odoo force-pushed the 19.0-Technical-Training-mawat branch from f83d668 to 74647bf Compare September 17, 2026 07:12

msho-odoo reviewed Sep 17, 2026

View reviewed changes

mawat-odoo added 2 commits September 17, 2026 14:00

mawat-odoo force-pushed the 19.0-Technical-Training-mawat branch 2 times, most recently from 88dd0a3 to 0f630ff Compare September 17, 2026 13:01

[FIX] tutorials: restore original .gitignore

d4dd647

Restore .gitignore to its initial branch state to eliminate unneeded changes from the PR diff history. Task ID: 6573564

mawat-odoo force-pushed the 19.0-Technical-Training-mawat branch from 0f630ff to d4dd647 Compare September 17, 2026 13:54

		@@ -1,3 +1,9 @@
		# Created by https://www.toptal.com/developers/gitignore/api/python,odoo

Conversation

mawat-odoo commented Sep 15, 2026

Uh oh!

robodoo commented Sep 15, 2026

Uh oh!

msho-odoo left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mawat-odoo commented Sep 17, 2026

Uh oh!

msho-odoo left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants