[IMP] mrp: Work order dependencies planning views#17829
Conversation
|
|
|
@theRealThagomizer This PR is ready for review when you have a moment. The 19.2 update changes the planning reports in a way that was hard to figure out how to document, but this was the most straightforward way I could see to make sure all of the nuances were captured. Let me know what you think! |
theRealThagomizer
left a comment
theRealThagomizer
left a comment
There was a problem hiding this comment.
Hiya, @erjer-odoo! Review complete. As is so often the case, the writing was great, but I had some minor suggestions you can take or leave about phrasing and some musings about the optimal way to convey the information. There's nothing here that required a massive rewrite, so I'd say this is ready to move forward once you review the feedback and decide what you want to do with it. Thanks!
Co-authored-by: theRealThagomizer <96515928+theRealThagomizer@users.noreply.github.com>
erjer-odoo
force-pushed
the
saas-19.2-mrp-work-order-dependencies-views-erjer
branch
from
c6bf2d9 to
ce2f88f
Compare
|
@Felicious This PR is ready for final review when you get a moment. Thank you! |
Felicious
left a comment
Felicious
left a comment
There was a problem hiding this comment.
Incredible work @erjer-odoo ! The overall structure of this page is so easy to navigate 😊
@robodoo r+
closes #17829 Signed-off-by: Felicia Kuan (feku) <feku@odoo.com> Co-authored-by: theRealThagomizer <96515928+theRealThagomizer@users.noreply.github.com>
4 participants
Task card.
Updated to bring current with the 19.2 design of the planning views.
This saas-19.2 PR should not be FWP. There are further changes in 19.3 that warrant their own update (TK).