Closed mtruj013 closed 3 months ago
Attention: Patch coverage is 66.66667%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 85.00%. Comparing base (
a136ecd
) to head (55311e3
). Report is 1 commits behind head on main.:exclamation: Current head 55311e3 differs from pull request most recent head b5e1961
Please upload reports for the commit b5e1961 to get more accurate results.
Files | Patch % | Lines |
---|---|---|
webapp/views.py | 66.66% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Done
Rationale
Before https://github.com/canonical/ubuntu-com-security-api/pull/137, version and status filtering had to be done together. For eg, to query for CVEs for jammy you'd need to write the query like so
version="jammy"&status=""
. Now that these have been decoupled the same functionality is achieved with justversion="jammy"
. This has already been refactored on the CVE overhaul branch, but the current CVE table form on u.com still uses the original querying functionality so this PR ensures there will be no errors while we finish the overhaul story.QA
Issue / Card
Fixes https://warthogs.atlassian.net/browse/WD-11570