[asterisk-dev] [Code Review] 3008: app_confbridge: Add an option 'announce_join_leave_review' for users that adds recording review to the 'announce_join_leave' name recording feature

Jonathan Rose reviewboard at asterisk.org
Fri Nov 15 16:38:56 CST 2013

This is an automatically generated e-mail. To reply, visit:

(Updated Nov. 15, 2013, 4:38 p.m.)


This change has been marked as submitted.

Review request for Asterisk Developers, Mark Michelson and rmudgett.


Committed in revision 402854

Repository: Asterisk


Pretty straight forward really.  Adds the same basic functionality provided by the 'i' option to MeetMe into Confbridge (which already implemented functionality similar to the 'I' option through announce_join_leave)


  /trunk/configs/confbridge.conf.sample 402603 
  /trunk/apps/confbridge/include/confbridge.h 402603 
  /trunk/apps/confbridge/conf_config_parser.c 402603 
  /trunk/apps/app_confbridge.c 402603 
  /trunk/CHANGES 402603 

Diff: https://reviewboard.asterisk.org/r/3008/diff/


Made sure the recording feature worked with and without the review option in the expected way (with and without recording review). It's a pretty straight forward patch though.


Jonathan Rose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20131115/9ec037b0/attachment-0001.html>

More information about the asterisk-dev mailing list