[asterisk-dev] [Code Review] 4095: Add a prompt to be read to the "winner" when a call is connected through app_followme

[rmudgett]

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



/trunk/CHANGES
<https://reviewboard.asterisk.org/r/4095/#comment24754>

    Replace followme-connecting-prompt with connecting_prompt.
    
    The followme-connecting-prompt isn't mentioned anywhere in the sample conf file.



/trunk/apps/app_followme.c
<https://reviewboard.asterisk.org/r/4095/#comment24751>

    With the previous change you don't need a silence/0 sound file.  If there is no sound file configured then nothing is played.
    
    With silence/0 as the default you now have to have silence/0 installed which doesn't seem to exist.



/trunk/configs/samples/followme.conf.sample
<https://reviewboard.asterisk.org/r/4095/#comment24752>

    Change to:
    
    connecting_prompt=>
    ;connecting_prompt=>tt-weasels
    ; The global default sound file name for 'Please say hello to the caller' message.  Setting to an empty string skips playing the prompt.  The default is no prompt file name.



/trunk/configs/samples/followme.conf.sample
<https://reviewboard.asterisk.org/r/4095/#comment24753>

    Change to:
    
    connecting_prompt=>
    ;connecting_prompt=>tt-weasels
    ; The sound file name for 'Please say hello to the caller' message.  Default is the global default.


- rmudgett


On Jan. 21, 2015, 6:23 p.m., Graham Mainwaring wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4095/
> -----------------------------------------------------------
> 
> (Updated Jan. 21, 2015, 6:23 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-24372
>     https://issues.asterisk.org/jira/browse/ASTERISK-24372
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This patch adds the ability to configure a prompt which will be read to the "winner" who pressed 1 (or the configured value) and received the call.
> 
> The context of this is that I have users who only receive outbound follow-me calls occasionally. These users have complained that when they are connected to the inbound caller, they don't always remember to say hello as if the call just arrived, so there is an awkward moment where the two parties figure out they are connected.
> 
> My prompt here says "Connecting. After the beep, please say hello to the caller. <beep>." This reminds the follow-me winner that they ought to say hello.
> 
> I have configured this with a default of silence/0 so that the system's behavior will not change for anyone who does not have this configured.
> 
> 
> Diffs
> -----
> 
>   /trunk/configs/samples/followme.conf.sample 430919 
>   /trunk/apps/app_followme.c 430919 
>   /trunk/CHANGES 430919 
> 
> Diff: https://reviewboard.asterisk.org/r/4095/diff/
> 
> 
> Testing
> -------
> 
> I have been running this in production for about three years with no issues. I have not done any sort of regression testing against other Asterisk features or in configurations other than my own production server.
> 
> 
> Thanks,
> 
> Graham Mainwaring
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150122/79564cab/attachment.html>