-
Notifications
You must be signed in to change notification settings - Fork 9.2k
[ADD] barcode: operations and commands barcode printing #13355
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
base: 18.0
Are you sure you want to change the base?
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.
Great work so far, @slinkous ! 😊 The doc is really in-depth and it's going to be SO helpful! 😍
I know you haven't requested a review from me, yet! I just did a quick scan 👀 😉 to catch some quick formatting and style things. I presumed that since you pushed this PR up at the end of the day, you'll still have it open when you come in tomorrow, so it'll be easy to do some fixes before you move onto another task 😊
The :guilabel:`Internal Transfers (WHINT)` operation barcode is available if the database is using | ||
the ::guilabel:`Storage Locations` to designate where products are located in the inventory. To |
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.
super nitpicky but we only use the guilabel tag when that field is currently visible on the UI. In this intro sentence, we haven't navigated to a specific page yet, so it's not yet on the reader's page. So, we use italics 😊
The :guilabel:`Internal Transfers (WHINT)` operation barcode is available if t 8000 he database is using | |
the ::guilabel:`Storage Locations` to designate where products are located in the inventory. To | |
The *Internal Transfers (WHINT)* operation barcode is available if the database is using | |
the *Storage Locations* to designate where products are located in the warehouse. To |
Note: extra :
in front of storage locations, and maybe missing the word "feature"?
optional: I'm not really sure if "products are located in the inventory" is a common phrase? How about "in the warehouse", or "in inventory" (without "the")?
The :guilabel:`Internal Transfers (WHINT)` operation barcode is available if the database is using | ||
the ::guilabel:`Storage Locations` to designate where products are located in the inventory. To | ||
enable this setting, go to :menuselection:`Inventory app --> Configuration --> Settings`, then, in | ||
the :guilabel:`Warehouse` section, tick the :guilabel:`Storage Locations` option. |
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.
guilabels properly used here! 😊
============================ | ||
|
||
Inventory operations, like :doc:`receipts and delivery | ||
orders<../../inventory/shipping_receiving/daily_operations/receipts_delivery_one_step>`, as well |
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.
orders<../../inventory/shipping_receiving/daily_operations/receipts_delivery_one_step>`, as well | |
orders <../../inventory/shipping_receiving/daily_operations/receipts_delivery_one_step>`, as well |
we need one space between link text and link (:
|
||
*Operation types* in the **Inventory** and **Barcode** apps define actions that will change the | ||
state of products in inventory. What operation types are available will depend on the settings which | ||
are enabled in the database. By default, barcodes can be printed for the :guilabel:`Receipts |
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.
are enabled in the database. By default, barcodes can be printed for the :guilabel:`Receipts | |
are enabled in the database. By default, barcodes can be printed for :guilabel:`Receipts |
maybe an extra "the"? Also see my later comment about possibly changing these guilabels to italics if the operation types aren't visible on the screen (:
.. warning:: | ||
Printing barcodes, by default, will either directly download a PDF file of the selected barcodes | ||
or open a new tab with a downloadable or printable PDF. To turn this into a direct printing | ||
operation, :doc:`connect a printer<../../../general/iot/devices/printer>`. |
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.
operation, :doc:`connect a printer<../../../general/iot/devices/printer>`. | |
operation, :doc:`connect a printer <../../../general/iot/devices/printer>`. |
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.
super minor nitpicks! planning to send this to tech review next so i'd like to take a look before then (:
but wonderful job!!! really helpful & comprehensive doc
are enabled in the database. By default, barcodes can be printed for the :guilabel:`Receipts | ||
(WHIN)`, :guilabel:`Delivery Orders (WHOUT)`, and :guilabel:`Manufacturing (WHMANUF)`. The | ||
:guilabel:`Internal Transfers (WHINT)` and :guilabel:`Internal Transfers (WHINT)` and | ||
:guilabel:`Pick (WHPICK)` operations can also be enabled. |
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.
all should change to italics or stay unformatted since we cant see them on the screen (:
------- | ||
|
||
The :guilabel:`Pick (WHPICK)` operation barcode is available if the database is using the | ||
::guilabel:`Multi-Step Routes` feature configured with a picking step. To enable this setting, go to |
There was a problem hiding this comment.
Ch 8000 oose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
italics
<../../inventory/shipping_receiving/daily_operations/receipts_delivery_two_steps>` | ||
- :doc:`Three-step delivery | ||
<../../inventory/shipping_receiving/daily_operations/delivery_three_steps>` | ||
|
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.
extra line
-------- | ||
|
||
- :guilabel:`MAIN MENU`: Scan to return to the main menu from an inventory adjustment. | ||
- :guilabel:`DISCARD`: TODO |
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.
let's delete this and all the other todos and revisit in follow-up PRs
- :guilabel:`DISCARD`: TODO |
- :guilabel:`DISCARD`: TODO | ||
- :guilabel:`VALIDATE`: Scan to confirm that an operation is correct and ready to be completed. | ||
- :guilabel:`CANCEL`: Scan from an operation to prevent that operation from being validated and set | ||
its status to "Canceled". |
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.
its status to "Canceled". | |
its status to *Canceled*. |
No description provided.