Skip to content

[16.0][IMP] mrp_bom_priority: moving function overwrite to a hook instead#1

Open
mihien wants to merge 3 commits into
grap:16.0-ADD-mrp_bom_priority-QDfrom
coopiteasy:16.0-ADD-mrp_bom_priority-QD
Open

[16.0][IMP] mrp_bom_priority: moving function overwrite to a hook instead#1
mihien wants to merge 3 commits into
grap:16.0-ADD-mrp_bom_priority-QDfrom
coopiteasy:16.0-ADD-mrp_bom_priority-QD

Conversation

@mihien

@mihien mihien commented Oct 28, 2025

Copy link
Copy Markdown

Contribution for the PR #1407.
This is an implementation of the solution suggested in this comment.

@github-grap-bot

Copy link
Copy Markdown

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

@mihien

mihien commented Oct 28, 2025

Copy link
Copy Markdown
Author

@legalsylvain Here are the changes I was talking about, if you could you give me a review on these I'll be delighted

@codecov

codecov Bot commented Oct 28, 2025

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Comment thread mrp_bom_priority/hooks.py
Comment on lines +42 to +43

if not hasattr(MrpBom, "_bom_find_original"):

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.

where is defined _bom_find_original ?

@legalsylvain legalsylvain force-pushed the 16.0-ADD-mrp_bom_priority-QD branch from 2fa162b to d07f737 Compare October 28, 2025 21:20
legalsylvain pushed a commit that referenced this pull request Oct 28, 2025
…-fr-i18n-QD

[16.0][i18n] mrp_production_back_to_draft add FR.po
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants