#242: errorlib: make SMS sending facility more generic
--------------------------+-------------------------------------------------
Reporter: simko | Owner: jluker
Type: enhancement | Status: assigned
Priority: critical | Milestone: v1.0
Component: MiscUtil | Version:
Resolution: | Keywords:
--------------------------+-------------------------------------------------
Changes (by simko):
* status: in_merge => assigned
Comment:
Thanks. As we spoken IRL, please make the following changes:
(i) simplify the config to use strings, not lists:
{{{
CFG_SITE_ADMIN_EMERGENCY_CONTACTS = {
'06:00-18:00': '[email protected],[email protected]',
'18:00-06:00': '[email protected]',
'Sunday 22:00-06:00': '[email protected]',
}
}}}
which will also take care of the default mutable argument issue
(`recipients=[]` -> `recipients=''`); (ii) put a descriptive
self-documented example into invenio.config, but use empty dict for
the default value; (iii) remove old CFG_SITE_EMERGENCY_PHONE_NUMBERS
from inveniocfg.py special case treatment; (iv) fix trailing
whitespace cosmetics.
--
Ticket URL: <http://invenio-software.org/ticket/242#comment:3>
Invenio <http://invenio-software.org>