Skip to content

Conversation

@fw-bot
Copy link
Collaborator

@fw-bot fw-bot commented Nov 15, 2023

Revision of Average inventory valuation cost method.
Task ID: #3255051

Summary

Hi! I'm newly assigned to the inventory scope and wanted to update this inventory valuation doc for Odoo 14 and elaborate the accounting concepts in layman's terms. If you are able to fact-check and expand on the accounting concepts, I would greatly appreciate it!

Other changes

  • Add math formulas to elaborate the journal entry calculations about inventory valuation
  • Add more headings for organization (because the document got LONG very quickly)
  • Renamed headings to be action-oriented and sentence case
  • Add external links to related docs
  • Remove instances of "I", "our", "we" and other pronouns
  • Rewritten for clarity (not brevity)

Things to look out for in the review

(in order of importance)

  1. What do you think about the Anglo-Saxon section (lines 186-266)
  2. Does the definition of AVCO make sense?
  3. Are the accounting concepts clear & accurate?
  4. Do math explanations make sense?
  5. Can they be shortened a little?
  6. How is the organization of examples?
  7. Do intros of sections seem redundant?

This PR is intended to be FW ported

Forward-Port-Of: #6424
Forward-Port-Of: #4005

X-original-commit: f48a4e8
Co-authored-by: toaa <104567387+toaa-odoo@users.noreply.github.com>
Co-authored-by: meng-odoo <101904966+meng-odoo@users.noreply.github.com>
@robodoo
Copy link
Collaborator

robodoo commented Nov 15, 2023

@fw-bot
Copy link
Collaborator Author

fw-bot commented Nov 15, 2023

This PR targets saas-15.2 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@fw-bot
Copy link
Collaborator Author

fw-bot commented Nov 15, 2023

@Felicious @StraubCreative the next pull request (#6458) is in conflict. You can merge the chain up to here by saying

@fw-bot r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@StraubCreative
Copy link
Contributor

@fw-bot r+

robodoo pushed a commit that referenced this pull request Nov 15, 2023
closes #6457

X-original-commit: f48a4e8
Signed-off-by: Zachary Straub (zst) <zst@odoo.com>
Co-authored-by: toaa <104567387+toaa-odoo@users.noreply.github.com>
Co-authored-by: meng-odoo <101904966+meng-odoo@users.noreply.github.com>
@robodoo robodoo closed this Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants