-
-
Notifications
You must be signed in to change notification settings - Fork 749
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
[IMP] project_type module #1235
base: 16.0
Are you sure you want to change the base?
Conversation
Can anyone tell me what is the problem ? |
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 are the OCA Guidelines: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst
Have a look at the pre-commit installation.
https://github.com/OCA/maintainer-tools/wiki/Install-pre-commit
pre-commit should format your code before committing it.
And you need to squash all your commits.
@@ -21,6 +22,7 @@ class ProjectType(models.Model): | |||
project_ok = fields.Boolean(string="Can be applied for projects", default=True) | |||
task_ok = fields.Boolean(string="Can be applied for tasks") | |||
code = fields.Char(copy=False) | |||
sequence = fields.Integer(string="Order") |
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.
You should not change the display name. Order is not optimal because of Sales Order / Purchase Order.
Add an integer sequence field and handle order management in tree view.