Skip to content

[17.0][IMP] tracking_manager_domain: support domains on one2many messages - #3247

Merged
OCA-git-bot merged 1 commit into
OCA:17.0from
c4a8-odoo:17.0-imp-tracking_manager_domain
Aug 21, 2025
Merged

[17.0][IMP] tracking_manager_domain: support domains on one2many messages#3247
OCA-git-bot merged 1 commit into
OCA:17.0from
c4a8-odoo:17.0-imp-tracking_manager_domain

Conversation

@CRogos

@CRogos CRogos commented Apr 2, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Apply domain filter on one2many messages to prevent this tracking messsage:
image

@CRogos
CRogos marked this pull request as draft April 3, 2025 12:42
@CRogos
CRogos force-pushed the 17.0-imp-tracking_manager_domain branch from 91ff4ff to 2f7ce9a Compare April 11, 2025 14:55
@CRogos CRogos mentioned this pull request Apr 11, 2025
Closed
@CRogos
CRogos marked this pull request as ready for review April 11, 2025 14:57
@CRogos CRogos changed the title [IMP] tracking_manager_domain: support domains on one2many messages [17.0][IMP] tracking_manager_domain: support domains on one2many messages May 21, 2025
@CRogos CRogos mentioned this pull request May 21, 2025
Merged
@CRogos

CRogos commented Aug 14, 2025

Copy link
Copy Markdown
Contributor Author

@pedrobaeza could you merge?

@pedrobaeza pedrobaeza added this to the 17.0 milestone Aug 14, 2025
@pedrobaeza

pedrobaeza commented Aug 14, 2025

Copy link
Copy Markdown
Member

/ocabot merge minor

@OCA-git-bot

OCA-git-bot commented Aug 14, 2025

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-3247-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 14, 2025
@OCA-git-bot
Merge PR #3247 into 17.0
d7b307b 
Signed-off-by pedrobaeza
@OCA-git-bot

OCA-git-bot commented Aug 14, 2025

Copy link
Copy Markdown
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-3247-by-pedrobaeza-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@CRogos
CRogos force-pushed the 17.0-imp-tracking_manager_domain branch from 2f7ce9a to 7acaed1 Compare August 14, 2025 16:12
@CRogos

CRogos commented Aug 14, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

test failed solved by: #3343

@CRogos

CRogos commented Aug 18, 2025

Copy link
Copy Markdown
Contributor Author

@pedrobaeza could you retry?

@pedrobaeza

pedrobaeza commented Aug 18, 2025

Copy link
Copy Markdown
Member

But the mentioned PR is for 18.0 and this is 17.0.

@CRogos

CRogos commented Aug 18, 2025

Copy link
Copy Markdown
Contributor Author

Uh... you are right. the correct PR would be #3345 . maybe you could merge this first?

@pedrobaeza

pedrobaeza commented Aug 21, 2025

Copy link
Copy Markdown
Member

/ocabot merge minor

@OCA-git-bot

OCA-git-bot commented Aug 21, 2025

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-3247-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 6f52061 into OCA:17.0 Aug 21, 2025
7 of 9 checks passed
@OCA-git-bot

OCA-git-bot commented Aug 21, 2025

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 26faf3e. 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

Reviewers

2 more reviewers
@JessBrandl JessBrandl JessBrandl approved these changes
@MohamedOsman7 MohamedOsman7 MohamedOsman7 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉

Projects

None yet

Milestone

17.0

Development

Successfully merging this pull request may close these issues.

5 participants

@CRogos @pedrobaeza @OCA-git-bot @JessBrandl @MohamedOsman7