<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/2906/">https://reviewboard.asterisk.org/r/2906/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Dan Jenkins wrote:
{quote}
David and I spoke about this at Astricon, seems like a good idea to me; if people aren't closed to making such a "big" change to the interface.
{quote}
While I don't want to turn the next few months into a "let's redo everything" - that's why we have code reviews and a process in the first place - I'm all for using the time 12 is in beta as an opportunity to get the API right before it is officially released. Without commenting on whether or not this change is the right thing to do, I'm good with changes like this before 12 is released.</pre>
<br />
<p>- Matt</p>
<br />
<p>On October 15th, 2013, midnight UTC, Paul Belanger wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Paul Belanger.</div>
<p style="color: grey;"><i>Updated Oct. 15, 2013, midnight</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">While spending my time with ARI over the last few days, I've been seeing some room for improvements. For example, today we explicitly return a user to pass toggle actions via the URL. I believe HTTP has a built-in function that will help make this easier. So, I am proposing we use the POST / DELETE functions from HTTP to handle some toggle events.
This change updates how we toggle a hold and moh, to be much cleaner IMO for HTTP.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">local development box</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>trunk/res/res_ari_channels.c <span style="color: grey">(400880)</span></li>
<li>trunk/rest-api/api-docs/channels.json <span style="color: grey">(400880)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2906/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>