search

crowdin / crowdin-api-client-dotnet

.NET client library for Crowdin API
https://www.nuget.org/packages/Crowdin.Api/
MIT License
53 stars 26 forks source link

fix: HttpRequestMessage instance per request #101

Closed PatrykPlewaOfficial closed 1 year ago

PatrykPlewaOfficial commented 1 year ago

Fixes the exception System.InvalidOperationException: The request message was already sent. Cannot send the same request message multiple times. when the HttpRequestMessage instance is being reused.

It's a breaking change No longer a breaking change.

codecov[bot] commented 1 year ago

Codecov Report

Merging #101 (be1f074) into main (040e8c1) will decrease coverage by 0.35%. The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
- Coverage   47.10%   46.75%   -0.35%     
==========================================
  Files         272      272              
  Lines        3603     3630      +27     
==========================================
  Hits         1697     1697              
- Misses       1906     1933      +27
Impacted Files Coverage Δ
src/Crowdin.Api/CrowdinApiClient.cs 6.70% <0.00%> (-0.99%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

andrii-bodnar commented 1 year ago

@PatrykPlewaOfficial thanks a lot for your contribution!

@innomaxx please review it