<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/3117/">https://reviewboard.asterisk.org/r/3117/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 9th, 2014, 7:38 p.m. CST, <b>Paul Belanger</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://reviewboard.asterisk.org/r/3117/diff/2/?file=51373#file51373line52" style="color: black; font-weight: bold; text-decoration: underline;">/branches/12/rest-api/api-docs/mailboxes.json</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">52</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="tb"> </span><span class="tb"> </span><span class="tb"> </span><span class="tb"> </span><span class="tb"> </span><span class="nt">"responseClass"</span><span class="p">:</span> <span class="s2">"void"</span><span class="p">,</span></pre></td>
</tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Think it would be good to return the mailbox here, in case the user wanted to confirm the settings.</pre>
</blockquote>
<p>On January 10th, 2014, 2:08 p.m. CST, <b>Jonathan Rose</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The precedent set by device states would suggest that I shouldn't do that. On the other hand, when POST is used to create a channel, the channel object is returned. Personally, I think it's a bit of a tossup. I could go either way on this... but if we do want to change PUT to return the mailbox here, we should probably do the same for device state.</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Returning the object that was just changed shouldn't be necessary here.
If you told it to change the state, and ARI returned a 2xx, then it better have changed the state. If ARI returned any other response code, it better not have changed the state.
I'm not sure that returning the object here adds much value, if any.</pre>
<br />
<p>- Matt</p>
<br />
<p>On January 9th, 2014, 5:38 p.m. CST, Jonathan Rose 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.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers, Matt Jordan and Mark Michelson.</div>
<div>By Jonathan Rose.</div>
<p style="color: grey;"><i>Updated Jan. 9, 2014, 5:38 p.m.</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;">This patch adds the ability to interface with res_external_mwi via AMI. The following commands are implemented:
PUT mailboxes/mailboxName
modifies mailbox state and implicitly creates new mailboxes
GET mailboxes/mailboxName
retrieves a JSON representation of a single mailbox if it exists
GET mailboxes
retrieves a JSON array of all mailboxes
DELETE mailbox/mailboxName
deletes a mailbox
Thanks to Richard's external MWI work, this is actually a very simple system.</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;">Tested unloads and reloads of the res_stasis_mailboxes module with res_mwi_external loaded and unloaded
Tested how commands respond from resource_mailboxes when res_stasis_mailboxes isn't loaded
Tested each of the commands with numerous parameters
Created testsuite test in https://reviewboard.asterisk.org/r/3118/ which confirms basic operation of all the new ARI functions.</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>/branches/12/rest-api/resources.json <span style="color: grey">(405265)</span></li>
<li>/branches/12/rest-api/api-docs/mailboxes.json <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/res/res_stasis_mailbox.exports.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/res/res_stasis_mailbox.c <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/res/res_ari_mailboxes.c <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/res/ari/resource_mailboxes.c <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/res/ari/resource_mailboxes.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/res/ari/ari_model_validators.c <span style="color: grey">(405265)</span></li>
<li>/branches/12/res/ari/ari_model_validators.h <span style="color: grey">(405265)</span></li>
<li>/branches/12/res/ari.make <span style="color: grey">(405265)</span></li>
<li>/branches/12/include/asterisk/stasis_app_mailbox.h <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3117/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>