[Asterisk-code-review] tests/apps/queues/reason pause ami: Add 'PausedReason' field... (testsuite[master])

Kevin Harwell asteriskteam at digium.com
Mon Jan 4 18:05:15 CST 2016


Kevin Harwell has posted comments on this change.

Change subject: tests/apps/queues/reason_pause_ami: Add 'PausedReason' field test AMI
......................................................................


Patch Set 2: Code-Review+1

>  If you think is better technical choise tell me and i will to make then in yaml

In my opinion either way is fine. I was mainly just pointing out that there was an alternative way to do it if you wanted, but it sounds like you were already aware.

-- 
To view, visit https://gerrit.asterisk.org/1848
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I04ad41f7a39a319f58cf40abeb9434fc02b6d767
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list