-
Notifications
You must be signed in to change notification settings - Fork 34
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 for Async controller methods #20
Comments
'Plan' is such a big word, but I've considered it :) I think it's a good idea. |
Just to be clear, if I pass in the async method without awaiting or invoking result your code works, but VS is raising warnings all over the place. So if there is a room in your 'plan' it would be nice to kill those warnings. Thanks On Sep 18, 2013, Mark Seemann notifications@github.com wrote:
-- Sent from my Android phone with K-@ Mail. Please excuse my brevity. |
The problem with adding support for this right now is that in order to do that, I'd have to upgrade Hyprlinkr to target .NET 4.5, and right now it only supports .NET 4.0. However, the up-coming Web API 2 (version 5.0.0 AFAICT) seems to support only .NET 4.5, so it seems reasonable to upgrade Hyprlinkr at that time too. After that, I'll add support for async controllers. |
I found a way to implement this on .NET 4.0, so the latest version of Hyprlinkr (0.9.9) now supports async Controller methods like this: Uri actual = linker.GetUriAsync((AsyncController c) => c.Get(id)).Result; |
Great, I'll give it a try and let you know how it goes. Thanks |
Just to confirm that everything is working as expected - without warnings ;) ... thanks again. |
Thank you for reporting back :) |
AsyncController is MVC specific, do you have any workaround for WebAPI? |
Hi @barbosatek, the above example is working in WebAPI so could you give us an example if the issue you have ? |
Discussion moved to #42 |
Hi,
do you plan to add support for async controllers ?
btw great library.
Thanks
The text was updated successfully, but these errors were encountered: