Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented missing QueueEntry event #98

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

alesf
Copy link

@alesf alesf commented Jan 13, 2016

QueueStatus action also returns QueueEntry event.

See:
http://www.asteriskdocs.org/en/2nd_Edition/asterisk-book-html-chunk/aste
risk-APP-F-32.html
@marcelog
Copy link
Owner

Hey thanks! :) Could you throw in a unit test too?

Added test for QueueEvent and updated readme events list.
@alesf
Copy link
Author

alesf commented Jan 14, 2016

I wasn't able to run the tests. I hope the test is OK.

@marcelog
Copy link
Owner

So we can't merge unless the build passes. Make sure that running ./vendor/bin/phing build yields no errors in your box. Travis will run the same command (more information is available in the README).

dkgroot pushed a commit to chan-sccp/PAMI that referenced this pull request Apr 23, 2019
QueueStatus action also returns QueueEntry event.
Added test for QueueEvent and updated readme events list.

Cherry-Picked 6abf8af from alesf (marcelog#98)
Cherry-Picked 92830d8 from alesf (marcelog#98)

DdG: Added small fix to Test for Position returning integer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants