<p> Attention is currently required from: N A, Joshua Colp. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/19474">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19474?tab=comments">Patch Set #4:</a> </p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">module load already expects <module name> as the next argument, so I'm not sure how this could be readily combined into that CLI command.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It is a bit clunky, but this is admittedly an edge case that likely nobody besides developers would encounter, and even then, only rarely. If a user knows this needs to be done then they would do it; if a user does not already know that the xmldocs need to be reloaded, then I don't really see how having a parameter to do that is going to improve the user experience, since they wouldn't know to do that.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Sorry, I meant...<br>"module load <module_name> [ with xmldoc ]" so the "with xmldoc" is optional at the end.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe we can update the warning messages wherever modules could fail to load to also say "If this module was recently built, run xmldoc reload and then load the module again"? That would probably have the most impact, since it makes the warning actionable on the off chance somebody gets it.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">That'll work but the updated command would also be helpful if you can do it.  If not, that's fine.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/19474">change 19474</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/19474"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4f265b0e5517e757c5453a0f241201a5788d3a07 </div>
<div style="display:none"> Gerrit-Change-Number: 19474 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 11 Nov 2022 12:02:36 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Comment-In-Reply-To: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>