| Summary: | Port default RENEWAL notice to Template Toolkit syntax | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Emily Lamancusa (emlam) <emily.lamancusa> |
| Component: | Notices | Assignee: | Emily Lamancusa (emlam) <emily.lamancusa> |
| Status: | Needs documenting --- | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | fridolin.somers |
| Version: | Main | Keywords: | Academy, roadmap_24_05 |
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | String patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: |
The default notice template for the RENEWAL notification now uses modern Template Toolkit rather than the old custom syntax. This will only affect new installations, unless the library chooses to reset their notices to the default.
|
Version(s) released in: |
24.05.00
|
| Circulation function: | |||
| Bug Depends on: | |||
| Bug Blocks: | 15278 | ||
| Attachments: |
Bug 36113: Adjust default RENEWAL notice template to use TT
Bug 36113: Adjust default RENEWAL notice template to use TT Bug 36113: Adjust default RENEWAL notice template to use TT |
||
|
Description
Emily Lamancusa (emlam)
2024-02-15 22:02:04 UTC
Created attachment 164896 [details] [review] Bug 36113: Adjust default RENEWAL notice template to use TT Test plan: 1. Apply patch 2. reset_all (or start your testing environment in the first place) 3. Enable the system preference RenewalSendNotice 4. Check an item out to a patron 5. Edit that patron: a. Enter a value for Primary email b. Set messaging preference to enable email notices for Item checkout and renewal 6. Renew the checked-out item 7. Check the patron's Notices tab for the Item renewal notice, and confirm that the notice is correct Created attachment 165069 [details] [review] Bug 36113: Adjust default RENEWAL notice template to use TT Test plan: 1. Apply patch 2. reset_all (or start your testing environment in the first place) 3. Enable the system preference RenewalSendNotice 4. Check an item out to a patron 5. Edit that patron: a. Enter a value for Primary email b. Set messaging preference to enable email notices for Item checkout and renewal 6. Renew the checked-out item 7. Check the patron's Notices tab for the Item renewal notice, and confirm that the notice is correct Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Created attachment 166411 [details] [review] Bug 36113: Adjust default RENEWAL notice template to use TT Test plan: 1. Apply patch 2. reset_all (or start your testing environment in the first place) 3. Enable the system preference RenewalSendNotice 4. Check an item out to a patron 5. Edit that patron: a. Enter a value for Primary email b. Set messaging preference to enable email notices for Item checkout and renewal 6. Renew the checked-out item 7. Check the patron's Notices tab for the Item renewal notice, and confirm that the notice is correct Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> This one is pretty basic and uses a different "signature" than others. Might also be something that could get prettified a little bit :) Maybe everytime we curerntly use biblio.title it would be worthwhile to use our include for the nice title display instead, that also pulls subject, parts etc. I did wonder about using nice includes for display yes.. but I also didn't want to get stuck in the weeds of new wording... It's also not marked as HTML.. and our include helpers are generally designed for HTML output.. so we'd likely want to upgrade it to HTML at the same time ;P Pushed for 24.05! Well done everyone, thank you! Not backported to 23.11.x |