docs: Update events.md to include reference to createEventBuilder #3526
Conversation
|
|
Also, while i'm here, we seem to have found a weird edge case here when trying to publish multiple events in a Promise.all, see the discussion on discord. but it breaks down to creating the payload inside the async loop sends the same payload each time, but creating the payload and then looping to publish works fine and sends separate payloads. // This example ends up executing the event handler lambda function 3 times with the same payload
async function publishEvent () {
await Events.Created.publish({
id: crypto.randomUUID()
})
}
export async function produceEvents () {
await Promise.all([
publishEvent(),
publishEvent(),
publishEvent()
])
}whereas this example works: // this example works as expected, where the event handler lambda function runs 3 times, but each time with a different event
async function publishEvent (eventPayload) {
await Events.Created.publish(eventPayload)
}
export async function produceEvents () {
const eventPayloads = [
{ id: crypto.randomUUID() },
{ id: crypto.randomUUID() },
{ id: crypto.randomUUID() },
]
await Promise.all(eventPayloads.map(payload => publishEvent(payload)))
}thanks to @srodriki for helping identify the issue @srodriki thought this was unlikely to be a SST issue, but i thought it might be nice to add a warning somewhere in this doc (or perhaps somewhere else?) explaining the edge case and how to avoid? |
I couldn't find any reference to createEventBuilder anywhere in the docs site, and it took me spelunking in the Console repo to find out what i was missing creating Events
I don't really know the full ins and outs of how createEventBuilder worked so i added it as an example here rather than give a full section for it.