<div dir="ltr">In the future, can you please respond through reviewboard, so that email filters can tag these properly?<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 24, 2013 at 6:11 PM, Olle E. Johansson <span dir="ltr"><<a href="mailto:oej@edvina.net" target="_blank">oej@edvina.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div>25 okt 2013 kl. 00:50 skrev "Paul Belanger" <<a href="mailto:reviewboard@asterisk.org" target="_blank">reviewboard@asterisk.org</a>>:</div>
<div class="im"><br><blockquote type="cite"><pre style="font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:-webkit-auto;text-indent:0px;text-transform:none;word-spacing:0px;white-space:pre-wrap;word-wrap:break-word">
My only comment is about the naming conventions, we should try and keep AMI events in sync.  Right now we have FooBegin / FooEnd, and FooStart / FooStop, and Foo / FooComplete.  I'd like us see about pick one format and sticking with it when possible.

I believe Begin / End has been the most recent addition, do we want to default to that?</pre></blockquote></div></div>I like better the design we decided upon in AMI 1.1 - to have ONE event name and a status field that tells the app if it's start/stop or being/end or something else. We have tried to stop FooBegin/FooEnd and similar constructs for a long time.<div>
<br></div><div>Like </div><div><br></div><div>Event: foo</div><div>Status: begin</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>/O</div></font></span></div><br>--<br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br></blockquote></div><br></div>