<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/2698/">https://reviewboard.asterisk.org/r/2698/</a>
</td>
</tr>
</table>
<br />
<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 and jrose.</div>
<div>By David Lee.</div>
<p style="color: grey;"><i>Updated July 30, 2013, 8:58 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Added comment for get_sounds params.
Also adjusted the generator so that empty /*! */ comments won't be generated.</pre>
</td>
</tr>
</table>
<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;">Swagger allows parameters to be specified as 'allowMultiple', meaning
that the parameter may be specified as a comma separated list of
values.
I had written some of the API docs using that, but promptly forgot
about implementing it. This patch finally fills in that gap.
The codegen template was updated to represent 'allowMultiple' fields
as array/size fields in the _args structs. It also parses the comma
separated list using ast_app_separate_args(), so quoted strings in the
argument will be handled properly.</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;">Lots of variations hitting the events websocket and adding channels to
bridges using comma separated lists.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/res/ari/resource_asterisk.h <span style="color: grey">(395729)</span></li>
<li>/trunk/res/ari/resource_bridges.h <span style="color: grey">(395729)</span></li>
<li>/trunk/res/ari/resource_bridges.c <span style="color: grey">(395729)</span></li>
<li>/trunk/res/ari/resource_events.h <span style="color: grey">(395729)</span></li>
<li>/trunk/res/ari/resource_events.c <span style="color: grey">(395729)</span></li>
<li>/trunk/res/ari/resource_sounds.h <span style="color: grey">(395729)</span></li>
<li>/trunk/res/res_ari_asterisk.c <span style="color: grey">(395729)</span></li>
<li>/trunk/res/res_ari_bridges.c <span style="color: grey">(395729)</span></li>
<li>/trunk/res/res_ari_channels.c <span style="color: grey">(395729)</span></li>
<li>/trunk/res/res_ari_endpoints.c <span style="color: grey">(395729)</span></li>
<li>/trunk/res/res_ari_events.c <span style="color: grey">(395729)</span></li>
<li>/trunk/res/res_ari_playback.c <span style="color: grey">(395729)</span></li>
<li>/trunk/res/res_ari_recordings.c <span style="color: grey">(395729)</span></li>
<li>/trunk/res/res_ari_sounds.c <span style="color: grey">(395729)</span></li>
<li>/trunk/rest-api-templates/ari_resource.h.mustache <span style="color: grey">(395729)</span></li>
<li>/trunk/rest-api-templates/param_cleanup.mustache <span style="color: grey">(PRE-CREATION)</span></li>
<li>/trunk/rest-api-templates/param_parsing.mustache <span style="color: grey">(395729)</span></li>
<li>/trunk/rest-api-templates/res_ari_resource.c.mustache <span style="color: grey">(395729)</span></li>
<li>/trunk/rest-api/api-docs/bridges.json <span style="color: grey">(395729)</span></li>
<li>/trunk/rest-api/api-docs/channels.json <span style="color: grey">(395729)</span></li>
<li>/trunk/rest-api/api-docs/events.json <span style="color: grey">(395729)</span></li>
<li>/trunk/rest-api/api-docs/sounds.json <span style="color: grey">(395729)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2698/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>