Skip to content

Conversation

@tomschlick
Copy link
Contributor

It's helpful to know which job has been exceeding it's max attempts.

@taylorotwell taylorotwell merged commit 1ed8450 into laravel:5.5 Nov 3, 2017
@taylorotwell
Copy link
Member

taylorotwell commented Nov 3, 2017

On second thought it may be more useful to use job->resolveName. If you want to make another PR. This will provide the proper job name for "wrapped" jobs like queued mailables.

@tomschlick
Copy link
Contributor Author

Yeah thats not a bad idea. PR incoming!

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