Draft
Conversation
keman-odoo
commented
Mar 11, 2026
keman-odoo
commented
- chapter :2 task
- initiated manifest.py
|
|
keman-odoo
force-pushed
the
19.0-tutorials-estate-keman
branch
from
2019566 to
29b047f
Compare
bit-odoo
reviewed
Mar 16, 2026
bit-odoo
left a comment
bit-odoo
left a comment
There was a problem hiding this comment.
Hello @keman-odoo
Great Start!
No need to do a separate commit for this. a2c3ce4, ea70a78, 29b047f, 2dfefb7
Can you please follow commit message guidelines?
https://www.odoo.com/documentation/19.0/contributing/development/git_guidelines.html
Thanks
estate/models/estate_property.py
Outdated
Comment on lines
+1
to
+2
| from odoo import models, fields | ||
| from dateutil.relativedelta import relativedelta |
There was a problem hiding this comment.
Can you please follow the coding import guidelines?
https://www.odoo.com/documentation/19.0/contributing/development/coding_guidelines.html#imports
estate/views/estate_menus.xml
Outdated
| <menuitem id="estate_advertisements_menu" | ||
| name="Advertisements" | ||
| parent="estate_menu_root"/> | ||
|
|
estate/views/estate_menus.xml
Outdated
| <odoo> | ||
|
|
||
| <menuitem id="estate_menu_root" name="Real Estate"/> | ||
|
|
| 'category': "Tutorials", | ||
| 'depends': ['base'], | ||
| 'summary': "manage properties, track buyers offers and handle property sales efficiently", | ||
| 'sequence': "3", |
Author
There was a problem hiding this comment.
'sequence': 3 is used to control the order in which the module appears in the Apps menu.
keman-odoo
force-pushed
the
19.0-tutorials-estate-keman
branch
9 times, most recently
from
27f3e5e to
4aeb068
Compare
bit-odoo
reviewed
Mar 25, 2026
There was a problem hiding this comment.
Hello
You did many unneccary differences. Can you please look into it?
Can you please apply the changes that I gave in the previous review - #1196 (review)
Thank
estate/models/estate_property.py
Outdated
Comment on lines
+1
to
+2
| from dateutil.relativedelta import relativedelta | ||
| from odoo import fields, models |
There was a problem hiding this comment.
Can you please follow the import coding guidelines?
| </field> | ||
| </record> | ||
|
|
||
| </odoo> No newline at end of file |
| <field name="tags_ids" widget="many2many_tags"/> | ||
| </h4> | ||
| </h1> | ||
|
|
| <field name="postcode"/> | ||
| <field name="date_availability"/> | ||
| </group> | ||
|
|
| <field name="selling_price"/> | ||
| </group> | ||
| </group> | ||
|
|
|
|
||
| <filter name="available" | ||
| string="Available" | ||
| domain="['|', ('state','=','new'), ('state','=','offer_received')]"/> |
- chapter :2 task - initiated manifest.py [FIX] estate : fixed the errors - added the required white spaces and new line [FIX] Resolve minor mistake [FIX] estate: resolve minor mistake
- Create estate.property model. - Add basic property fields. - Add garden orientation selection. [FIX] resolve minor issues in estate property model - Fixed minor errors in estate.property model - Corrected small issues from previous commit
- Created security/ir.model.access.csv - Added the access file to __manifest__.py
- - Ensured estate.property model is properly defined
- Added tree and form views for estate.property - Chapter - 5 - Added menu items for estate module - Completed First UI tutorial exercise
- Add list, form and search views for estate.property with Available filter and Group By Postcode (Chapter 6)
- Created estate.property.type model - Added window action for property types - Added menu item under Settings - Users can now create property types (name field)
- Added property type (Many2one) - Added buyer and salesperson fields - Added tags using Many2many - Updated form view and menus
- Created estate.property.offer model - Added fields: price, status, partner_id, property_id - Implemented One2many relation (offer_ids) in estate.property - Created list and form views for offers - Added access rights for offer model
keman-odoo
force-pushed
the
19.0-tutorials-estate-keman
branch
2 times, most recently
from
c4786bc to
fecc8b6
Compare
- Implement computed field total_area using @api.depends - Calculate total_area as sum of living_area and garden_area - Use @api.depends for automatic updates
keman-odoo
force-pushed
the
19.0-tutorials-estate-keman
branch
from
fecc8b6 to
c6dec2b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.