-
Notifications
You must be signed in to change notification settings - Fork 10.8k
[ADD] inventory: addition logistics dashboards kpis #14802
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
916882f to
70b3648
Compare
70b3648 to
58556c8
Compare
58556c8 to
2878e0c
Compare
|
Hi @Felicious, A few points to note about this PR that might help your review:
Looking forward to hearing your thoughts on this PR, as it's possibly an approach we'll take for some other important Odoo dashboards in the future, like the accounting ones. |
2878e0c to
c252d33
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.
Wonderful and detailed work, @mcsi-odoo !
I just had really minor suggestions and a quick question. All edits are optional, as you have a clear understanding of the subject matter, and that's the most important thing! I agreed with all of your changes to rename dashboard to report, and feel free to merge this whenever!
@robodoo delegate+
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/aging.rst
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Show resolved
Hide resolved
mcsi-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.
Thanks for the feedback and the delegation, @Felicious !
I accepted pretty much all your comments, and I'm following up with ANLE on the Avg Delay and Avg Lead Time points in the Operations analysis dashboard.
Hope to get this merged soon!
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/dashboards.rst
Outdated
Show resolved
Hide resolved
c252d33 to
eef1328
Compare
|
@robodoo r+ |

task-5122922