Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support RouteShorthandOptions to Receive Async Hooks #4943

Conversation

BrendenInhelder
Copy link
Contributor

Checklist

@github-actions github-actions bot added the typescript TypeScript related label Jul 31, 2023
@Uzlopak
Copy link
Contributor

Uzlopak commented Jul 31, 2023

Sry, but this is not valid.

@BrendenInhelder
Copy link
Contributor Author

@Uzlopak Could you (or anyone else) explain further why my approach is incorrect? I believe you, I would just like to understand so I can attempt to solve the issue in another way.

@Uzlopak
Copy link
Contributor

Uzlopak commented Jul 31, 2023

Because you are basically defining that the route short hand should contain async hooks in the corresponding hookNameAsync attributes.

@BrendenInhelder
Copy link
Contributor Author

@Uzlopak Would you like me to continue working on this issue? It seems that #4945 may address this.

@Uzlopak
Copy link
Contributor

Uzlopak commented Aug 1, 2023

#4945 is about handling correctly addHook() calls. but this one is about e.g. server.get() calls.

It doesnt seem to be trivial.

@BrendenInhelder
Copy link
Contributor Author

@Uzlopak Thanks for the info. This issue bewilders me because even forcing a Promise as a return does not seem to match the call to use an async hook handler. I believe I will try working on something else.

@Uzlopak
Copy link
Contributor

Uzlopak commented Aug 1, 2023

Lets discuss this more in #4859

@Uzlopak
Copy link
Contributor

Uzlopak commented Aug 5, 2023

I close this for now, because there is no obvious way to solve it.

@Uzlopak Uzlopak closed this Aug 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
typescript TypeScript related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants