[asterisk-dev] [Code Review] 4350: AMI: Add documentation for the missing Cdr/CEL events.

[Joshua Colp]

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

Ship it!


Ship It!

- Joshua Colp


On Jan. 17, 2015, 5:32 p.m., Matt Jordan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4350/
> -----------------------------------------------------------
> 
> (Updated Jan. 17, 2015, 5:32 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-24671
>     https://issues.asterisk.org/jira/browse/ASTERISK-24671
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This patch adds AMI event documentation for the Cdr and CEL AMI events.
> 
> Note that while these events do share fields with each other and with other channel related events, they do not contain all of the fields in a standard channel snapshot, nor is the description of the fields identical. As such, I went ahead and wrote a brief blurb for each field, for each event.
> 
> Not too many electrons were wasted in this effort.
> 
> 
> Diffs
> -----
> 
>   /branches/13/cel/cel_manager.c 430751 
>   /branches/13/cdr/cdr_manager.c 430751 
> 
> Diff: https://reviewboard.asterisk.org/r/4350/diff/
> 
> 
> Testing
> -------
> 
> Documentation is displayed correctly in 'manager show event <eventname>'.
> 
> 
> Thanks,
> 
> Matt Jordan
> 
>

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