-
Notifications
You must be signed in to change notification settings - Fork 2.3k
[ADD] estate_property: added real estate module #840
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
Draft
ksoz-odoo
wants to merge
7
commits into
odoo:18.0
Choose a base branch
from
odoo-dev:18.0-training-ksoz
base: 18.0
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,392
−26
Conversation
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
deso-odoo
reviewed
Jul 14, 2025
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.
Here is the review!
Some comments:
Follow commit message guidelines. This tool can be helpful.
- Last 2 Commits are not needed you should use commit amend and fix it in the original commit.
- Many commits includes changes that are not related to that specific commit, i.e, you change code that should instead be amended in the original commit itself, can you fix such issues.
- Can you have a look at linting in all your files. I see too many linting issues.
639e92e
to
368f2a9
Compare
Creation of Estate App and properly understanding the flow of creation of app. Creating models folder and adding estate property model with mentioned field
64182b3
to
2b96d72
Compare
Added security files for estate_property module Implemented search functionality for estate properties Developed listing and filtering features for better property management
Added estate property model and tags model. Made computed fields and inverse fields. Also Learnt about button action.
Conditional views added to the garden field In offer model conditional fields added Added constraints to the estate_property table and estate_property_tag Editable list view for the tags and offer Smart button added in type of estate property to view its related offer [FIX] estate_property: fixed new estate_account and estate_property Fixed Check style errors Added trailing spaces and removed whitespaces [FIX] estate_property: fixed new estate_account and estate_property Fixed Check style errors Added trailing spaces and removed whitespaces
c6d8cef
to
67c8864
Compare
Added security for agent and manager specifying about what they can view and what they can edit Added report and everything properly Update the security csv removed base group user and Added separate role for agent and manager
67c8864
to
925c947
Compare
Implemented fundamental Owl concepts including: Initial project setup and basic rendering. State management with and refs with . Reusable Counter component with sum functionality. Solid understanding and application of props. Adhered to proper folder structure. Developed a complete Todo List application(add, remove, mark as completed).
43b2a66
to
60a0128
Compare
add dashboard layout, navigation, stats, and lazy loading Integrated Layout component with control panel Added Customers and Leads navigation buttons Created reusable DashboardItem component with size prop Fetched and displayed stats via RPC (/statistics route) Registered statistics service with caching (memoize) Added PieChart with lazy-loaded Chart.js Made stats reactive with periodic refresh (10s) Moved dashboard to /dashboard and enabled lazy loading via LazyComponent
60a0128
to
59f6a64
Compare
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.
This pull request contains the completed Real Estate module from the Odoo
developer tutorial. It features comprehensive data models for properties, types,
tags, and offers, all linked with relational fields (Many2one, One2many,
Many2many). Business logic is handled through computed fields, ORM overrides
(create, ondelete), and constraints. The user interface includes Form, List,
Kanban, and Search views with dynamic buttons and a link to the Accounting
module for invoicing.