Skip to content

Add type definition for R2 event notification message body#2789

Open
bthwaites wants to merge 1 commit into
cloudflare:mainfrom
bthwaites:bradley/r2-event-notification-type
Open

Add type definition for R2 event notification message body#2789
bthwaites wants to merge 1 commit into
cloudflare:mainfrom
bthwaites:bradley/r2-event-notification-type

Conversation

@bthwaites

@bthwaites bthwaites commented Sep 24, 2024

Copy link
Copy Markdown
Contributor

Adding a type that can be used as a parameter for MessageBatch when consuming events from a Queue that were created by an R2 Event Notification.

An example of the message format is also documented here: https://developers.cloudflare.com/r2/buckets/event-notifications/#message-format

@bthwaites bthwaites requested review from a team as code owners September 24, 2024 17:57
@bthwaites bthwaites requested review from dom96 and hoodmane September 24, 2024 17:57
Comment thread types/defines/r2.d.ts Outdated
@bthwaites bthwaites force-pushed the bradley/r2-event-notification-type branch from 2d0ce6f to f090b5c Compare November 5, 2024 20:05
Comment thread types/defines/r2.d.ts

@petebacondarwin petebacondarwin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks OK to me but I was not able to verify it against the actual types internally as I don't know where they are stored.

@dom96 dom96 removed their request for review March 17, 2025 14:09
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.

3 participants