Closed cnlucas closed 3 months ago
Attention: Patch coverage is 54.54545%
with 5 lines
in your changes missing coverage. Please review.
Project coverage is 86.72%. Comparing base (
4a6d55d
) to head (a9bf00b
).
Files | Patch % | Lines |
---|---|---|
webservices/resources/legal.py | 50.00% | 5 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary (required)
(Include a summary of proposed changes and connect issue below)
Required reviewers
(Include who is required to review prior to merge. For example: One designer and two front end developer reviews are required prior to merge)
Impacted areas of the application
General components of the application that this PR will affect:
-
Screenshots
(Include a screenshot of the new/updated features in context (“in the wild”). If it is an interface change, include both before and after screenshots)
Related PRs
Related PRs against other branches:
How to test
(Include any information that may be helpful to the reviewer(s). This might include links to sample pages to test or any local environmental setup that is unusual such as environment variable (never credentials), API version to point to, etc)
-
System architecture updates (if applicable)
(If this pull request changes our current system diagram, include a description of those changes here and create a new ticket to update the system diagram)