Skip to content

Conversation

zubricks
Copy link
Member

Addresses feedback from #10460 It is a bit misleading the way it is worded currently—so I've updated the verbiage accordingly and provided code snippets

They're not wrong, it is a bit misleading and I've updated the verbiage accordingly and provided examples
```ts
const afterChange = ({ doc }) => {
// Trigger side-effect without blocking
pingAnalyticsService(doc.id)
Copy link
Member

Choose a reason for hiding this comment

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

We can use the void keyword to signalize that this is a floating promise (unawaited async function call). By default, typescript-eslint will throw a warning if this is not set:

void pingAnalyticsService(doc.id)

Copy link
Member Author

Choose a reason for hiding this comment

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

finally got around to making this small change

@JarrodMFlesch JarrodMFlesch added the area: docs Improvements or additions to documentation label Sep 18, 2025
@zubricks zubricks removed the request for review from DanRibbens October 1, 2025 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: docs Improvements or additions to documentation created-by: Payload team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants