-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implemented Real Estate for Technical Training #347
base: 18.0
Are you sure you want to change the base?
Conversation
…tributes of the tags
@@ -0,0 +1,2 @@ | |||
from . import models | |||
# from . import views |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove unnecessary comments.
"depends": ["base"], | ||
"application": True, | ||
"description": """ | ||
Real Estate Test Description |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Provide appropriate description.
@@ -0,0 +1,140 @@ | |||
from odoo import models, fields, api |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Import as per per coding guideline.
("west", "West"), | ||
], | ||
) | ||
# Following active field refer to the visiblity of field in search |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only add comments if required.
property_seller_id = fields.Many2one( | ||
"res.users", string="Salesman", default=lambda self: self.env.user | ||
) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove extra blank lines
_description = "Real Estate Property Tags" | ||
|
||
name = fields.Char(string="Tag", required=True) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove extra lines and extra comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove extra blanks line and comments.
<menuitem id="estate_model_menu_action" action="estate_model_action" /> | ||
</menuitem> | ||
<menuitem id="estate_type_first_level_menu" name="Settings"> | ||
<menuitem id="estate_type_model_menu_action" action="estate_type_model_action" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -0,0 +1,11 @@ | |||
<odoo> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing <?xml version="1.0" encoding="utf-8"?>
</form> | ||
</field> | ||
</record> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove blank line.
No description provided.