-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
16.0 mig agreement #7
Changes from 1 commit
d294911
d8b44d7
98d84f5
54e3bfa
ed9361b
2e7d8dd
71c9891
f513319
b7995d6
cbb57ad
33de2da
2124cf5
f0fd1c4
77f1ee5
472622c
84151db
7cc5c69
0c98659
e593cd0
0524780
504b81c
66e448a
129ef19
fcea20d
9ba17f6
d1a61e0
32b84d5
1db7032
9fa7ae1
154b128
19dc58c
48c80e6
7fc0513
d5016b6
7537f1b
4a1a701
5381486
97d9e7e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,19 +31,20 @@ | |
<group name="main"> | ||
<group name="left"> | ||
<field name="code" /> | ||
<field | ||
name="is_template" | ||
groups="agreement.group_use_agreement_template" | ||
/> | ||
<field | ||
name="partner_id" | ||
attrs="{'required': [('is_template', '=', False)]}" | ||
groups="agreement.group_use_agreement_template" | ||
/> | ||
<field | ||
name="agreement_type_id" | ||
groups="agreement.group_use_agreement_type" | ||
/> | ||
<field name="domain" widget="radio" /> | ||
<field | ||
name="is_template" | ||
groups="agreement.group_use_agreement_template" | ||
/> | ||
</group> | ||
<group name="right"> | ||
<field name="signature_date" /> | ||
|
@@ -55,13 +56,13 @@ | |
/> | ||
</group> | ||
</group> | ||
<notebook> | ||
<page | ||
name="config" | ||
string="Configuration" | ||
attrs="{'invisible': True}" | ||
/> | ||
</notebook> | ||
<!-- <notebook>--> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why are you commenting this? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good question |
||
<!-- <page--> | ||
<!-- name="config"--> | ||
<!-- string="Configuration"--> | ||
<!-- invisible="1"--> | ||
<!-- />--> | ||
<!-- </notebook>--> | ||
</sheet> | ||
<div class="oe_chatter"> | ||
<field name="message_follower_ids" widget="mail_followers" /> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
../../../../agreement | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should comes in a separated commit named [MIG] agreement: pre-commit execution There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should comes in a separated commit named [MIG] agreement: pre-commit execution |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
import setuptools | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should comes in a separated commit named [MIG] agreement: pre-commit execution There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should comes in a separated commit named [MIG] agreement: pre-commit execution |
||
|
||
setuptools.setup( | ||
setup_requires=['setuptools-odoo'], | ||
odoo_addon=True, | ||
) |
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.
Why do you need this group now?
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.
He added this group because the attr need the value of is_template but it is protected by a group. I prefer to remove the group on partner_id and add before the field is_template with invisible="1". In this way, the result of the attr can be evaluated by odoo.