[Koha-bugs] [Bug 30555] Add more sample notice for sms messages

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30555

--- Comment #4 from David Nind <david at davidnind.com> ---
Hi Shi Yao Wang.

I think this is a great idea, but it needs a test plan. I've created a draft
test plan - I use koha-testing-docker for testing.

I think this enhancement will require a database update for existing
installations - where they don't have notices already, these should be updated.

I'm not sure about this and how it should be done, so please check with some
other developers. And apologies in advance if I'm not correct! The format or
database updates was recently updated, see
https://wiki.koha-community.org/wiki/Database_updates#How_to_write_an_atomicupdate_file

Draft test plan:
1. Check that there are no SMS messages for these notices (Tools > Notices &
slips):
   . CHECKIN - Item check-in (digest)
   . CHECKOUT - Item check-out (digest)
   . DUE - Item due reminder
   . DUEDGST - Item due reminder (digest)
   . PREDUE - Advance notice of item due
   . PREDUEDGST - Advance notice of item due (digest)
   . HOLD - Hold available for pickup
2. Apply the patch.
3. Update the database (updatedatabase when using koha-testing-docker) (I think
a database update is required for this patch to work correctly for existing
installations - maybe check with some other developers on how to do this. It
should check that if there are no SMS messages for the notices, then the new
default messages should be added. See
https://wiki.koha-community.org/wiki/Database_updates#How_to_write_an_atomicupdate_file
)
4. flush_memcached and restart_all (when using koha-testing-docker).
5. Check that there are now SMS notices for the notices listed in step 1.
6. To check a new installation:
   . reset_all (for koha-testing-docker)
   . login and check that there are now SMS messages for the notices listed in
step 1
7. Sign-off!

Additional item: I think the QA team will probably want the commit message
formatted correctly. In this case, the title should formatted with no space
after the bug number - Bug 30555: Add .... See
https://wiki.koha-community.org/wiki/Commit_messages

David

-- 
You are receiving this mail because:
You are watching all bug changes.