Skip to content

[13.x] Add isManagedQueue to CloudManager - #61568

Merged
taylorotwell merged 3 commits into
laravel:13.xfrom
jackbayliss:13.x-is-managed-queue
Sep 13, 2026
Merged

[13.x] Add isManagedQueue to CloudManager#61568
taylorotwell merged 3 commits into
laravel:13.xfrom
jackbayliss:13.x-is-managed-queue

Conversation

@jackbayliss

@jackbayliss jackbayliss commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

I've macro'd this, but I think it's worth adding on the manager as it's easier than going through the queue connection (as in most people won't realise this is there)

This gives anyone an easy answer of "Is this queue managed", which is very useful if you're multiple drivers- useful locally, and useful if you're using fifo in cloud.

This means we can do stuff like this:

if (Cloud::isManagedQueue('notifications.fifo')) {
    Queue::forward('notifications', 'notifications.fifo', 'cloud');
}

Feels cool to me, and imagine I won't be the only person to reach for this, but, feel free to adjust to throw away- just something I macro'd 🫡

@github-actions

Copy link
Copy Markdown

Thanks for submitting a PR!

Note that draft PRs are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

@jackbayliss
jackbayliss marked this pull request as ready for review September 12, 2026 19:21
@taylorotwell
taylorotwell merged commit 3af76db into laravel:13.x Sep 13, 2026
54 checks passed
@jackbayliss
jackbayliss deleted the 13.x-is-managed-queue branch September 13, 2026 17:25
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