Skip to content

[18.0][IMP] dms_field: Set the group_ids field as required if there is no parent directory#497

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
Tecnativa:18.0-imp-dms_field-issue-494-4
Jul 21, 2026
Merged

[18.0][IMP] dms_field: Set the group_ids field as required if there is no parent directory#497
OCA-git-bot merged 1 commit into
OCA:18.0from
Tecnativa:18.0-imp-dms_field-issue-494-4

Conversation

@victoralmau

Copy link
Copy Markdown
Member

Set the group_ids field as required if there is no parent directory

Related to #494 (Bug 4)

Please @pedrobaeza and @CarlosRoca13 can you review it?

@Tecnativa

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @CarlosRoca13,
some modules you are maintaining are being modified, check this out!

@CarlosRoca13 CarlosRoca13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

@eduezerouali-tecnativa eduezerouali-tecnativa left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-497-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit f1fa766 into OCA:18.0 Jul 21, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@pedrobaeza
pedrobaeza deleted the 18.0-imp-dms_field-issue-494-4 branch July 21, 2026 10:57
@pedrobaeza pedrobaeza added this to the 18.0 milestone Jul 22, 2026
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