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

Entering a patron barcode doesn't work when the checkin form contains one or more lines #2735

Closed
pronguen opened this issue Mar 1, 2022 · 0 comments · Fixed by rero/rero-ils-ui#834
Assignees
Labels
bug Breaks something but is not blocking client request Issue reported by production libraries f: circulation Concerns the circulation interface or backend p-High High priority (to be solved in the 2-3 next months)
Milestone

Comments

@pronguen
Copy link
Contributor

pronguen commented Mar 1, 2022

Bug description:

Entering a patron barcode doesn't work when the checkin form contains one or more lines.

Expected behavior:

Entering a patron barcode always opens directly the checkout form for this patron

  • except if an item has exactly the same barcode: then it displays a modal (already implemented)

Steps to Reproduce:

  1. Open the page /professional/circulation/checkout
  2. Do a checkin of an item on loan
  3. Then in the same page enter a patron barcode
  4. See the flash message

Context

Server:

Version:

v1.9.0

@pronguen pronguen added bug Breaks something but is not blocking f: circulation Concerns the circulation interface or backend p-High High priority (to be solved in the 2-3 next months) labels Mar 1, 2022
@PascalRepond PascalRepond added the client request Issue reported by production libraries label Mar 8, 2022
@pronguen pronguen added this to the v1.11.0 milestone Mar 16, 2022
@vgranata vgranata self-assigned this Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Breaks something but is not blocking client request Issue reported by production libraries f: circulation Concerns the circulation interface or backend p-High High priority (to be solved in the 2-3 next months)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants