-
Notifications
You must be signed in to change notification settings - Fork 2.9k
First try #1130
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?
First try #1130
Conversation
artn-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.
Looks good so far !
All there is to say are small styling nitpicks, but the tests won't pass if they are not fixed.
We try to always leave an empty line at the end of a file, if you are using vscode, this can be done automatically by checking the Insert Final Newline option.
We also have pretty strict guidelines for the commit name and definition, which can be found here. https://www.odoo.com/documentation/16.0/contributing/development/git_guidelines.html
Co-authored-by: Arthur Nanson <artn@odoo.com>
Co-authored-by: Arthur Nanson <artn@odoo.com>
This addition will allow to make the newly created model usable through the Odoo frontend and to validate access rights and views integration.
This commit enhances the estate.property model by introducing reserved fields and field attributes required in the UI.
The default auto-generated views are not suitable for real usage. This change provides a clearer and more efficient UI for users by presenting the key information in a structured way and enabling faster retrieval of relevant properties via dedicated search options.

No description provided.