Opt-in setting for overdue/predue emails

[Jeff Davis]

It would be nice if there were a standard user setting that allowed users to opt out of receiving email notifications about overdue items, so that libraries don't spam their patrons with unwanted emails.

Action trigger opt-in settings make this very easy to implement -- in fact, we've had a setting like this in place for years at Sitka, and I imagine we're not the only ones. I'll share a version of what we're doing. All that's needed is to define a user setting and then set it as the opt_in_setting for the relevant a/t event definitions.

[Jeff Davis (jdavis-sitka)]

Working branch user/jeffdavis/lp1915219-overdue-email-opt-in-setting has an implementation:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1915219-overdue-email-opt-in-setting

[Garry Collum (gcollum)]

Jeff. This would be a nice feature, and could be easily extended to SMS, as well.

In your instructions for the event_definition the query reads "opac.default_circ_notices_enabled" instead of "circ.default_overdue_notices_enabled".

[Jeff Davis (jdavis-sitka)]

Whoops! Thanks for catching that, Garry. I have force-pushed a fix to the same branch, so the correct setting name should be used everywhere.

[Garry Collum (gcollum)]

Hi Jeff,

I get a foreign key constraint error when I test the seed-values file. '(grp)=(circ) is not present in table "settings_group"'.

It looks like the config.settings_group values are defined right after your code to insert the usr_setting_type.

[Jeff Davis (jdavis-sitka)]

Sheesh. Thanks again, Garry. I've pushed an additional commit that removes the (nonexistent) settings group from the user setting, consistent with most other such settings. I've also confirmed that the seed data actually gets loaded correctly now.

[Garry Collum (gcollum)]

Perfect. A sign-off branch is at https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/gcollum/lp1915219-overdue-email-opt-in-setting-signoff

[Galen Charlton (gmc)]

Pushed to master for inclusion in 3.7. Thanks, Jeff and Garry!