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

docs_lp2053050_strict_barcode #251

Conversation

gmontimantis
Copy link
Contributor

Input for strict barcode info

Input for strict barcode info
@abneiman
Copy link
Member

Happy to commit this if someone more familiar with strict barcode would like to review & signoff.

@gmcharlt gmcharlt self-requested a review February 21, 2024 20:34
Copy link
Contributor

@gmcharlt gmcharlt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noting that one of the commits changes the line endings from Unix-style to DOS-style. This gets in the way of reviewing the changes.

Regarding the substance, I would suggest wording along the lines of:

  • The 'Strict Barcode' checkbox can appear next to barcode inputs on the checkout, checkin, renewal, and offline circulation pages.
  • If checked, it will examine the barcode value to see if it contains only digits and meets the "Mod10" checkdigit algorithm. This checkdigit algorithm is often used by library barcodes that use the "Codabar" symbology.
  • If the barcode does not meet the conditions, a "bad barcode" alert will be displayed. Staff are given the option to accept and continue using the barcode as input.
  • "Strict Barcode" is only useful for libraries whose barcodes use the Mod10 checkdigit algorithm. However, several other checkdigit algorithms and symbologies are in use by libraries.

(I also note that it does not check whether the barcode is exactly 14 digits long)

@abneiman
Copy link
Member

Committed to main & picked back to rel_3_10.

Thanks, Gina & Galen!

@abneiman abneiman closed this Mar 19, 2024
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.

None yet

3 participants