<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/2100/">https://reviewboard.asterisk.org/r/2100/</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;">There are only a few problems I can find
* Section 2.3.1, the "pre-dial handler" link is red. I think the issue is that you need a hyphen between "pre" and "dial" instead of a space in the link.
* I'm not sure if the "Syntax" block in section 3.1 is going to be more harmful than helpful. First, people may get the idea that "groups" and "range" are potentially names of options that can be configured since their notation is not different from configuration examples. Second, I would not assume that the demographic reading the Asterisk wiki is necessarily familiar with BNF or similar grammars, so the notation may not mean much to them.
* Section 3.1 should state the bounds on numeric pickup groups.
* Incredibly nitpicky, but the second note in section 3.2 should be written in a more general form so that it cannot be misconstrued that your statement applies to the literal '4' instead of any number given to a call/pickup group.
Most of the data on the page is general call pickup information, so I agree with Alec's previous claim that placing it in the Asterisk 11 space seems a bit odd. The only part that specifically applies to Asterisk 11 is the new named groups feature. I think this could be moved some place else in the AST space, perhaps somewhere in "Configuration and Operation". Then, you'd need to specifically mention in the named groups section that the feature applies to Asterisk 11+ only.</pre>
<br />
<p>- Mark</p>
<br />
<p>On September 6th, 2012, 3:05 p.m., rmudgett wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/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 rmudgett.</div>
<p style="color: grey;"><i>Updated Sept. 6, 2012, 3:05 p.m.</i></p>
<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;">A new wiki page for review that describes call pickup is at
https://wiki.asterisk.org/wiki/display/AST/Call+Pickup</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-20277">ASTERISK-20277</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2100/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>