[IMP] MRP: Subcontractor Stock on BOM Overview#19079
Conversation
|
|
|
Hi @theRealThagomizer got a PR ready for your review when you get a chance - thank you |
theRealThagomizer
left a comment
theRealThagomizer
left a comment
There was a problem hiding this comment.
Hiya, @stepperpig! Looks like this doc has had some of the same issues I've run into with other 19.4 updates -- it looks like there's been a bunch of knock-on effects caused by the changes that the devs made. I'm marking this as "Request changes" because the workflows this doc captures appear to have changed substantially enough in places that they can't be followed as outlined. I also cleaned up a few style convention things. Sorry to turn this into more work for you, but at least it ought to be worth more points now!
f2f6092 to
01118ad
Compare
|
@theRealThagomizer just went and fixed a few things. this doc might need some further work in the future (e.g., separating it out into separate docs), but for now i'll restrict any changes to just making sure it appears and functions correctly for 19.4. |
theRealThagomizer
left a comment
theRealThagomizer
left a comment
There was a problem hiding this comment.
Good changes, @stepperpig! I had a few philosophical notes you can take or leave, but this is otherwise ready to move on!
eab6002 to
98e8563
Compare
|
Thanks for the review @theRealThagomizer |
erjer-odoo
left a comment
erjer-odoo
left a comment
There was a problem hiding this comment.
One small correction that I'd like you to make. In the future, we may want to consider a PR that just goes ahead and fixes every anchor in this file, but for right now, I'll leave the other ones alone.
@robodoo delegate+
Co-authored-by: theRealThagomizer <96515928+theRealThagomizer@users.noreply.github.com> Co-authored-by: theRealThagomizer <96515928+theRealThagomizer@users.noreply.github.com> Co-authored-by: Erin J <erjer@odoo.com>
84ce515 to
d252aaf
Compare
|
@robodoo r+ |
closes #19079 Signed-off-by: Kai Wen Li (kaili) <kaili@odoo.com> Co-authored-by: theRealThagomizer <96515928+theRealThagomizer@users.noreply.github.com> Co-authored-by: theRealThagomizer <96515928+theRealThagomizer@users.noreply.github.com> Co-authored-by: Erin J <erjer@odoo.com>
4 participants
Description
This PR updates the 'BOM Overview' section of the 'Bill of Materials' page to reflect the addition of the 'Subcontractor Stock' column for subcontracted products when the Forecast mode is enabled in BOM Overview in 19.4. Previously, this column did not exist and the overview tracked only the available stock at the subcontractor-level. This feature adds a separate subcontractor-specific column to separate stock information between the company- and subcontractor-levels, keeping track of each individually for clarity.
Task card
Changelog
Version Scope
This PR can be FWP up to master.
Additional PRs may be opened to address missing BOM overview document pre-19.3.