Skip to content

Improve handling of 'self checkout' action in backoffice loan page#659

Merged
kpsherva merged 2 commits intoinveniosoftware:masterfrom
JakobMiesner:feature/self-checkout-fixes
Sep 17, 2025
Merged

Improve handling of 'self checkout' action in backoffice loan page#659
kpsherva merged 2 commits intoinveniosoftware:masterfrom
JakobMiesner:feature/self-checkout-fixes

Conversation

@JakobMiesner
Copy link
Copy Markdown
Contributor

Omit option to self_checkout on loan with no items in backoffice. Previously this produced the following error, as the system does not know which item to 'self checkout'.
Screenshot 2025-08-26 at 11 15 23

Also changed formatting of the 'self_checkout' action button text from capitalize first letter to start case
Screenshot 2025-08-26 at 11 16 49

* using the action on a loan with no items produced a TransitionConstraintsViolationError
* instead of captializing the first letter, start case is used
@JakobMiesner JakobMiesner marked this pull request as ready for review August 26, 2025 09:22
@kpsherva kpsherva merged commit 64fcae7 into inveniosoftware:master Sep 17, 2025
1 check passed
@JakobMiesner JakobMiesner deleted the feature/self-checkout-fixes branch October 7, 2025 09:20
@salomerohr
Copy link
Copy Markdown

@JakobMiesner

  1. We are not sure to understand the error thing. Could you please explain it to us? It also displays the message when an item is available.
  2. Librarians won't do 'self-checkout' so the button is misleading in backoffice. To avoid wrong actions, it should be disabled?Librarians are only doing regular check out for patrons.

@JakobMiesner
Copy link
Copy Markdown
Contributor Author

JakobMiesner commented Oct 27, 2025

Hi @salomerohr, the deploy to sandbox unfortunately did not go through for the this issue. I will fix it as soon as possible and let you know when this ticket can be tested again! :)

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.

3 participants