Allow users to add notes when placing a hold in opac.
Created attachment 15752 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac The option of adding a note is controlled by new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: 1 Verify if new pref is disabled by default. Place a hold. You can't add a note. 2 Enable the pref. Place a hold and add a note. Check in staff if you can see the note in Catalogue Detail/Holds tab. 3 Toggle SingleBranchmode, AllowHoldDateInFuture/OPACAllowHoldDateInFuture, OPACShowHoldQueueDetails, or OPACItemHolds. Check the display of columns when placing a hold from opac. 4 Place a few holds with notes from opac search results in one run (enable DisplayMultiPlaceHold). Check results in staff again. Remark: A few lines already refer to mandatory note reasons. This is handled in a subsequent report. No reason to worry.
Created attachment 15753 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac Database revision for report 9722, adding new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: Run [your server]/cgi-bin/koha/installer/install.pl?step=3&op=updatestructure Optionally, run a new install to test sysprefs.sql. Or: Edit sysprefs.sql, leave at least the last few lines including the new one. And run from command line: mysql -p [yourdatabase] < [your-clone]/installer/data/mysql/sysprefs.sql
Created attachment 15806 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac Database revision for report 9722, adding new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: Run [your server]/cgi-bin/koha/installer/install.pl?step=3&op=updatestructure Optionally, run a new install to test sysprefs.sql. Or: Edit sysprefs.sql, leave at least the last few lines including the new one. And run from command line: mysql -p [yourdatabase] < [your-clone]/installer/data/mysql/sysprefs.sql Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 15807 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac The option of adding a note is controlled by new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: 1 Verify if new pref is disabled by default. Place a hold. You can't add a note. 2 Enable the pref. Place a hold and add a note. Check in staff if you can see the note in Catalogue Detail/Holds tab. 3 Toggle SingleBranchmode, AllowHoldDateInFuture/OPACAllowHoldDateInFuture, OPACShowHoldQueueDetails, or OPACItemHolds. Check the display of columns when placing a hold from opac. 4 Place a few holds with notes from opac search results in one run (enable DisplayMultiPlaceHold). Check results in staff again. Remark: A few lines already refer to mandatory note reasons. This is handled in a subsequent report. No reason to worry. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 15808 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac Database revision for report 9722, adding new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: Run [your server]/cgi-bin/koha/installer/install.pl?step=3&op=updatestructure Optionally, run a new install to test sysprefs.sql. Or: Edit sysprefs.sql, leave at least the last few lines including the new one. And run from command line: mysql -p [yourdatabase] < [your-clone]/installer/data/mysql/sysprefs.sql Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Looks good but a couple things that should probably be taken care of before passing qa: 1) The "Show notes" buttons should probably be titled "Add notes" instead. 2) When doing a multi-hold, clicking one "Show notes" button hides the previously visible note textarea. This is jarring and may confuse patrons into thinking they can only have a note on one and only one of the holds. It would be better to make each "Show notes" button a toggle to hide it's own note textarea, and clear that textarea if it gets hidden. You could even change the text from "Add notes" to "Remove notes" when the textarea is visible.
Created attachment 15834 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac (dbrev) Database revision for report 9722, adding new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: Run [your server]/cgi-bin/koha/installer/install.pl?step=3&op=updatestructure Optionally, run a new install to test sysprefs.sql. Or: Edit sysprefs.sql, leave at least the last few lines including the new one. And run from command line: mysql -p [yourdatabase] < [your-clone]/installer/data/mysql/sysprefs.sql Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 15836 [details] [review] Bug 9722: Toggle display of hold notes with Edit notes button Small adjustments, requested by Kyle: 1) Rename Show notes button. It is called now: Edit notes. 2) Do not hide all other notes fields. The Edit button now only toggles the corresponding hold notes field.
(In reply to comment #6) > 1) The "Show notes" buttons should probably be titled "Add notes" instead. Changed it to Edit notes. > 2) When doing a multi-hold, clicking one "Show notes" button hides the > previously visible note textarea. This is jarring and may confuse patrons > into thinking they can only have a note on one and only one of the holds. It > would be better to make each "Show notes" button a toggle to hide it's own > note textarea, and clear that textarea if it gets hidden. You could even > change the text from "Add notes" to "Remove notes" when the textarea is > visible. The button now only toggles its own textarea. I do not prefer to remove the notes with one click. Shift-select or Ctrl+A plus delete does the job well enough :) Thanks for testing.
Created attachment 17717 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac The option of adding a note is controlled by new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: 1 Verify if new pref is disabled by default. Place a hold. You can't add a note. 2 Enable the pref. Place a hold and add a note. Check in staff if you can see the note in Catalogue Detail/Holds tab. 3 Toggle SingleBranchmode, AllowHoldDateInFuture/OPACAllowHoldDateInFuture, OPACShowHoldQueueDetails, or OPACItemHolds. Check the display of columns when placing a hold from opac. 4 Place a few holds with notes from opac search results in one run (enable DisplayMultiPlaceHold). Check results in staff again. Remark: A few lines already refer to mandatory note reasons. This is handled in a subsequent report. No reason to worry. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 17718 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac (dbrev) Database revision for report 9722, adding new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: Run [your server]/cgi-bin/koha/installer/install.pl?step=3&op=updatestructure Optionally, run a new install to test sysprefs.sql. Or: Edit sysprefs.sql, leave at least the last few lines including the new one. And run from command line: mysql -p [yourdatabase] < [your-clone]/installer/data/mysql/sysprefs.sql Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 17719 [details] [review] Bug 9722: Toggle display of hold notes with Edit notes button Small adjustments, requested by Kyle: 1) Rename Show notes button. It is called now: Edit notes. 2) Do not hide all other notes fields. The Edit button now only toggles the corresponding hold notes field. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Rebased three patches (small merge conflicts). Kyle: The third patch is more or less a "string patch", but could formally use your signoff too. Could you add that please? Thanks.
Created attachment 17726 [details] [review] Bug 9722: Toggle display of hold notes with Edit notes button Small adjustments, requested by Kyle: 1) Rename Show notes button. It is called now: Edit notes. 2) Do not hide all other notes fields. The Edit button now only toggles the corresponding hold notes field. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
QA comment: Good integration into a really ugly code/page. 2 small remarks: - Maybe the new pref should be under the OPAC/Appearance instead of Features. All OPACShow* pref are under appearande. - The toggle stuff can be made with a simple $("#notesrow_"+biblioNum).toggle(); not blocker Marked as Passed QA.
Created attachment 18342 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac The option of adding a note is controlled by new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: 1 Verify if new pref is disabled by default. Place a hold. You can't add a note. 2 Enable the pref. Place a hold and add a note. Check in staff if you can see the note in Catalogue Detail/Holds tab. 3 Toggle SingleBranchmode, AllowHoldDateInFuture/OPACAllowHoldDateInFuture, OPACShowHoldQueueDetails, or OPACItemHolds. Check the display of columns when placing a hold from opac. 4 Place a few holds with notes from opac search results in one run (enable DisplayMultiPlaceHold). Check results in staff again. Remark: A few lines already refer to mandatory note reasons. This is handled in a subsequent report. No reason to worry. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 18343 [details] [review] Bug 9722: Allow users to add notes when placing a hold in opac (dbrev) Database revision for report 9722, adding new pref OpacShowHoldNotes. This development is part of a larger one (see umbrella report 9721). Test plan: Run [your server]/cgi-bin/koha/installer/install.pl?step=3&op=updatestructure Optionally, run a new install to test sysprefs.sql. Or: Edit sysprefs.sql, leave at least the last few lines including the new one. And run from command line: mysql -p [yourdatabase] < [your-clone]/installer/data/mysql/sysprefs.sql Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 18344 [details] [review] Bug 9722: Toggle display of hold notes with Edit notes button Small adjustments, requested by Kyle: 1) Rename Show notes button. It is called now: Edit notes. 2) Do not hide all other notes fields. The Edit button now only toggles the corresponding hold notes field. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Pushed to master along with a follow up to allow the hold note to be displayed in the patron summary in the OPAC (as it doesn't make sense to allow patrons to set a note without being able to view it later, particularly if it's meant to later contain notes about requesting a specific part). Thanks, Marcel!
(In reply to comment #19) > Pushed to master along with a follow up to allow the hold note to be > displayed in the patron summary in the OPAC (as it doesn't make sense to > allow patrons to set a note without being able to view it later, > particularly if it's meant to later contain notes about requesting a > specific part). Thanks for that, Galen!
(In reply to comment #15) > - Maybe the new pref should be under the OPAC/Appearance instead of > Features. All OPACShow* pref are under appearande. Thanks for QAing, Jonathan. I thought about moving it. But I think that this pref that allows or not allows adding notes should be under Features. But I agree that you have a point about the name of the pref. Probably, the name is somewhat misleading. Do you have a suggestion? Would OpacHoldNotes be an improvement?
(In reply to comment #15) > - The toggle stuff can be made with a simple > $("#notesrow_"+biblioNum).toggle(); Submitted small followup on bug 10321 for that. Thx.
(In reply to comment #21) > (In reply to comment #15) > > - Maybe the new pref should be under the OPAC/Appearance instead of > > Features. All OPACShow* pref are under appearande. > > Thanks for QAing, Jonathan. I thought about moving it. But I think that this > pref that allows or not allows adding notes should be under Features. But I > agree that you have a point about the name of the pref. Probably, the name > is somewhat misleading. > Do you have a suggestion? Would OpacHoldNotes be an improvement? OpacHoldNotes is probably more expressive.