We want to customize the default help message on the Self checkout page, as our library doesn't have bar codes and therefore doesn't have scanning. We only have a number that users input when using self checkout. The help message confused our users. This doesn't seem to be something that many libraries will have. Therefore as an easy solution, I propose to create a patch that adds a div with an id around the default help message. Then we can add our own help content in "SCOMainUserBlock" and hide the default message with CSS in "SCOUserCSS". Otherwise I could also see there being a new flag that specifies whether a library has scanning or not and changing the text based on that. That would require lots of translation effort that doesn't seem worth it to me. I will attach a patch to this with my proposed solution but I am also open to implementing other solutions.
Created attachment 143299 [details] [review] Proposed patch to add div with id to allow hiding it and customize with SCOMainUserBlock.
Created attachment 144089 [details] [review] Bug 32115: Add ID to check-out default help message dialog to allow customization This adds a div with the id attribute selfcheckdefaulthelpmessage to the first paragraph of the web based self check help page. To test: * Enable WebBasedSelfCheck * Go to: /cgi-bin/koha/sco/sco-main.pl in the OPAC * Log in and open help * Verify by looking at the source code, that the first paragraph starting with "If this is your first time..." now has an id. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Hi Mona, I believe this has been your first patch for Koha? Welcome! :)
Yes, this is my first patch as I only recently started to work at a library :) What are the next steps to get the patch merged?
This is signed-off now, which means you passed the first round of testing. Now it' s waiting for the QA team to have a look.
Created attachment 144497 [details] [review] Bug 32115: Add ID to check-out default help message dialog to allow customization This adds a div with the id attribute selfcheckdefaulthelpmessage to the first paragraph of the web based self check help page. To test: * Enable WebBasedSelfCheck * Go to: /cgi-bin/koha/sco/sco-main.pl in the OPAC * Log in and open help * Verify by looking at the source code, that the first paragraph starting with "If this is your first time..." now has an id. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Welcome aboard Mona, The patch is nice and simple and works well.. I'd love to see this block moved to a News block as that way it becomes easier for staff to change in the staff client and have translations working.. but that can be another bug. No regressions found and the QA script is happy. Passing QA. Congratulations on your first patch, now it's over to the Release Manager to get it pushed :)
Pushed to master for 23.05. Nice work everyone, thanks!
Strictly this one is an enhancement, but as it's so unobtrusive I'm opting to backport it.. welcome to the stables Mona :)
Nice work everyone! Pushed to 22.11.x for the next release
I tend to agree with Martin. Adding a class/ID is unobtrusive and helpful. Opting to backport, welcome to the oldstables Mona. :)
Backported to 22.05.x for upcoming 22.05.09
applied to 21.11.x for 21.11.16
Not backported to 21.05.x