<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Joshua, thank you for your kind review, always. :)</p><p style="white-space: pre-wrap; word-wrap: break-word;">Btw, I don't agree with about generally not used part... </p><p style="white-space: pre-wrap; word-wrap: break-word;">Simply imagine, if the inbound call/channel executes the stasis and connected with other channel using the bridge. And it needs to be hung up with the same hangup cause with the bridged channel.<br>Then it might have a challenge.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this will happen a lot - delivering the same hangup code to the other channel.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Anyways, I can see the DELETE /channel was not designed for this kind of use case. But could you please consider this once again?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I was referring to many of the ones you've added. They won't get used in practice, and confuse normal users. Allowing the hangup cause code to be explicitly specified is fine for cases where you need granular control.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/11287">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/11287">change 11287</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/+/11287"/><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: I1cf1d991ffd759d0591b347445a55f416ddc3ff2 </div>
<div style="display:none"> Gerrit-Change-Number: 11287 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 01 May 2019 23:55:31 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>