Skip to content
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

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

agent-z28
Copy link

This module adds "account_dms_field"

@agent-z28 agent-z28 force-pushed the 16.0-account_dms_field-add-module branch 2 times, most recently from db0cc4c to a403e98 Compare September 30, 2024 16:39
Copy link
Member

@victoralmau victoralmau left a 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

Comment on lines 7 to 14
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()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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()

Comment on lines 19 to 21
<function model="dms.field.template" name="_create_dms_directory">
<value eval="[ref('field_template_account')]" />
</function>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<function model="dms.field.template" name="_create_dms_directory">
<value eval="[ref('field_template_account')]" />
</function>

@agent-z28
Copy link
Author

@bizzappdev please review

@agent-z28 agent-z28 force-pushed the 16.0-account_dms_field-add-module branch 2 times, most recently from 6cf41e2 to 42d3cb3 Compare October 16, 2024 06:44
fix author and correct Readme
@agent-z28 agent-z28 force-pushed the 16.0-account_dms_field-add-module branch from 42d3cb3 to 0d48aef Compare October 16, 2024 07:22
Copy link

@bizzappdev bizzappdev left a 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

@pedrobaeza pedrobaeza added this to the 16.0 milestone Oct 16, 2024
@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-364-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 8f118ac into OCA:16.0 Oct 16, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at b536f8a. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants