[ADD] barcode: serial numbers and lots

[slinkous]

• Coverage for printing and managing barcoders for product serial numbers and lot numbers

[robodoo]

[slinkous]

@Felicious this is going to fail CI checks because it's not in the tree, but both my current PR's will go to the same place, so I just need to figure out which one of them gets approved first.

[Felicious]

Fantastic work with this doc, @slinkous!

It's a very comprehensive doc that covers a variety of use cases and considers many tedious tasks a warehouse user might face, including getting blocked when a shipping label is damaged during transit! That's something I would've never thought of! I can tell you did a lot of research from the master class and watching BE's OXP videos from the previous 2-3 years 😊

I just had some minor nitpicks on wording in this review, and it's ready to go to the next stage after looking through them!

In the case where there are merge conflicts (one will occur in setup.rst), we write in the PR description which one is meant to be merged first. After we merge the first PR, we'll rebase the second PR, resolve the merge conflicts, push the second PR back up, wait for the tests to finish, then merge the second!

After addressing my comments, let's just add the serial_numbers_lots.rst file to setup.rst, so the checks pass for the next reviewer (:

[Felicious]

Hi @slinkous ! Due to the changes to final review, I'd like to take a quick review on this PR after you address my comments, then we'll pass it to sam for technical review (: sound good? :D

[slinkous]

was there a particular reason why the instructions below are about enabling mandatory scan, but the image has optional scan highlighted? For consistency, it might be better for the image to match the instructions and the heading of the section

The instructions are about selecting the appropriate setting, it's just that the UI calls this Mandatory Scan. Renaming section to clairfy.

[slinkous]

was there a particular reason why the instructions below are about enabling mandatory scan, but the image has optional scan highlighted? For consistency, it might be better for the image to match the instructions and the heading of the section

The instructions are about selecting the appropriate setting, it's just that the UI calls this Mandatory Scan. Renaming section to clairfy.

[Felicious]

LTGM @slinkous !

There's only an extra blank link so moving this onto you, @samueljlieber !

[slinkous]

This is ready for TR, @samueljlieber tyty

[samueljlieber]

Hi @slinkous! Awesome work on this doc, it is a great addition to this scope! I like how you structured the information– it is a nice read :)

Approving with a couple comments on the images, please address before merge. Thank you!
.....
@robodoo delegate=slinkous

[samueljlieber]

Rename to lowercase, could even remove since the written instruction is sufficent :)

- serial_numbers_lots/enable-GS1-barcodes.png
+ serial_numbers_lots/enable-gs1-barcodes.png

[samueljlieber]

Please remove this image (content/applications/inventory_and_mrp/barcode/setup/serial_numbers_lots/disable-serials-lots-for-op.png)

[slinkous]

@robodoo r+