Skip to content

[16.0][IMP] ddmrp: add purchase state hook#587

Merged
OCA-git-bot merged 1 commit intoOCA:16.0from
ForgeFlow:16.0-imp-ddmrp-purchase_state_hook
Mar 5, 2026
Merged

[16.0][IMP] ddmrp: add purchase state hook#587
OCA-git-bot merged 1 commit intoOCA:16.0from
ForgeFlow:16.0-imp-ddmrp-purchase_state_hook

Conversation

@DavidJForgeFlow
Copy link
Contributor

No description provided.

@OCA-git-bot
Copy link
Contributor

Hi @JordiBForgeFlow, @LoisRForgeFlow, @ChrisOForgeFlow,
some modules you are maintaining are being modified, check this out!

result["domain"] = [("id", "in", moves.ids)]
return result

def _get_purchase_order_states(self):
Copy link
Contributor

Choose a reason for hiding this comment

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

@DavidJForgeFlow maybe _get_unconfirmed_po_states is a more descriptive name? _get_purchase_order_states sound too broad

@DavidJForgeFlow DavidJForgeFlow force-pushed the 16.0-imp-ddmrp-purchase_state_hook branch from 1b6d51e to dd958bd Compare March 4, 2026 11:39
@DavidJForgeFlow DavidJForgeFlow force-pushed the 16.0-imp-ddmrp-purchase_state_hook branch from dd958bd to 07e05b9 Compare March 4, 2026 11:51
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

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
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-587-by-LoisRForgeFlow-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 4c86c34 into OCA:16.0 Mar 5, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

3 participants