-
-
Notifications
You must be signed in to change notification settings - Fork 171
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][ADD] account_dms_field: New module #364
[16.0][ADD] account_dms_field: New module #364
Conversation
db0cc4c
to
a403e98
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.
Since #368 these changes are no longer necessary
def _create_dms_directory(self): | ||
"""New method to define the directory automatically in demo data avoiding | ||
error if it already exists.""" | ||
self.ensure_one() | ||
if not self.dms_directory_ids: | ||
self.with_context( | ||
res_model=self._name, res_id=self.id | ||
).create_dms_directory() |
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.
def _create_dms_directory(self): | |
"""New method to define the directory automatically in demo data avoiding | |
error if it already exists.""" | |
self.ensure_one() | |
if not self.dms_directory_ids: | |
self.with_context( | |
res_model=self._name, res_id=self.id | |
).create_dms_directory() |
<function model="dms.field.template" name="_create_dms_directory"> | ||
<value eval="[ref('field_template_account')]" /> | ||
</function> |
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.
<function model="dms.field.template" name="_create_dms_directory"> | |
<value eval="[ref('field_template_account')]" /> | |
</function> |
@bizzappdev please review |
6cf41e2
to
42d3cb3
Compare
fix author and correct Readme
42d3cb3
to
0d48aef
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.
Code and functional test LGTM
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at b536f8a. Thanks a lot for contributing to OCA. ❤️ |
This module adds "account_dms_field"