[Home]

Summary:ASTERISK-09142: [patch] Added a talk request for user in mute
Reporter:Ian Esper (imesper)Labels:
Date Opened:2007-03-29 11:44:52Date Closed:2007-10-25 14:30:19
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_meetme
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_meetme_v2.patch
( 1) app_meetme_v3.patch
( 2) app_meetme.patch
Description:First: I couldn't understand why app_meetme uses f = ast_read_noaudio(c), it kept from being able to accessing menu while in mute.
Second: I added the capability for user in mute to request talking, it reflects on cli.  
Comments:By: Serge Vecher (serge-v) 2007-03-29 12:19:18

Could you please get a disclaimer on file (see bottom of http://bugs.digium.com/main_page.php) and confirm with a note here when done?

also, the patch needs to be against trunk, not 1.4.
By: Ian Esper (imesper) 2007-03-29 13:02:26

The new patch is against trunk
The disclaimer will be send next week, I can't do it right now (no fax machine)
By: Ian Esper (imesper) 2007-03-29 13:05:06

The new patch is against trunk app_meetme.c Revision 58895
By: Ian Esper (imesper) 2007-04-03 10:03:48

Disclaimer sent 03/04/2007 by fax
By: Joshua C. Colp (jcolp) 2007-06-13 14:22:53

Can you clarify the issue with ast_read_noaudio you saw? I was unable to reproduce it and I would like to conquer that first.
By: Ian Esper (imesper) 2007-06-13 14:55:03

Hi, when I was in a conference put myself on mute, I couldn't listen to musiconhold and could not listen to menu again.
By: Joshua C. Colp (jcolp) 2007-06-18 13:04:05

Mute on the phone or through the Meetme IVR Menu?
By: Ian Esper (imesper) 2007-06-18 20:41:51

Mute on ivr or cli, I will test on the current svn branch this week to check if it is still occurring.
thanks
By: Jason Parker (jparker) 2007-06-26 14:50:28

Would you be willing to move the DTMF part of this issue over to 10065?

This patch would only be able to go into trunk, and would not fix the issue in 1.2 or 1.4.
By: Ian Esper (imesper) 2007-06-26 14:58:11

Yes, no problem moving it. It is exactly the same thing, but it never happened in 1.2
By: Ian Esper (imesper) 2007-07-18 16:23:21

What happened with the  Attached Files? I've already sent the disclaimer
By: Russell Bryant (russell) 2007-07-18 23:31:38

The attached patch is visible again.  Sorry for the confusion.
By: Jared Smith (jsmith) 2007-07-19 07:32:42

If I can make a suggestion, I'd also like to see a Manager event happen when the user requests to talk.  This way, people who manage their conferences via the Manager interface have the same information that shows on the CLI.
By: Ian Esper (imesper) 2007-10-05 08:30:20

I did a manager event (app_meetme_v2.patch). I really don't know if was the best way to do it and I reverted the changes on ast_read_noaudio(), it was fixed as reported on bug 10065.
By: Russell Bryant (russell) 2007-10-25 13:46:03

Your second patch doesn't compile.  :)  It's a trivial fix, though ... s/TALKREQUEST/T_REQUEST/
By: Digium Subversion (svnbot) 2007-10-25 13:57:49

Repository: asterisk
Revision: 87040

U   trunk/apps/app_meetme.c

------------------------------------------------------------------------
r87040 | russell | 2007-10-25 13:57:49 -0500 (Thu, 25 Oct 2007) | 9 lines

Add support for a muted user to request to talk.  The '2' option in the user
menu will adjust this status if a user is muted.  The talk request status will
be reflected in the CLI commands as well as the manager interface.

(closes issue ASTERISK-9142)
Reported by: imesper
Patches:
     app_meetme_v2.patch uploaded by imesper (license 275)

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