[IMP] MRP: updated bills of materials rst content
Reviewed the 15.0 doc for Bills of Materials and made some edits to clean up inconsistencies and made minor wording changes. Also cleaned up the line breaks and added alt captions.
@mivu-odoo I'm going through the existing user docs for MRP 15.0 and cleaning up the wording and some of the rst formatting. No major content changes were made to this particular doc, it was just a little bit of tidying up to make things flow better :) Thank you!
@mivu-odoo I implemented your suggested edits. I had a little trouble rebasing/pushing bad commits, so Zac helped me out, but it should all be fixed.
Hi @weka-odoo @StraubCreative !
If I remember correctly, alternative_wc.rst is only in the files changed tab because we were running into merge conflicts and found that an image file we were trying to rename in bill_configuration.rst was also being used in alternative_wc.rst. I didn't do content review for alternative_wc.rst as @weka-odoo has been opening new PRs for each existing user doc, since almost all of them have to be heavily rewritten. @weka-odoo were you planning to open a separate PR for alternative_wc.rst? If so, let us know or if you just want to knock it out on this PR, we can do that too, we just have to set this back a stage for you to make your changes and then do final content review for it. Sorry for not being clear on this earlier before we sent this PR to you for final technical review @StraubCreative !
As for product_variants.rst, not sure why that file is in the files changed tab for this PR. Wesley opened PR #2648 for his changes to that RST file, and I already did final content review there. It's just waiting for final technical review.
@mivu-odoo @StraubCreative Hello! Yes - Michelle is correct. I'm planning on opening a separate PR for the Alternative Work Centers user doc, I just haven't gotten to it yet. I can reference Zac's current edits when I get to opening the PR/rewriting the Alt Work Centers doc! Let me know if you need any further clarifications.
@weka-odoo @mivu-odoo
Per request, and to limit the scope of this PR to just bill_configuration.rst, I just reset alternatives_wc.rst and product_variants.rst to the live copies that are on 15.0.
The only change I made was to alternatives_wc.rst to address the image file conflict we ran into earlier, otherwise the content updates are limited back to bill_configuration.rst.
Please update your local copies by doing a fresh git pull on 15.0, and a git pull --rebase on 15.0-MRP-billsofmaterialsedits-weka. After you update your local copy of the feature branch, run git log --online, and check to see that the commit at the top says c7e42f55.
Assuming those checks pass, please also review bill_configuration.rst to be sure all of the changes you wanted to make (and passed content review) are there.
Thanks / let me know 🙏
@StraubCreative I implemented your suggested edits. Please take a look whenever you get the chance!
And just as a heads up, I have terms capitalized in the headings (i.e. "Bills of Materials", "Operations", "By-Products"). In the body of the text, however, I only capitalize terms when first defining them, and then use sentence case for all remaining incidences.
@StraubCreative made the suggested edits! Also fixed those pesky notes. Feel free to review it whenever you get the chance :)
Pushing up changes in 7c64b71, as follows:
- edit line break on
bill_configuration.rstin approval comment above - solve
product_variants.rstmerge conflict - RST formatting for
alternative_wc.rstfor future PR where content is revised (might as well clean it up now since we had to make an edit to the file anyway 😉) - renamed generic
alternative_wcimages (same reason as above)
Note @weka-odoo, I only formatted the main body content. The only real things I added were tags and line breaks, along with alt descriptions to the images because those were missing. All this should make it easier to edit in future PR 👍
@odoo/doc-review this PR is ready for review :)
robodoo r+
I'm sorry, @StraubCreative: I must know your email before you can review PRs. Please contact an administrator.
robodoo r+
I'm sorry, @mivu-odoo: I must know your email before you can review PRs. Please contact an administrator.
robodoo r+