Skip to content

Add $dispatchAfterCommit when running via custom worker #496

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

Closed
wants to merge 2 commits into from
Closed

Add $dispatchAfterCommit when running via custom worker #496

wants to merge 2 commits into from

Conversation

wietsewarendorff
Copy link

No description provided.

@Ilyes512
Copy link

@vyuldashev Hey, could you allow the workflow to run so we can at least see if its green (✅ )? Thanks!

jeromegamez added a commit to jeromegamez/vyuldashev-laravel-queue-rabbitmq that referenced this pull request Jan 3, 2023
@mattgrande
Copy link
Collaborator

Hi @wietsewarendorff, thanks for your submission. Unfortunately, we can't accept it as-is, since it would break any existing workers as soon as it's merged. We're evaluating other options (most likely a Service Provider) for use in the next major version.

@mattgrande mattgrande closed this Jan 27, 2023
@khepin
Copy link
Collaborator

khepin commented Jan 27, 2023

Duplicate of #513 , closing this one and moving the convo over there.

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.

4 participants