[asterisk-dev] [Code Review] 2626: ARI POST /channels: Add support for sending an originated call to a context/extension/priority.

[Paul Belanger]

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



/trunk/res/stasis_http/resource_channels.c
<https://reviewboard.asterisk.org/r/2626/#comment17723>

    trivial comment, wouldn't it be more consistent to add the logic to check for context being empty up above where you did it for timeout?
    
    Same comment about priority.
    
    Either way, dealers choice.


- Paul Belanger


On June 25, 2013, 8:29 p.m., Jason Parker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2626/
> -----------------------------------------------------------
> 
> (Updated June 25, 2013, 8:29 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-21658
>     https://issues.asterisk.org/jira/browse/ASTERISK-21658
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> Add a way to send the call to a CEP.
> 
> This removes the Local channel madness that existed before, since it didn't make much sense.  The context/extension were re-purposed to be the secondary destination.
> 
> 
> Diffs
> -----
> 
>   /trunk/res/res_stasis_http_channels.c 392899 
>   /trunk/res/stasis_http/resource_channels.h 392899 
>   /trunk/res/stasis_http/resource_channels.c 392899 
>   /trunk/rest-api/api-docs/channels.json 392899 
> 
> Diff: https://reviewboard.asterisk.org/r/2626/diff/
> 
> 
> Testing
> -------
> 
> Calls work to either an app in Stasis or an extension (and fail if neither are specified).
> 
> 
> Thanks,
> 
> Jason Parker
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130628/95c4d3de/attachment.htm>