[Home]

Summary:ASTERISK-02453: [patch] Eject last user
Reporter:outtolunc (outtolunc)Labels:
Date Opened:2004-09-23 14:32:49Date Closed:2008-01-15 15:21:40.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_meetme
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_meetme.diff
( 1) app_meetme.diff
( 2) press-6-to-eject.gsm
Description:added option 'e' to MeetMeAdmin
added dtmf '6' to admin menu  (*6 (86) get it <G>)

the option to press '6' is not in the conf-adminmenu sound file.  if this feature is added allison will need to make another.

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

be sure you (as admin) are NOT the last one that entered that conference, it does not currently care. <G>
Comments:By: James Golovich (jamesgolovich) 2004-09-23 14:43:04

How about making it check to make sure the user your are going to kick isnt the user doing the kicking?  Should be pretty easy to do, probably just strcmp usr->chan->name and chan->name and if they match dont do it

By: outtolunc (outtolunc) 2004-09-23 16:09:22

added, currently plays conf-invalid, if admin was the last in and attempts to kick himself.

By: Mark Spencer (markster) 2004-10-03 00:25:31

Can you put the script we need so we can allison to do the recordings?

By: Mark Spencer (markster) 2004-10-22 15:24:26

test

By: outtolunc (outtolunc) 2004-10-22 15:36:38

changed conf-invalid to conf-errormenu
-----------
conf-adminmenu currently says

press 1 to mute or unmute yourself,
or press 2 to lock or unlock the conference.

need to add
or press 6 to eject last user

By: Russell Bryant (russell) 2004-11-07 19:56:29.000-0600

updates?

-- Housekeeping

By: outtolunc (outtolunc) 2004-11-08 12:14:11.000-0600

this is just waiting for allison to add
----------------------------
conf-adminmenu currently says

press 1 to mute or unmute yourself,
or press 2 to lock or unlock the conference.

need to add
or press 6 to eject last user

you could always add the patch, it just wouldn't tell you the option 6 is there, it still would work.

By: outtolunc (outtolunc) 2004-11-12 14:13:04.000-0600

{tap} {tap} testing 1 2 3

By: Russell Bryant (russell) 2004-11-14 22:40:09.000-0600

Allison isn't going to do this on her own.  Someone has to pay her to do it.  Either you need to pay for the needed prompt or find someone to sponsor it.

By: andrewkt (andrewkt) 2004-11-15 08:39:22.000-0600

Can this not be committed as is?

We can poke bkw (or someone else) and get him to take up a round of donations for some more Allison recordings like was done earlier this year.

If bkw doesn't want to be involved, and can forward me a copy of the email instructions he sent to her for the last batch, I will make the arrangements. The only thing with me is, I can't do the chopping/splicing/etc afterwards.

By: Russell Bryant (russell) 2004-11-15 08:54:18.000-0600

The needed prompt would cost $12 US through digium.  See http://thevoice.digium.com to have a prompt made.

If you get the prompt, you can post it here.  You can try to find someone on IRC who will take a few minutes to take care of it for you.

Thanks!

By: outtolunc (outtolunc) 2004-11-15 11:06:47.000-0600

drumkilla, please understand that i'm not in a very good mood today...

but, if you could read you would have seen this statement my mark himself

markster 10-03-04 00:25  Can you put the script we need so we can allison to do the recordings?

so, please refrain from commenting on this bug.

By: twisted (twisted) 2004-11-17 00:03:43.000-0600

Let's try to refrain from flaming someone who's trying to point you in the right direction to get things moving along.  We operate in a thankless position, and frankly, more along the lines of custodial duties at times, so having someone come down on us for simply just pointing you in the right direction to move along shouldn't warrant this.   Just an FYI, nothing personal at all.

Again, Thanks for the contribution!

By: Russell Bryant (russell) 2004-11-17 00:06:06.000-0600

Woah, woah ... I'm not trying to upset anyone.  I'm just trying to make sure that everything comes together so this patch can make it in as soon as possible.  I was not pointing those comments to anyone in particular.

That was just for anyone who might read through and want to know what needs to be done to get it into CVS.

Sorry for any confusion.

By: Mark Spencer (markster) 2004-11-17 00:07:52.000-0600

Out of curiosity, why are we skipping from 2 to 6 in the menu?

By: outtolunc (outtolunc) 2004-11-17 00:24:10.000-0600

the reason i went for '6' was...
"added dtmf '6' to admin menu (*6 (86) get it <G>)"

so people could remember it without having to listen to the whole prompt.

it's been a really long few weeks for me.  sorry for being a <blank>

By: twisted (twisted) 2004-12-01 01:13:11.000-0600

We may be better off sticking to the order of things, however, I don't really think it should matter which option this is, as it's still a good addition.

Does this still apply to cvs?  Are the recordings done?

I'm marking this acknowledged for now...

By: twisted (twisted) 2004-12-15 20:40:28.000-0600

Two weeks ago I asked for some status information, and have yet to receive a response.   This is going into feedback required status, which means we must get updated or it will be closed due to lack of interest.  I would really hate to see that happen.

--Housekeeping

By: outtolunc (outtolunc) 2004-12-16 11:50:48.000-0600

i doubt it matters, drop it.

By: mochouinard (mochouinard) 2004-12-29 20:28:22.000-0600

ASTERISK-2352382 is doing most of what this patch does no ?

edited on: 01-07-05 00:12

By: outtolunc (outtolunc) 2004-12-30 01:58:51.000-0600

the reason for this patch was simple.  when you have a conference going and a user joins and the room quality goes to ****, we wanted a way to 'easily' kick that user.

plain and simple.  

thats all it does, cycle to last user in the conf list and set a kick flag.

By: mochouinard (mochouinard) 2004-12-30 08:20:48.000-0600

ha ok I see, sorry, didnt read, make sence

By: robf (robf) 2005-01-07 16:14:39.000-0600

I've attached the requested audio prompt.

By: twisted (twisted) 2005-01-13 22:21:59.000-0600

Added to cvs with slight modification - Thanks!

By: Mark Spencer (markster) 2005-01-13 22:36:29.000-0600

This should be option '3' to be consecutive.

By: Mark Spencer (markster) 2005-01-13 23:06:24.000-0600

Okay set back to option '3'.  I feel so leet, having used "dd" to fix the audio to avoid transcoding.

By: Digium Subversion (svnbot) 2008-01-15 15:21:35.000-0600

Repository: asterisk
Revision: 4785

U   trunk/apps/app_meetme.c
U   trunk/sounds/conf-adminmenu.gsm
U   trunk/sounds.txt

------------------------------------------------------------------------
r4785 | twisted | 2008-01-15 15:21:35 -0600 (Tue, 15 Jan 2008) | 2 lines

Add meetme option to kick last user (bug ASTERISK-2453 with slight modification)

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

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

By: Digium Subversion (svnbot) 2008-01-15 15:21:40.000-0600

Repository: asterisk
Revision: 4790

U   trunk/apps/app_meetme.c
U   trunk/sounds/conf-adminmenu.gsm

------------------------------------------------------------------------
r4790 | markster | 2008-01-15 15:21:39 -0600 (Tue, 15 Jan 2008) | 2 lines

Change option '6' back to more logical option '3' (bug ASTERISK-2453)

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

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