-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[ADD] inventory: stock report #9289
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
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 @Felicious - great new doc! I like the idea for the structure, having reporting being a whole seciton with each report having it's own doc. I do think there's some room for expansion and to link to more docs/sections of docs. It references lots of types of things which maybe a user might need more info on. When these are all publishe,d this will be a great section! I am approving now since I think you can make these suggestions if you want- nothing is requied, just things I htink could make this a bit 'beefire' or add some more info. But again, they are not necessary, just suggesitons. lMK if you have any quesitons!
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Outdated
Show resolved
Hide resolved
a722db1
to
d34abed
Compare
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 @Felicious
I think this is a good start to a series of reporting docs we can build out for Inventory and the other materials apps. Good UI and menu item coverage here, nice job 👍
What you have here should get users started with understanding the UI right away, which is helpful! Later, for this particular dashboard, I believe there are some deep use cases involving specific filters, that we will either want to add to this doc to continue building it out, or can make new docs.
Couple quick comments for you below. Once those are addressed you can merge 🚀
.....
@robodoo delegate=Felicious
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/warehouses_storage/reporting/stock.rst
Show resolved
Hide resolved
Co-authored-by: larm-odoo <121518652+larm-odoo@users.noreply.github.com> Co-authored-by: Zachary Straub <zst@odoo.com>
c6dbd7d
to
f28d10a
Compare
@robodoo r+ |
@Felicious this pull request has forward-port PRs awaiting action (not merged or closed): |
1 similar comment
@Felicious this pull request has forward-port PRs awaiting action (not merged or closed): |
Summary
New doc explaining what the Stock report displays.
Note to reviewers: As this is the first of all Inventory reporting docs, let me know if you agree/disagree on where these docs are located in the toc tree!
Task