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

[MOV] MRP: Create subcontracting scope #8969

Closed
wants to merge 1 commit into from

Conversation

hojo-odoo
Copy link
Contributor

This PR creates a new subsection for subcontracting docs within the Manufacturing scope.

@robodoo
Copy link
Collaborator

robodoo commented Apr 24, 2024

@C3POdoo C3POdoo requested a review from a team April 24, 2024 20:54
@hojo-odoo hojo-odoo requested review from ksc-odoo and removed request for a team April 24, 2024 20:59
@hojo-odoo hojo-odoo added the 5 label Apr 24, 2024
@hojo-odoo
Copy link
Contributor Author

Hey @ksc-odoo,

This is ready for the first round of review.

Thanks,
hojo

Copy link
Contributor

@ksc-odoo ksc-odoo left a comment

Choose a reason for hiding this comment

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

Hey @hojo-odoo -- just finished my Peer Review. nice work on this [MOV] PR -- like the addition of the cards up top -- nice touch 👍 think this one is ready for the next stage whenever you're ready.

@hojo-odoo
Copy link
Contributor Author

Hey @StraubCreative,

This is ready for final review.

Thanks,
hojo

Copy link
Contributor

@StraubCreative StraubCreative left a comment

Choose a reason for hiding this comment

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

LGTM 👍
Nice touch with the cards!

@StraubCreative
Copy link
Contributor

@samueljlieber can you take a look?

Copy link
Contributor

@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

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

Looks good to me @hojo-odoo, nice job! 👍
...
@robodoo r+

@fw-bot
Copy link
Collaborator

fw-bot commented May 4, 2024

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

Successfully merging this pull request may close these issues.

None yet

6 participants