-
Notifications
You must be signed in to change notification settings - Fork 10.7k
[ADD] MRP: One-step manufacturing #5158
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
833ca8d to
6a04cb4
Compare
Felicious
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.
Hi @hojo-odoo !
I just finished my peer review and it's really a clear and straight-to-the-point document! I definitely have a lot to learn from you in terms of writing good documentation. I just had a few minor suggestions, and everything else seemed clear to me when I followed the flow. Great job!
Also loved the 16.3 screenshots, they look so clean (:
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
fddd63c to
36aaa59
Compare
Felicious
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.
Amazing job, @hojo-odoo!
Thank you for addressing my suggestions, the doc looks flawless to me 😄
Warm regards,
Felicia
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.
@hojo-odoo Great work on this doc. Only had a handful of minor adjustments that need your attention -- but once those are addressed, I think this will be ready for the next stage. Nicely done! 👍
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
36aaa59 to
e2b291a
Compare
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/manufacturing/management/one_step_manufacturing.rst
Outdated
Show resolved
Hide resolved
|
@robodoo delegate=hojo-odoo |
e2b291a to
512481c
Compare
|
@robodoo r+ |
|
@hojo-odoo unable to stage: merge conflict |
512481c to
ac95d3a
Compare
|
@robodoo r+ |
closes #5158 Signed-off-by: John Holton (hojo) <hojo@odoo.com>
|
@hojo-odoo this pull request has forward-port PRs awaiting action (not merged or closed): |
2 similar comments
|
@hojo-odoo this pull request has forward-port PRs awaiting action (not merged or closed): |
|
@hojo-odoo this pull request has forward-port PRs awaiting action (not merged or closed): |
|
@hojo-odoo this pull request has forward-port PRs awaiting action (not merged or closed): |
This PR adds a new doc detailing how to manufacture products using one step.
NOTE: There is a ref tag in this doc that does not currently work because the doc it is referencing is not yet live. The doc being referenced should be live by the time this doc is published, at which point, the ref tag will work.