-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[ADD] estate: create new module #1016
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
base: 19.0
Are you sure you want to change the base?
Conversation
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, Good Work!
I have left some comments.
When you push your new code, please do not merge it with the previous commit.
And make sure that your runbot is always green.
Thanks
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, Good work!
I have left some remarks.
As I mentioned in my last comment, make sure that your runbot is always green.
|
Hello! Please update your PR title and message. |
create new module estate and create model for new module and add security
Create module Real Estate add menu list set column default, read-only
In estate module add basic views list view and form view Also add filter for new or offer received stage
In estate module add basic views list view and form view Also add filter for new or offer received stage improve syntax in estate_property model
Add new menu item property type and tags Add buyer and salesperson fields Add offer section
compute field like total area best offer add validate and deadline fields with computation add object type action on sold and cancel button and buyer field add accept and refuse button in offer form
add SQL and Python both type constraints in database
86d4d16 to
12b8775
Compare
add status bar in property set order in model add handle list view of property type add tag color
Implemented stat button on property type default filter in property page decorations in property offer and property make editable list view for property tag
restrict user to delete property which is not in new or sold state add property id field in user model using inheritance
created a new module to connect Real Estate with accounting features. create invoice when property sold
8f2fa92 to
191f8cc
Compare
add kanban view group by with property id disable drag and drop in kanban view
ad0a5b6 to
26b7079
Compare
Improve syntax of menu item Improve syntax in all views Set id name coding guidelines wise
bit-odoo
left a comment
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, i have added some comments please have a look into it.
fa73df3 to
0667675
Compare
Make estate module UI user-friendly Add counter and card component and display in awesome owl module
0667675 to
e06e590
Compare
add todo list in awesome_owl
34f14c3 to
b633817
Compare
bea9059 to
dc30f37
Compare
add controlpanel layout add customers and leads button in controlpanel layout add delete todo and toggle todo funtion add property id field after customer field in account move form
dc30f37 to
3aa257c
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,
I have added some comments. Please do not repeat the same mistake again and again.
| <Card title="'This is title1'" content="'Hello from card1'"/> | ||
| <Card title="'This is title2'" content="'Hello from card2'"/> | ||
| </t> | ||
|
|
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.
unnecessary diff.
| @@ -1,8 +1,15 @@ | |||
| <?xml version="1.0" encoding="UTF-8" ?> | |||
| <templates xml:space="preserve"> | |||
|
|
|||
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.
unnecessary diff.
| </t> | ||
| </Layout> | ||
| </t> | ||
|
|
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.
unnecessary diff.
awesome_owl/static/src/card/card.xml
Outdated
| <div class="card-body"> | ||
| <t t-slot="default"/> | ||
| </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.
Wrong indentation.
| <p>Counter: <t t-esc="state.value"/></p> | ||
| <button class="increment-button" t-on-click="increment"> | ||
| Increment | ||
| </button> |
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.
same.
awesome_owl/static/src/playground.js
Outdated
|
|
||
| } | ||
|
|
||
| totalSum(newVal,Checker){ |
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.
| totalSum(newVal,Checker){ | |
| totalSum(newVal,Checker) { |
awesome_owl/static/src/playground.js
Outdated
| } | ||
|
|
||
| totalSum(newVal,Checker){ | ||
| if(Checker) |
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.
| if(Checker) | |
| if (Checker) { |
| @@ -1,10 +1,23 @@ | |||
| <?xml version="1.0" encoding="UTF-8" ?> | |||
| <templates xml:space="preserve"> | |||
|
|
|||
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.
unnecessary diff.
| <templates xml:space="preserve"> | ||
| <t t-name="awesome_owl.TodoItem"> | ||
| <div class="p-1" t-att-class="props.todo.isCompleted ? 'text-decoration-line-through text-muted' : ''"> | ||
| <input class="form-check-input" type="checkbox" t-on-click="onChangeToggle"/> |
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.
indentation issue.
aa6286c to
6624d73
Compare
add dashboard item which is number card and pie chart card add lazy loading functionality for dashboard making our dashboard generic added functionality add and remove dashboard items
6624d73 to
ae9a44d
Compare

Create new module estate module for manage estate property with feature like create new property manage buy and sell system
Create Card, Counter and Todo component
add customers and leads button in layout