Skip to content

Add acknowledgement deadline #1168906818845353 #4

New issue

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

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

Already on GitHub? # to your account

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

Conversation

aman00323
Copy link
Contributor

No description provided.

@aman00323 aman00323 requested a review from munir131 April 2, 2020 11:01
@@ -165,6 +165,15 @@ public function pop($subscriber = null)
'returnImmediately' => true,
'maxMessages' => 1,
]);
$queue = $this->getQueue($subscriber);
if ($this->config && $this->config['subscribers'] && $queue && isset($this->config['subscribers'][$queue])) {
$deadline = $this->config['subscribers']['deadline'];
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aman00323 It must be $this->config['subscribers'][$queue]['deadline'] . correct? Please add sample config in readme

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants