Bug 27339 - Display OPAC notes and OPAC messages on check-in
Summary: Display OPAC notes and OPAC messages on check-in
Status: Failed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Didier Gautheron
QA Contact: Testopia
URL:
Keywords: release-notes-needed
Depends on:
Blocks:
 
Reported: 2021-01-05 08:55 UTC by Didier Gautheron
Modified: 2021-02-08 08:26 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 27339: Display opac notes and opac messages on check-in (3.54 KB, patch)
2021-01-05 10:15 UTC, Didier Gautheron
Details | Diff | Splinter Review
Bug 27339: Display opac notes and opac messages on check-in (3.59 KB, patch)
2021-02-01 10:11 UTC, PTFS Europe Sandboxes
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Didier Gautheron 2021-01-05 08:55:38 UTC
Currently, Circulation note (borrowers.borrowernotes) and item notes (items.itemnotes and items.itemnotes_nonpublic) are displayed during check-in.
    
This patch adds Opac note (borrowers.opacnote) and Opac messages (messages table).
    
Test plan:
- Add an Opac note and one or several Opac messages to a patron.
- Checkout and check-in an item for this patron.
- Check that Opac note and Opac message(s) are not displayed on check-in.
- Apply the patch.
- Checkout and check-in an item for this patron.
- Check that Opac note and Opac message(s) are displayed on check-in.
Comment 1 Didier Gautheron 2021-01-05 10:15:47 UTC
Created attachment 114856 [details] [review]
Bug 27339: Display opac notes and opac messages on check-in

.
Adds Opac note (borrowers.opacnote) and Opac messages (messages table) during check-in.

Test plan:
1- Add an Opac note and one or several Opac messages to a patron.
2- Checkout and check-in an item for this patron.
3- Check that Opac note and Opac message(s) are not displayed on check-in.
4- Apply the patch.
5- Checkout and check-in an item for this patron.
6- Check that Opac note and Opac message(s) are displayed on check-in.
Comment 2 Lucas Gass 2021-01-22 21:40:59 UTC
Hi Didier,

I'm just curious why we display the OPAC message but not the staff internal - message here?
Comment 3 PTFS Europe Sandboxes 2021-02-01 10:11:44 UTC
Created attachment 116144 [details] [review]
Bug 27339: Display opac notes and opac messages on check-in

 .
Adds Opac note (borrowers.opacnote) and Opac messages (messages table) during check-in.

Test plan:
1- Add an Opac note and one or several Opac messages to a patron.
2- Checkout and check-in an item for this patron.
3- Check that Opac note and Opac message(s) are not displayed on check-in.
4- Apply the patch.
5- Checkout and check-in an item for this patron.
6- Check that Opac note and Opac message(s) are displayed on check-in.

Signed-off-by: Stina Hallin <stina.hallin@ub.lu.se>
Comment 4 Katrin Fischer 2021-02-07 11:52:53 UTC
Hi Didier, I can see that you were active on the patch, but the given author is Author: Matthias Meusburger <matthias.meusburger@biblibre.com>. Is this correct?
Comment 5 Katrin Fischer 2021-02-07 11:58:45 UTC
I like that you have given each note an individual class, that would make it possible to hide things for libraries as these notes could get a bit much for some depending on their use.

Now one note is still missing: The staff message. I think these might often be used to give some kind of history about the patron so could really turn into too much information here. I assume libraries are more likely to remove OPAC messages, that are no longer needed. So I am ok with that.
Comment 6 Katrin Fischer 2021-02-07 12:02:40 UTC
I have been looking at this a bit longer... and I'd suggest to create a new column for the OPAC notes in the table and add it to the column configuration.

With the current solution we cannot see the context of the notes, I think this would make things clearer. What do you think?
Comment 7 Stina Hallin 2021-02-08 08:26:35 UTC
I think your suggestion is good. You might end up with too many notes in one column, and some of them might not be important in the checkin workflow. Best if the coulmn only is visible if there are any notes to display.