Skip to content
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

Use source generation for JSON serialization metadata #490

Merged
merged 1 commit into from
Sep 26, 2022

Conversation

VMelnalksnis
Copy link
Owner

Changes proposed in this pull request:

  • Use a JsonSerializerContext in Metadata mode in WebApi and WebApi.Client

@VMelnalksnis VMelnalksnis added type:enhancement New feature or request area:ui Relating to user interface area:api labels Sep 25, 2022
@codecov
Copy link

codecov bot commented Sep 26, 2022

Codecov Report

Merging #490 (882e3b3) into master (94a13d3) will increase coverage by 0.01%.
The diff coverage is 96.01%.

@@            Coverage Diff             @@
##           master     #490      +/-   ##
==========================================
+ Coverage   66.71%   66.73%   +0.01%     
==========================================
  Files         279      279              
  Lines        8478     8480       +2     
  Branches      548      548              
==========================================
+ Hits         5656     5659       +3     
+ Misses       2641     2640       -1     
  Partials      181      181              
Impacted Files Coverage Δ
...shade.WebApi.Client/ServiceCollectionExtensions.cs 0.00% <ø> (ø)
source/Gnomeshade.WebApi.Models/Owners/Owner.cs 0.00% <0.00%> (ø)
...ource/Gnomeshade.WebApi.Client/GnomeshadeClient.cs 78.57% <83.33%> (ø)
...onia.Core/DesignTime/DesignTimeGnomeshadeClient.cs 73.46% <100.00%> (ø)
...e.WebApi.Client/GnomeshadeJsonSerializerOptions.cs 100.00% <100.00%> (ø)
...Gnomeshade.WebApi.Client/TokenDelegatingHandler.cs 72.97% <100.00%> (ø)
...ource/Gnomeshade.WebApi.Models/Accounts/Account.cs 100.00% <100.00%> (ø)
...omeshade.WebApi.Models/Accounts/AccountCreation.cs 100.00% <100.00%> (ø)
...eshade.WebApi.Models/Accounts/AccountInCurrency.cs 100.00% <100.00%> (ø)
...ebApi.Models/Accounts/AccountInCurrencyCreation.cs 100.00% <100.00%> (ø)
... and 34 more

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

@VMelnalksnis VMelnalksnis merged commit 882e3b3 into master Sep 26, 2022
@VMelnalksnis VMelnalksnis deleted the serializer-context branch September 26, 2022 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:api area:ui Relating to user interface type:enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant