Skip to content

Make storeDocumentHooks throw errors instead of silently ignoring them.#904

Closed
janthurau wants to merge 1 commit intomainfrom
feature/throwErrorInStoreDocumentHooks
Closed

Make storeDocumentHooks throw errors instead of silently ignoring them.#904
janthurau wants to merge 1 commit intomainfrom
feature/throwErrorInStoreDocumentHooks

Conversation

@janthurau
Copy link
Contributor

Make debounce function asynchronous and improve usage

Refactor the debounce function to return a Promise, enabling better handling in async workflows. Adjust related code to properly await hooks execution and error catching.

Simplify logic in storeDocumentHooks for improved readability and maintainability.

related to #895

Make debounce function asynchronous and improve usage

Refactor the debounce function to return a Promise, enabling better handling in async workflows. Adjust related code to properly await hooks execution and error catching.

Simplify logic in storeDocumentHooks for improved readability and maintainability.
@janthurau janthurau force-pushed the feature/throwErrorInStoreDocumentHooks branch from 6ab3de9 to 1893b60 Compare February 19, 2025 21:31
@janthurau janthurau closed this Feb 19, 2025
@janthurau
Copy link
Contributor Author

nevermind. this breaks the entire hook architecture (as you won't be able to stop the hook chain for a given hook name)

@janthurau janthurau deleted the feature/throwErrorInStoreDocumentHooks branch February 17, 2026 21:29
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.

1 participant