#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>

Reply via email to