Skip to content

adding queue notification consumer type - #15381

Draft
jkoe-cf wants to merge 5 commits into
mainfrom
jkoe/queue-notification-consumer
Draft

adding queue notification consumer type#15381
jkoe-cf wants to merge 5 commits into
mainfrom
jkoe/queue-notification-consumer

Conversation

@jkoe-cf

@jkoe-cf jkoe-cf commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Fixes MQ-1430

Adds a new notification consumer type for queues


  • Tests
    • [ x] Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because:

A picture of a cute animal (not mandatory, but encouraged)

@changeset-bot

changeset-bot Bot commented Aug 26, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 52da7cd

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Comment thread packages/deploy-helpers/src/triggers/queue-consumers.ts Outdated
@ask-bonk

ask-bonk Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

I'm Bonk, and I've done a quick review of your PR.

PR #15381 adds Queue notification consumer CLI commands and display support.

  1. High queue-consumers.ts:319: removal fails when another consumer precedes the notification consumer. An inline suggestion was posted.

github run

@pkg-pr-new

pkg-pr-new Bot commented Aug 26, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

npm i https://pkg.pr.new/@cloudflare/autoconfig@15381

@cloudflare/build-output-utils

npm i https://pkg.pr.new/@cloudflare/build-output-utils@15381

@cloudflare/codemods

npm i https://pkg.pr.new/@cloudflare/codemods@15381

@cloudflare/config

npm i https://pkg.pr.new/@cloudflare/config@15381

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@15381

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@15381

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@15381

miniflare

npm i https://pkg.pr.new/miniflare@15381

@cloudflare/pages-functions

npm i https://pkg.pr.new/@cloudflare/pages-functions@15381

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@15381

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@15381

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@15381

@cloudflare/vitest-plugin

npm i https://pkg.pr.new/@cloudflare/vitest-plugin@15381

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@15381

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@15381

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@15381

wrangler

npm i https://pkg.pr.new/wrangler@15381

commit: 52da7cd

Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.

2 participants