[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