hotwax / receiving

HotWax Commerce Receiving App
https://receiving.hotwax.io/
Apache License 2.0
8 stars 40 forks source link

Barcode Scanning: Concatenating SKU Entries Instead of Overwriting #367

Closed Adameddy26 closed 6 months ago

Adameddy26 commented 7 months ago

Note

This issue can only be replicated when a physical barcode scanner is used to scan the product.

Expected behavior

Upon scanning a new product barcode, the scan barcode field should display the SKU of the newly scanned product, overwriting any previously displayed SKU.

Steps to reproduce the issue

Open the system interface. Navigate to the section where barcode scanning is utilized. Scan the barcode of a product (Product A). Observe that the SKU of Product A is displayed in the scan barcode field. Scan the barcode of another product (Product B). Note that instead of replacing the existing SKU, the SKU of Product B is appended to the previous SKU, resulting in an incorrect concatenated SKU.

Environment details

App version : V2.25.0 App Environment : UAT

amansinghbais commented 7 months ago

Here in this issue, we are using a barcode reader as a keyboard input for the ion-input we have created for scanning product SKU. Now the barcode reader works such as, it takes the value of the barcode as a keyboard input in the ion-input and then hit the enter event triggering to scan item.

The logic currently we have appends the new SKU with the old scanned SKU present in the ion-input. To avoid the following scenario, i have identified following way:--

Adameddy26 commented 6 months ago

Issue verified and working fine in release v2.26.0. Hence, closing it.

Adameddy26 commented 1 month ago

Changes related to toast message are verified and working fine in release v2.27.0.