The setting's label in administration > patron categories is missing a colon at the end: Can be guarantee:
Created attachment 156012 [details] [review] Bug 34446: Added missing colon to Can be guarantee To test: 1. Go to Administration > patron categories and edit or create a new category. Notice that “Can be guarantee” does not have a colon. 2. Apply patch and refresh the page. 3. Can be guarantee now has a colon — yay! 4. Sign off and have a great day. :)
Hi Laura, your patch is missing the nice test plan you wrote - I believe you might have added it in the 'git bz attach' patch. This only adds it to bugzilla, but not to your patch file. You need to add it with "git commit".
Created attachment 156021 [details] [review] Bug 34446: Added missing colon to Can be guarantee To test: 1. Go to Administration > patron categories and edit or create a new category. Notice that “Can be guarantee” does not have a colon. 2. Apply patch and refresh the page. 3. Can be guarantee now has a colon — yay! 4. Sign off and have a great day. :)
Created attachment 156022 [details] [review] Bug 34446: (QA follow-up) Add colons on delete confirm page This adds the missing colon on the 'confirm deletion' page.
Using my super powers :) - PQA
(In reply to Katrin Fischer from comment #2) > Hi Laura, your patch is missing the nice test plan you wrote - I believe you > might have added it in the 'git bz attach' patch. This only adds it to > bugzilla, but not to your patch file. You need to add it with "git commit". I'll make sure to do that next time. Thank you, Katrin!
Pushed to master for 23.11. Nice work everyone, thanks!
Pushed to 23.05.x for 23.05.05
Nice work everyone! Pushed to oldstable for 22.11.x