[asterisk-dev] [Code Review] 4107: Wiki: Some new PJSIP-related pages

Mark Michelson reviewboard at asterisk.org
Fri Oct 31 10:57:25 CDT 2014



> On Oct. 29, 2014, 10:07 p.m., Kevin Harwell wrote:
> > "Configuring res_pjsip for Presence Subscriptions"
> > 
> > Under the "Configuration" section, 3rd paragraph, the "context" option description - (just to be sure) the subscribing endpoint's context must be set to that of the hint? 
> > 
> > -------------------------------------------------------------
> > 
> > "Resource List Subscriptions (RLS)"
> > 
> > Under "Batching Notifications", 1st paragraph the following sentence fragment sounds a little strange: ..."any further state changes of states in the list ..."
> > 
> > -------------------------------------------------------------
> > 
> > "Configuring Outbound Registrations" 
> > 
> > Under the "Authentication" last sentence: "Details about what options are available in auth sections can be found here."  <-- I'm guessing "here" was meant to be a link?
> > 
> > Also, the "Authentication" section is under "Dealing with Failure".  I'd probably move the "Dealing with Failure" section below the "Authentication" part.
> > 
> > Under "Manually Unregistering" I'd probably add a note that after manually unregistering the specified outbound registration will continue attempting re-registers based on it last registration expiration.
> > 
> > -------------------------------------------------------------
> > 
> > "Asterisk PJSIP Troubleshooting Guide"
> > 
> > There is a gap between the "Overview" section and the next.  Can this be shrunk or is it due to some kind of wiki formatting "thing"?
> > 
> > Personally, I'd lean toward breaking the troubleshooting guide into sub-pages.  Especially if you think more will be added later as things come up.  Breaking things up would certainly make it easier to find a particular section when navigating via the content tree.

I've addressed everything here except breaking the troubleshooting page up. I think that breaking it up is a good idea if the page starts to grow larger than it is right now. For the time being, the table of contents at the top of the page makes navigation reasonable.


- Mark


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4107/#review13623
-----------------------------------------------------------


On Oct. 22, 2014, 9:37 p.m., Mark Michelson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4107/
> -----------------------------------------------------------
> 
> (Updated Oct. 22, 2014, 9:37 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Description
> -------
> 
> I have written the following wiki pages:
> 
> https://wiki.asterisk.org/wiki/display/AST/Configuring+res_pjsip+for+Presence+Subscriptions
> https://wiki.asterisk.org/wiki/pages/viewpage.action?pageId=30278158
> https://wiki.asterisk.org/wiki/display/AST/Configuring+Outbound+Registrations
> https://wiki.asterisk.org/wiki/display/AST/Asterisk+PJSIP+Troubleshooting+Guide
> 
> The first three pages are tutorial pages for Asterisk PJSIP usage. The first focuses on setting up presence subscriptions, the second focuses on setting up resource list subscriptions, and the third focuses on configuring outbound registrations.
> 
> The fourth page is a general PJSIP troubleshooting guide. The intent of this page is not to be exhaustive at the moment, since this is a page that likely will be updated frequently as specific issues are encountered. This page may need to be split into multiple pages, but I'll leave that judgment to the reviewers.
> 
> 
> Diffs
> -----
> 
> 
> Diff: https://reviewboard.asterisk.org/r/4107/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mark Michelson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20141031/5f5bb8c7/attachment-0001.html>


More information about the asterisk-dev mailing list