[ADD] real_estate: added real estate module #327
Draft
+260
−0
Conversation
This file contains 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
|
|
nmak-odoo
force-pushed
the
18.0-training-nmak
branch
from
7a0aca0 to
03625d0
Compare
Day 1 : Chapter 2 : Created Folder Structure for Real-estate module Chapter 3 : Created estate_properties model with all the basic fields [IMP] real-estate: security , ui and search filters Day 2 : Chapter 4: Understood Data files(CSV) and gave access-rights Chapter 5: Understood Data files(XML) and Implemented actions , menus and fields Chapter 6: Understood Basic views and Implemented List , Form and Search filters
nmak-odoo
force-pushed
the
18.0-training-nmak
branch
from
03625d0 to
876288a
Compare
nmak-odoo
force-pushed
the
18.0-training-nmak
branch
from
8676a02 to
f7b3476
Compare
Day 3 - Chapter 7: Added Many2one relationship for property types (dropdown selection). Added Many2many relationship for property tags (below name field). Partially implemented One2many relationship for property offers.
nmak-odoo
force-pushed
the
18.0-training-nmak
branch
from
f7b3476 to
fb43c28
Compare
maad-odoo
reviewed
Feb 6, 2025
| @@ -0,0 +1 @@ | |||
| hello | |||
There was a problem hiding this comment.
This file is not needed. Please remove it
| { | ||
| "name": "Real Estate", | ||
| "author": "nmak", | ||
| "website": "https://www.odoo.com/apps/estate", |
There was a problem hiding this comment.
Not a good idea to URL that is not legit
| @@ -0,0 +1 @@ | |||
| from . import models | |||
There was a problem hiding this comment.
| "name": "Real Estate", | ||
| "author": "nmak", | ||
| "website": "https://www.odoo.com/apps/estate", | ||
| "category": "tutorials/estate", |
There was a problem hiding this comment.
Suggested change
| "category": "tutorials/estate", | |
| 'category': 'Tutorials/estate', |
| "installable": True, | ||
| "application": True, | ||
| "auto_install": False, | ||
| "license": "LGPL-3", |
There was a problem hiding this comment.
Smae
Suggested change
| "license": "LGPL-3", | |
| 'license': 'LGPL-3', |
Same as above
Comment on lines
+3
to
+4
| class PropertyTag(models.Model): | ||
| _name = "estate.property.tag" |
There was a problem hiding this comment.
Suggested change
| class PropertyTag(models.Model): | |
| _name = "estate.property.tag" | |
| class EstatePropertyTag(models.Model): | |
| _name = "estate.property.tag" |
|
|
||
| class PropertyTag(models.Model): | ||
| _name = "estate.property.tag" | ||
| _description = "tags involved in real estate" |
There was a problem hiding this comment.
Please adapt the description message, it not up to the mark
Comment on lines
+1
to
+3
| from odoo import models,fields | ||
|
|
||
| class PropertyTag(models.Model): |
There was a problem hiding this comment.
Suggested change
| from odoo import models,fields | |
| class PropertyTag(models.Model): | |
| from odoo import models,fields | |
| class PropertyTag(models.Model): |
We usually leave 2 empty line before class definition
Comment on lines
+1
to
+3
| from odoo import models, fields | ||
|
|
||
| class EstatePropertyType(models.Model): |
There was a problem hiding this comment.
Suggested change
| from odoo import models, fields | |
| class EstatePropertyType(models.Model): | |
| from odoo import models, fields | |
| class EstatePropertyType(models.Model): |
2 empty line before class definition
| <field name="arch" type="xml"> | ||
| <form string="Property"> | ||
| <sheet> | ||
| <h1 class=""> |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Created the real estate module for technical training