-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[ADD] estate: Manage Your Properties, Maximize Your Returns #135
base: 17.0
Are you sure you want to change the base?
Conversation
374f472
to
818ba2b
Compare
-create new estate model -create offer for a property and relation between properties like one2Many , many2Many, many2One -create computed fields -create action fields and action buttons for sold & cancle and offer accept & refused -create views for property, type, tag and offer -create security for all the the above model
818ba2b
to
ea4b032
Compare
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.
Hello @aneg-odoo
Some comments mostly regarding the guidelines.
Can you please check.
Thanks!!
}), | ||
]" /> | ||
</record> | ||
</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.
Leave a blank line at the EOF
Make sure to check in other files also.
estate/models/estate_property.py
Outdated
from odoo import api, models, fields | ||
from datetime import datetime | ||
from dateutil.relativedelta import relativedelta | ||
from odoo.exceptions import UserError, ValidationError |
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.
from odoo import api, models, fields | |
from datetime import datetime | |
from dateutil.relativedelta import relativedelta | |
from odoo.exceptions import UserError, ValidationError | |
from datetime import datetime | |
from dateutil.relativedelta import relativedelta | |
from odoo import api, fields, models | |
from odoo.exceptions import UserError, ValidationError |
Generally, we follow a convention in which we first import external libraries and then we make import from odoo in alphabetical order. Do same for other files also
estate/models/estate_property.py
Outdated
# def check_limit(self, vals): | ||
# self.status = "offer received" | ||
|
||
# if vals.get("price") <= self.best_price: | ||
# raise ValidationError("the offer must be higher than best price") |
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.
Try to avoid pushing the commented code.
<div class="d-flex justify-content-end"> | ||
<header> | ||
<button name="%(estate.estate_property_offer_action)d" type="action" | ||
class="oe_stat_button" | ||
icon="fa-money" string="Offers"> | ||
<field name="offer_count" string="Offer Count" /> | ||
</button> | ||
</header> | ||
</div> |
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.
<div class="d-flex justify-content-end"> | |
<header> | |
<button name="%(estate.estate_property_offer_action)d" type="action" | |
class="oe_stat_button" | |
icon="fa-money" string="Offers"> | |
<field name="offer_count" string="Offer Count" /> | |
</button> | |
</header> | |
</div> | |
<div name="button_box"> | |
<button class="oe_stat_button" type="action" name="%(estate.action_open_offer)d" | |
icon="fa-ticket"> | |
<field string="Offers" name="offer_count" widget="statinfo"/> | |
</button> | |
</div> |
You can do something like this for better alignment of stat button
<filter string="group_by_postcode" name="postcode" | ||
context="{'group_by' :'postcode'}" /> | ||
</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.
Remove extra blank lines.
Check for other files also
…eatures (estate): add property offers report and sample data for testing - Created a new report for estate property offers (PDF format) - Added sample data for testing new estate property features - Fixed XML issues related to ir.actions.report and external ID conflicts
76028e2
to
fa30f74
Compare
-create new estate model
-create offer for a property and relation between properties like one2Many , many2Many, many2One
-create computed fields
-create action fields and action buttons for sold & cancle and offer accept & refused
-create views for property, type, tag and offer
-create security for all the the above model