[Asterisk-code-review] app queue: Added reason pause of member (asterisk[13])
Matt Jordan
asteriskteam at digium.com
Tue Oct 20 09:07:46 CDT 2015
Matt Jordan has posted comments on this change.
Change subject: app_queue: Added reason pause of member
......................................................................
Patch Set 1: Code-Review-1
As a new feature being added to Asterisk 13, this *must* have an accompanying test in either the Test Suite or unit test in Asterisk.
Tests are always a "good thing", but for modifications to an LTS, they are a requirement. See https://wiki.asterisk.org/wiki/display/AST/Software+Configuration+Management+Policies#SoftwareConfigurationManagementPolicies-FeaturePolicy for more information.
If you'd like some pointers on writing a test for this in the Asterisk Test Suite, please just comment on here and I'd be happy to provide some guidance.
Given the functionality being
--
To view, visit https://gerrit.asterisk.org/1448
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia5db503482f50764c15e2020196c785f59d4a68e
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: No
More information about the asterisk-code-review
mailing list