-
Notifications
You must be signed in to change notification settings - Fork 22
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
Async Extension #6
Comments
Maybe I will add this feature in the future, but I think is not necessary now. Thank for your advice 😃 |
If I could help in this. |
`
` |
Thank you for your idea, I will check it later. 😃 |
How about this? |
I add it to version 3.1.1. You can give me some feedback or recommendation if it is convenient. Thanks 😊 |
Nice, thanks I expected the async code to use the ToListAsync from EFCore |
Uhhh...In my opinion, I want to keep this library simple and not depend too more on an external library. P.S. I thought that .Net Core 2.2 may be able to use the way you suggested. |
I understand, you're right not to put any mode dependencies. |
The current implementation uses Task.Run which will just spawn a threadpool thread. This is detrimental to some applications. |
I think that this project isn't alive anymore |
Have you ever consider to make ?
.ToDataSourceResultAsync(request)
The text was updated successfully, but these errors were encountered: