[asterisk-dev] [Code Review] LibPRI changes for Generic AOC

[David Vossel]

> On 2010-04-14 16:04:16, rmudgett wrote:
> > /team/rmudgett/aoc_event/libpri.h, lines 1564-1568
> > <https://reviewboard.asterisk.org/r/619/diff/1/?file=9261#file9261line1564>
> >
> >     Another parameter needs to be added to specify a bias for units or currency.  These messages should be consistent since the units and currency messages can encode charging info not available and free of charge.
> >     
> >     More discussion is needed as to whether we remember what has happened before or this is a configurable option.

This has not been addressed yet.  It hasn't been forgotten.


- David


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


On 2010-04-20 12:51:37, David Vossel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/619/
> -----------------------------------------------------------
> 
> (Updated 2010-04-20 12:51:37)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This review compliments the Asterisk changes in review https://reviewboard.asterisk.org/r/552/.  These are the LibPRI changes required to send ETSI AOC messages generated by Asterisk.
> 
> 
> Diffs
> -----
> 
>   /team/rmudgett/aoc_event/pri_facility.h 1641 
>   /team/rmudgett/aoc_event/pri_facility.c 1641 
>   /team/rmudgett/aoc_event/pri_internal.h 1641 
>   /team/rmudgett/aoc_event/q931.c 1641 
>   /team/rmudgett/aoc_event/libpri.h 1641 
>   /team/rmudgett/aoc_event/pri_aoc.c 1641 
> 
> Diff: https://reviewboard.asterisk.org/r/619/diff
> 
> 
> Testing
> -------
> 
> These routines have been tested back to back with Asterisk, but I am going to need some community help to test interoperability.
> 
> To test this code, check out both my Asterisk and Libpri changes for AOC
> 
> Asterisk Changes: svn/asterisk/team/dvossel/generic_aoc
> LibPRI Changes: svn/libpri/team/dvossel/aoc_send 
> 
> 
> Thanks,
> 
> David
> 
>