Skip to content
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

Multiple suggestions #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

khoivan88
Copy link
Contributor

Hi Felix,
Here is some explanation for this pull request.

  • fixed bug in Terminal mode: barcodeTerminalAsync.php and lib_language_en.php
    while doing inventory for a container (inventory mode or "Set storage
    for all following containers"), if you scan a non-existing barcode,
    the location will be removed. When a non-existent barcode is scanned,
    an error pop-up window appears.

  • modified History log text to add storage_name; also added History log text
    when changing storage in edit mode (lib_db_manip.php, lib_db_manip_edit.php)

  • import.php, lib_import.php: added importing function for locations and
    users using tab-separated text file

  • lib_import.php: fix for importing chemical_storage_barcode bug.
    When import tab-dilimited text file of chemical containers, if the
    barcode column is the last column, it will add white space or \n
    character, making the barcodes inaccurate. The fix will trim all the
    white space (\t\n) on the right side of the input column

  • lib_language_en.php, sidenav.php, barcode_autogeneration.php:
    Creating option for admin user to auto generate all location and
    user barcodes while using "Existing barcodes" functions

  • lib_db_manip.php: edit logging text to reflect chemical containers when
    being moved from one location to another

  • Fixed "Set storage for all following containers" in Terminal

  • Added barcode Type 128 generation for user using existing barcode

  • import.php, lib_import.php: Fixed added order_date and open_date in
    Import tab-separated text file function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant