[Asterisk-code-review] app_queue: fixed ordering of events for queue agent dial (...asterisk[16])

[Nathan Bruning]

Nathan Bruning has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/11058 )

Change subject: app_queue: fixed ordering of events for queue agent dial
......................................................................


Patch Set 1:

> Patch Set 1:
> 
> > Patch Set 1:
> > 
> > Where would you have found or seen such a warning if you were looking?
> 
> One good place would be the documentation of the AgentConnect event, stating that "the called channel is already dialing, so events on that channel may or may not have been sent already".
> 
> Also a general explanation of event ordering on https://wiki.asterisk.org/wiki/display/AST/Interfaces would be a good idea, as this applies both the AMI and ARI.

(sorry, meant to say AgentCalled instead of AgentConnect)


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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: Id0865fb306136f30a1aa8be82f3753c47059ff2d
Gerrit-Change-Number: 11058
Gerrit-PatchSet: 1
Gerrit-Owner: Nathan Bruning <nathan at iperity.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Nathan Bruning <nathan at iperity.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Comment-Date: Tue, 05 Mar 2019 08:47:08 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190305/d811bd42/attachment.html>