[Home]

Summary:ASTERISK-15659: [patch] Set conterence options on pin-less RealTime conferences
Reporter:dea (dea)Labels:
Date Opened:2010-02-18 15:47:45.000-0600Date Closed:2010-04-02 18:56:46
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_meetme
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) rt-meetme-options.txt
Description:RealTime conferences can use different flags/options depending on
the pin entered (user/admin).  However it is also legal to have pin-less
conferences, and flags/options should be supported in such a case.


****** ADDITIONAL INFORMATION ******

The first hunk in the patch is a loosely related change to create a new application variable MEETMEBOOKID to export a RealTime conferences bookId for use in CDR records
Comments:By: Leif Madsen (lmadsen) 2010-03-24 06:58:41

It may also be useful to put this on reviewboard. If there is a ship-it given to it, then I could commit it. Of course anyone who can test or review the code is of course appreciated.

By: Russell Bryant (russell) 2010-03-25 12:22:45

I just gave it a "ship it".  I'd be okay with providing commit access to allow DEA to commit his own patches in the future, if you'd like.

By: Leif Madsen (lmadsen) 2010-03-31 10:22:00

DEA: if you'd like commit access, then please contact Russell as he'll have to gather some information from you to generate a certificate.

By: dea (dea) 2010-03-31 11:54:56

I appreciate the offer and am honored, but to be honest job focus changes
have me performing next to no coding on Asterisk.  If that changes, I will
contact Russell to work out the details.

By: Digium Subversion (svnbot) 2010-04-02 18:55:58

Repository: asterisk
Revision: 256019

U   trunk/apps/app_meetme.c

------------------------------------------------------------------------
r256019 | russell | 2010-04-02 18:55:57 -0500 (Fri, 02 Apr 2010) | 10 lines

Export MEETMEBOOKID and fix pin-less conferences with realtime conferences

(closes issue ASTERISK-15659)
Reported by: DEA
Patches:
     rt-meetme-options.txt uploaded by DEA (license 3)
Tested by: DEA

Review: https://reviewboard.asterisk.org/r/582/

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=256019

By: Digium Subversion (svnbot) 2010-04-02 18:56:46

Repository: asterisk
Revision: 256020

_U  branches/1.6.2/
U   branches/1.6.2/apps/app_meetme.c

------------------------------------------------------------------------
r256020 | russell | 2010-04-02 18:56:45 -0500 (Fri, 02 Apr 2010) | 17 lines

Merged revisions 256019 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r256019 | russell | 2010-04-02 18:55:57 -0500 (Fri, 02 Apr 2010) | 10 lines
 
 Export MEETMEBOOKID and fix pin-less conferences with realtime conferences
 
 (closes issue ASTERISK-15659)
 Reported by: DEA
 Patches:
       rt-meetme-options.txt uploaded by DEA (license 3)
 Tested by: DEA
 
 Review: https://reviewboard.asterisk.org/r/582/
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=256020