-
Notifications
You must be signed in to change notification settings - Fork 2.8k
19.0 estate tutorial hatiw #1051
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 @hatiw-odoo
You did a good job 🥳
I have added some suggestions. And can you please improve your commit message, and pr messag,e and title regarding these guidelines - https://www.odoo.com/documentation/19.0/contributing/development/git_guidelines.html
One more point, that no need to do a separate commit for this type of change .827b4a98419c2942d554368e20590ce5d9a63cb1
Thanks
| @@ -0,0 +1,17 @@ | |||
| { | |||
| 'name': "Real Estate", | |||
| 'version': '1.0', | |||
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.
Please follow the same quote throughout the files.
estate/security/ir.model.access.csv
Outdated
| @@ -0,0 +1,2 @@ | |||
| "id","name","model_id:id","group_id:id","perm_read","perm_write","perm_create","perm_unlink" | |||
| "estate.access_estate_property","access_estate_property","estate.model_estate_property","base.group_user",1,0,0,0 No newline at end of file | |||
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.
I think you should give all access write to the user.
And there should be one empty line at the end of the file.
estate/views/estate_menus.xml
Outdated
| name="Configuration" | ||
| parent="estate_menu_root" | ||
| action="action_estate_configure" /> | ||
| </odoo> No newline at end of file |
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.
There should be one empty line at the end of the file.
| </p> | ||
| </field> | ||
| </record> | ||
| </odoo> No newline at end of file |
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.
There should be one empty line at the end of the file.
Chapter 2 – Created the base structure for a new custom Odoo application. Chapter 3 – Implemented primary models with essential fields and basic logic. Chapter 4 – Added ACLs and basic security rules for controlled data access.
Chapter 5 – Built initial UI components including menus, views, and actions.
827b4a9 to
1fd9b0b
Compare
|
Changes updated in the commit message as per the direction |

created the real estate application