[asterisk-dev] [Code Review] 4167: Allow mis-dialed DTMF-initiated transfers to be re-attempted.

Joshua Colp reviewboard at asterisk.org
Thu Nov 13 10:31:07 CST 2014


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

Ship it!


Ship It!

- Joshua Colp


On Nov. 12, 2014, 6:32 p.m., Mark Michelson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4167/
> -----------------------------------------------------------
> 
> (Updated Nov. 12, 2014, 6:32 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> There have been complaints about the fact that when a DTMF transfer is initiated, and an incorrect number is dialled, the prompt says "Please try again" but then doesn't give the transferer a chance to do so without re-initiating the transfer.
> 
> This patch adds a features.conf option called "transferdialattempts" (yes, the name is gross, but it fits in with the convention of other options in features.conf). This option controls how many attempts a transferer has to dial a valid extension before being booted back out to the call with the transferee. The option defaults to 3. As with other options from features.conf, this can also be set using the FEATURE() dialplan function so that it may be changed per-call if desired.
> 
> 
> Diffs
> -----
> 
>   /trunk/main/features_config.c 427676 
>   /trunk/main/bridge_basic.c 427676 
>   /trunk/include/asterisk/features_config.h 427676 
> 
> Diff: https://reviewboard.asterisk.org/r/4167/diff/
> 
> 
> Testing
> -------
> 
> See /r/4168
> 
> 
> Thanks,
> 
> Mark Michelson
> 
>

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


More information about the asterisk-dev mailing list