[Asterisk-code-review] Basic PBX: Add Queues. (...asterisk[16])

[Sean Bright]

Sean Bright has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/12951 )

Change subject: Basic PBX: Add Queues.
......................................................................


Patch Set 1: Code-Review-2

So I started cherry picking this to the other branches and there was a merge conflict. 13 already has a queue configuration:

https://github.com/asterisk/asterisk/commit/cfbf5fbe918bc34f3d600760fc0b6f13a3a9a0ed#diff-2b2e3cc97ea8d14f40979ab1231b96a3

But I guess it was never merged forward because it doesn't even appear in the 14 branch.

Would the better course of action here be to cherry-pick the change from 13 and then (if necessary) a second patch set that updated the config in all supported branches?


-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/12951
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I11554be5b99a6b87139f041ec3358c59572a972f
Gerrit-Change-Number: 12951
Gerrit-PatchSet: 1
Gerrit-Owner: cmaj <chris at penguinpbx.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-Comment-Date: Thu, 26 Sep 2019 13:17:29 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190926/e7e83ecf/attachment.html>