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

Fix etag calculation for updates when the content language in the req… #197

Merged

Conversation

gcotelli
Copy link
Member

…uest don't match the target language negotiated with the Accept header.

…uest don't match the target language negotiated with the Accept header.
Copy link

codecov bot commented Aug 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.58%. Comparing base (da18739) to head (1af252e).
Report is 2 commits behind head on release-candidate.

Additional details and impacted files
@@                Coverage Diff                 @@
##           release-candidate     #197   +/-   ##
==================================================
  Coverage              99.58%   99.58%           
==================================================
  Files                    166      166           
  Lines                  13585    13596   +11     
==================================================
+ Hits                   13529    13540   +11     
  Misses                    56       56           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gcotelli gcotelli merged commit 7ada0ab into release-candidate Aug 23, 2024
114 checks passed
@gcotelli gcotelli deleted the etag_calculation_when_content_language_present branch August 23, 2024 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants