Skip to content

[HSCHA]: Estate Module Implementation - Tutorials - #1400

Draft
Hansil-Chapadiya wants to merge 17 commits into
odoo:19.0from
odoo-dev:19.0-tutorial-hscha
Draft

[HSCHA]: Estate Module Implementation - Tutorials#1400
Hansil-Chapadiya wants to merge 17 commits into
odoo:19.0from
odoo-dev:19.0-tutorial-hscha

Conversation

@Hansil-Chapadiya

Copy link
Copy Markdown

Description

Implementation of tutorial modules covering estate and app functionality:

  • Initialized estate module along with code formatting and security fixes.
  • Initialized app module including initial setup, model fields with default values, and security configurations.
  • Added views, webicon, menu splits, and custom filters/search elements for app.

Add the initial structure of the estate module, including
the manifest, basic model, menu, and application icon,
as part of the Odoo Server Framework 101 training.
@robodoo

robodoo commented Aug 31, 2026

Copy link
Copy Markdown

Pull request status dashboard

@Hansil-Chapadiya
Hansil-Chapadiya changed the base branch from master to 19.0 August 31, 2026 07:30

@bit-odoo bit-odoo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @Hansil-Chapadiya
Good Start!

why you created module name with 'app' ? Why are there two models one is ' estate.proper.model' and second one is ' estate.property'?
Also i have added some comments please look into it.

Thanks

Comment thread estate/models/estate_property.py Outdated
name = fields.Char(string="Property Name", required=True)
reference = fields.Char(string="Reference")
description = fields.Text(string="Description")

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unneccary extra line.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in d49cd43

Comment thread estate/models/estate_property.py Outdated
bedrooms = fields.Integer(string="Bedrooms")
living_area = fields.Float(string="Living Area (sqm)")
garden_area = fields.Float(string="Garden Area (sqm)")

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unneccary extra line.

Comment thread estate/models/estate_property.py Outdated

expected_price = fields.Float(string="Expected Price")
selling_price = fields.Float(string="Selling Price")

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unneccary extra line.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in new version of estate module

Comment thread .gitignore

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unneccary diff.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in 1944db4

Comment thread estate/views/estate_menus.xml Outdated
<field name="description"/>
</page>
</notebook>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unneccary extra line.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in new version of estate

Comment thread estate/__manifest__.py Outdated
'application': True,
'license': 'LGPL-3',
'author': 'Hansil Chapadiya',
'website': 'https://www.gstatic.com',

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not related to our app.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it was for testing and it has deleted now

Comment thread estate/views/estate_menus.xml Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

file name is not proper.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed in 18055be

@Hansil-Chapadiya

Copy link
Copy Markdown
Author

Hello @Hansil-Chapadiya Good Start!

why you created module name with 'app' ? Why are there two models one is ' estate.proper.model' and second one is ' estate.property'? Also i have added some comments please look into it.

Thanks

It was for testing purpose, "estate" -> dummy module and "app" -> original module
now it is fixed and rename to "estate" and dummy "estate" is deleted.

@Hansil-Chapadiya Hansil-Chapadiya changed the title [ADD] estate, app: tutorial module implementation and initializations [ADD] estate: tutorial module implementation and initializations Sep 3, 2026
@smitinit smitinit changed the title [ADD] estate: tutorial module implementation and initializations [HSCHA]: Estate Module - Tutorials Sep 3, 2026
@Hansil-Chapadiya Hansil-Chapadiya changed the title [HSCHA]: Estate Module - Tutorials [HSCHA]: Estate Module Implementation - Tutorials Sep 4, 2026
…added

Extend the estate.property model by introducing Many2one links for
Property Type, Buyer,Salesperson.

Link Property Type with estate.property.type
Link Buyer with res.partner
Link Salesperson with res.users
Create the required property type model
Configure appropriate access permissions
Add menus for managing property types
Update the property form view to display and showcase the new relational fields
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants