-
Notifications
You must be signed in to change notification settings - Fork 10.7k
[ADD] PLM: engineering change order doc #5238
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
Conversation
80e154b to
3ea0863
Compare
hojo-odoo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Felicious,
Good job on this. I've made a few comments that should be applied throughout the doc. Also, one of the .rst files is missing a blank line at the end.
Thanks,
hojo
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
9dae96c to
cb49c41
Compare
cb49c41 to
b48e20f
Compare
|
Hello @odoo/crm-doc-review (: This Product management lifecycle (PLM) doc is ready for the second round of peer review! If you get to this before 8/10, that's amazing because that's when I'll be asking an MRP expert from support to do another review. It'll be so nice if ya'll are able to help me make the doc sound snappier 😊 If not, that's okay! You'll be reviewing an even more accurate and detailed doc 😄 Thanks in advance for your help! |
odoo-shke
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
77118d2 to
bb8c26f
Compare
ksc-odoo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alrighty @Felicious just finished with my Peer Review - fantastic work! Only had a handful of minor suggestions - but other than that, everything looks great! Nice job! 👍
content/applications/inventory_and_mrp/manufacturing/management/bill_configuration.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
3d55248 to
6b95b31
Compare
|
Hi @ksc-odoo!! Thanks for reviewing my doc (: Do you see this message? |
cab41d7 to
ff4db9e
Compare
|
Hello @odoo/us-doc-review 😄 This PLM doc is ready for your review! Thank you in advance for your help. |
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/plm/engineering_change_orders.rst
Outdated
Show resolved
Hide resolved
|
@ksc-odoo @hojo-odoo think this could use another round of peer review so I'm kicking back to you. |
e4d74b0 to
95122af
Compare
StraubCreative
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Felicious approving with some small comments.
Merge when ready, thanks!
@robodoo delegate=Felicious
| inventory_and_mrp/barcode | ||
| inventory_and_mrp/quality | ||
| inventory_and_mrp/maintenance | ||
| inventory_and_mrp/plm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small thing I noticed:
The toctree links in this doc aren't exactly lined up with the indentation of the tag 😄
content/applications/inventory_and_mrp/manufacturing/management/bill_configuration.rst
Outdated
Show resolved
Hide resolved
| manage_changes/eco_type | ||
| manage_changes/engineering_change_orders |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would switch the order of these since ECO doc introducing the concept would ideally come first.
fwiw doc content is not organized alphabetically in English.
| Step` pop-up window. Doing so creates an instructional quality control point that will be | ||
| reviewed in the following section. | ||
|
|
||
| .. image:: engineering_change_orders/add-a-step.png |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this image compressed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, updating! thanks for catching that
| Materials` radio button is selected in the :guilabel:`Apply on` field, and the :guilabel:`Start | ||
| Revision` button has been pressed. | ||
|
|
||
| .. image:: engineering_change_orders/eco-form.png |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here— is this compressed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for catching that!
| Drag and reorder the `Check for broken switches` instruction, by clicking and dragging its "6 | ||
| dots" icon to move it from the bottom to the second position. | ||
|
|
||
| .. image:: engineering_change_orders/reorder.png |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this one doesn't appear to be compressed given how small the dimensions are
Co-authored-by: hojo-odoo <123424465+hojo-odoo@users.noreply.github.com> Co-authored-by: Sho Ketchum <96748231+odoo-shke@users.noreply.github.com> Co-authored-by: ksc-odoo <73958186+ksc-odoo@users.noreply.github.com> Co-authored-by: Zachary Straub <zst@odoo.com>
7da4352 to
20b555c
Compare
|
@robodoo r+ |
Summary
Create a new doc that details:
Looking for feedback (in particular) on document organizational structure: given the flow of using ECOs, do the section titles make sense? Are they in an order that makes most sense?
And of course, I welcome feedback on any wording, brevity, accuracy of the flow suggestions 😄