documentation icon indicating copy to clipboard operation
documentation copied to clipboard

[REF] inventory: create inventory adj section

Open Felicious opened this issue 1 year ago • 3 comments

Summary of changes

  • Move inventory adjustment docs into its own section, as per @cru-odoo's request.
  • Add redirects to moved files

Reviewers

  • Since this PR is just to move files, no need to review for content (: If you have any suggestions about what other docs could go in this new inventory adjustment section or the order of the docs, feel free to share here!

FWport

Yes, all the way to master !

Felicious avatar Mar 18 '24 21:03 Felicious

@odoo/inventory-doc-review Hello! This PR requires a different kind of review this time. I didn't change any files-- only moved inventory adjustment docs into its own section, so you don't need to review for content. Please let me know if you have any suggestions for doc structure:

  • Name of the new section I'm making
  • Order of the docs
  • Order of the inventory adjustment sectrion within the Warehouse/Storage parent folder
  • Any thoughts are welcome!

Felicious avatar Mar 18 '24 22:03 Felicious

Hi @ksc-odoo, @StraubCreative, and @jcs-odoo! I'm thinking of moving these inventory adjustment docs to their own section. What do you all think? (:

Felicious avatar Mar 21 '24 23:03 Felicious

Hi @ksc-odoo, @StraubCreative, and @jcs-odoo! I'm thinking of moving these inventory adjustment docs to their own section. What do you all think? (:

Hi :)

Good idea. And I was about to suggest the same as @samueljlieber :) It's best to have content in the category pages. And even better if the content in it helps navigate to the sub-pages.

The commit message and PR title should rather have the tag MOV instead of REF. REF is rather for changes in the code. For structural changes, MOV is the right one.

Cheers :)

jcs-odoo avatar Mar 26 '24 14:03 jcs-odoo

Hi @samueljlieber ! Thank you for the suggestions to improve the searchability of the docs during your first review. This PR is ready for you to take another look at (:

So, it turns out, I misunderstood @cru-odoo a little-- she meant that she wanted all of the inventory adjustments grouped together, but they don't need to have their own, named section. So, I made the necessary adjustments to the redirects.txt and folders. Let me know what you think! (:

Felicious avatar Apr 08 '24 18:04 Felicious

Appreciate your help with this so much, @samueljlieber ! Thank you for catching that redirect link. I applied your suggestions and restored the previous link. It's ready for another look!

Felicious avatar Apr 09 '24 21:04 Felicious

@robodoo r+

samueljlieber avatar Apr 09 '24 22:04 samueljlieber

Forward-porting to 'master' (from odoo/documentation#8629).

fw-bot avatar Apr 10 '24 12:04 fw-bot

Forward-porting to 'master' (from odoo/documentation#8630).

fw-bot avatar Apr 10 '24 12:04 fw-bot