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

Mark Michelson reviewboard at asterisk.org
Fri Nov 14 09:30:07 CST 2014


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


This review has been shipped. I wrote the feature against trunk, but I'm curious if 13's proposed policy would allow for such a change to be added to it as well. Thoughts?

- Mark Michelson


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/20141114/dc351153/attachment.html>


More information about the asterisk-dev mailing list