From 1532a1bbb691edbdf40e78c70f2d5bb8e8ed000d Mon Sep 17 00:00:00 2001 From: Josh Stompro Date: Tue, 11 Nov 2014 13:07:36 -0600 Subject: [PATCH] LP#1124498 - Patron Notice when account expires - fixes and enhancements Changes from Michael Peters initial commit - Added repeat delay - Removed group_field since that doesn't make sense in this usage scenario. - Commented out the user param for email sender, not needed. - Removed error-to header, not commonly used anymore. - Added Auto-Submitted header since email sent out via this action are automatically generated. - Changed references to card expiring to account expiring. It is the patron's account that is expiring, not their card. - Fixed up the address lines so it tries to grab the mailing_address first, then the billing address. - Set the notices to be disabled by default. So it takes some admin action to enable. Signed-off-by: Josh Stompro Signed-off-by: Michele Morgan Signed-off-by: Ben Shum --- .../Pg/upgrade/XXXX.30day.expiration.prenotice.sql | 26 +++++++++++----------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.30day.expiration.prenotice.sql b/Open-ILS/src/sql/Pg/upgrade/XXXX.30day.expiration.prenotice.sql index 4b0c3959e2..4dbafd4df8 100644 --- a/Open-ILS/src/sql/Pg/upgrade/XXXX.30day.expiration.prenotice.sql +++ b/Open-ILS/src/sql/Pg/upgrade/XXXX.30day.expiration.prenotice.sql @@ -2,27 +2,27 @@ BEGIN; --create hook for actor.usr.expire_date INSERT INTO action_trigger.hook (key, core_type, description, passive) - VALUES ('expire', 'au', 'Card is expired', 't'); + VALUES ('expire', 'au', 'Account is expired', 't'); - --SQL to create event definition for 30 day card pre-expiration notice +--SQL to create event definition for 30 day account pre-expiration notice --Inactive, owned by top of org tree by default. Modify to suit needs. --Can set reactor to 'ProcessTemplate' for testing. Will generate emails in DB, but not actually send. -INSERT INTO action_trigger.event_definition (active, owner, name, hook, validator, reactor, delay, delay_field, group_field, max_delay, template) - VALUES ('t', '1', '30 Day Card Expiration Courtesy Notice', 'expire', 'NOOP_True', 'SendEmail', '-30 days', 'expire_date', 'usr', '-29 days', - $$ +INSERT INTO action_trigger.event_definition (active, owner, name, hook, validator, reactor, delay, delay_field, max_delay, repeat_delay, template) + VALUES ('f', '1', '30 Day Account Expiration Courtesy Notice', 'expire', 'NOOP_True', 'SendEmail', '-30 days', 'expire_date', '-29 days', '30 days', +$$ [%- USE date -%] [%- user = target -%] [%- lib = target.home_ou -%] To: [%- params.recipient_email || user.email %] From: [%- helpers.get_org_setting(target.home_ou.id, 'org.bounced_emails') || lib.email || params.sender_email || default_sender %] Reply-To: [%- helpers.get_org_setting(target.home_ou.id, 'org.bounced_emails') || lib.email || params.sender_email || default_sender %] -Errors-To: [%- helpers.get_org_setting(target.home_ou.id, 'org.bounced_emails') || lib.email || params.sender_email || default_sender %] -Subject: Courtesy Notice - Library Card Expiration in 30 days +Subject: Courtesy Notice - Library Account Expiration in 30 days +Auto-Submitted: auto-generated Dear [% user.first_given_name %] [% user.family_name %], -Our records indicate your library card is due to expire in 30 days. Please visit your local library at your convenience to renew your card in order to avoid a disruption in access to library service. +Our records indicate your library account is due to expire in 30 days. Please visit your local library at your convenience to renew your account in order to avoid a disruption in access to library service. Sincerely, [% lib.name %] @@ -32,9 +32,9 @@ Contact your library for more information: [% lib.name %] [%- SET addr = lib.mailing_address -%] [%- IF !addr -%] [%- SET addr = lib.billing_address -%] [%- END %] -[% lib.mailing_address.street1 %] [% lib.mailing_address.street2 %] -[% lib.mailing_address.city %], [% lib.mailing_address.state %] -[% lib.mailing_address.post_code %] +[% addr.street1 %] [% addr.street2 %] +[% addr.city %], [% addr.state %] +[% addr.post_code %] [% lib.phone %] $$); @@ -43,8 +43,8 @@ $$); SELECT currval('action_trigger.event_definition_id_seq'); --send from email address parameter -INSERT INTO action_trigger.event_params (event_def, param, value) - VALUES (CURRVAL('action_trigger.event_definition_id_seq'), 'sender_email', 'evergreen@example.org'); +--INSERT INTO action_trigger.event_params (event_def, param, value) +-- VALUES (CURRVAL('action_trigger.event_definition_id_seq'), 'sender_email', 'evergreen@example.org'); --insert environment values INSERT INTO action_trigger.environment (event_def, path) -- 2.11.0