-
Notifications
You must be signed in to change notification settings - Fork 15
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 Methods #4
Comments
Hi, do you have any plan to add async API version? it's really annoying using sync version of API with .NET Core project and it has performance impacts on the application because of thread blocking. |
Hi Dear . We have a merge request from Mr @HesamKashefi , after we checked it we get very good changes and we must accept it ,but in due to the Corona virus crisis and because all of our team members have been working remotely these points maybe execute with delay .We are committed to doing this as soon as possible. |
Perhaps after two months, you had enough time to merge @mhadieskandari, hadn't you? By the way, I sent another pull request with minimal changes but I don't think it merged just like HesamKashefi 's pull request. |
We are sorry for the delay. We appreciate you for your kindness and patience. We have decided to create another repo for dotnet core to prevent some problems for old customers that need some changes and updates for old technologies. Could you please do a pull request again on https://github.com/kavenegar/Kavenegar.Core? |
Sure, I do ASASP. |
I did mine, |
@HesamKashefi Thanks a lot. I will check it and will give you a response. |
It's great if you can add async methods to the project.
Thanks for your professional work ...
The text was updated successfully, but these errors were encountered: