Skip to content

Synchronous Calls? #23

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

Closed
Mike-E-angelo opened this issue Nov 1, 2014 · 3 comments
Closed

Synchronous Calls? #23

Mike-E-angelo opened this issue Nov 1, 2014 · 3 comments

Comments

@Mike-E-angelo
Copy link

Hello,

Once again, I would like to express my total appreciation for this project! Very happy to see someone put this together (and even more surprised that it's not a first-class component to the MVC framework).

It appears that this project currently produces asynchronous operations. Which is great if you are not managing async/tasks within your application. However, some of us have components that do manage this for us (and are also, um, apprehensive, towards async to begin with... beware of zombies. :)) and prefer synchronous calls. It would be great if this project supported them. I would be interested in making this happen with a pull request if that would help.

If this is an easy existing setting that I am overlooking that would be great to know, too.

Thank you,
Michael

@faniereynders
Copy link
Member

Hi Michael,
Thanks for your great feedback. I really want to enhance the project even more, but struggle to find the time to do so. I've been getting quite a few suggestions over the past couple of months and will try to implement them soon. You are welcome to fork it and implement your own features, then send me a PR when you're done :)

Thanks again.

@Mike-E-angelo
Copy link
Author

Oh man... yeah, I am looking through the Asp.Net code and everything looks like async now. It's a zombie invasion, alright. What a mess. :) HttpClient is all async. async in the method decoration and "Async" in the name. Plus coupling everything to Task and Task. So messy. I will definitely look into getting a PR for you and this great project!

@faniereynders
Copy link
Member

Resolved in #33

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

No branches or pull requests

2 participants