Skip to content
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: create an estate app #335

Draft
wants to merge 5 commits into
base: 18.0
Choose a base branch
from

Conversation

dvsh-odoo
Copy link

  • Created an estate application for odoo
  • Added some basic models and fields
  • Added security for the models
  • Added menu, basic views (Form and List)

- Created an estate application for odoo
- Added some basic models and fields
- Added security for the models
- Added menu, basic views (Form and List)
@robodoo
Copy link

robodoo commented Feb 6, 2025

Pull request status dashboard

</form>
</field>
</record>

Choose a reason for hiding this comment

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

Suggested change

Choose a reason for hiding this comment

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

In views the patch for all the views(list, form, search and etc.) should be before the patch of it's action window

- Added list, form, search view to the estate app
- Added filters and group by options to the estate app
@dvsh-odoo dvsh-odoo force-pushed the 18.0-tutorials-dvsh branch from 4204bb3 to d1afe73 Compare February 7, 2025 07:45
@dvsh-odoo
Copy link
Author

@sami-odoo Thanks for the suggestion, all mentioned changes have been done.

- Added estate_property_tag, estate_property_type, estate_property_offer models.
- Added  Many2one, Many2many, One2many relations between different models in the
estate app.

Choose a reason for hiding this comment

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

@dvsh-odoo the name of the app is "Real Estate"

Choose a reason for hiding this comment

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

It's better to have one line of space between different records

- Added total_area field to compute dynamically.
- Added computation & inverse computation to required fields.
- Added Sold and Cancel button and did error handling.
- Added SQL and Python constraints to expected price,selling price accordingly.
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