[asterisk-dev] [Code Review] Custom devstate for ConfBridge conferences

[Jared Smith]

> On 2010-03-17 16:23:14, Sean Bright wrote:
> > trunk/apps/app_confbridge.c, line 468
> > <https://reviewboard.asterisk.org/r/572/diff/2/?file=8770#file8770line468>
> >
> >     My only other comment is that we choose the name ('confbridge') carefully so that we don't end up wanting to change it later but having to keep it because of backwards compat.  I don't have a preference one way or another, just some food for thought.

I thought a bit about this, but couldn't come up with something more descriptive than 'confbridge', as it matches the existing 'meetme' device state handler.  I'm open to other options, though.


> On 2010-03-17 16:23:14, Sean Bright wrote:
> > trunk/CHANGES, line 144
> > <https://reviewboard.asterisk.org/r/572/diff/2/?file=8769#file8769line144>
> >
> >     Minor nit - Tack a period on the end of the second sentence.  Looks like the surrounding entries do.

Done!


- Jared


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/572/#review1699
-----------------------------------------------------------


On 2010-03-17 17:03:33, Jared Smith wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/572/
> -----------------------------------------------------------
> 
> (Updated 2010-03-17 17:03:33)
> 
> 
> Review request for Asterisk Developers, Russell Bryant and Joshua Colp.
> 
> 
> Summary
> -------
> 
> This patch adds custom device state handling for ConfBridge conferences, matching the devstate handling of the MeetMe conferences, as explained in bug #16972.
> 
> 
> This addresses bug 16972.
>     https://issues.asterisk.org/view.php?id=16972
> 
> 
> Diffs
> -----
> 
>   trunk/CHANGES 253245 
>   trunk/apps/app_confbridge.c 253245 
> 
> Diff: https://reviewboard.asterisk.org/r/572/diff
> 
> 
> Testing
> -------
> 
> Added the patch, created a hint that referenced the custom devstate for a ConfBridge conference.  Checked the status of the hint using "core show hints" CLI command before, during, and after the audio conference.
> 
> 
> Thanks,
> 
> Jared
> 
>