fecgov / openFEC

The first RESTful API for the Federal Election Commission. We're aiming to make campaign finance more accessible for journalists, academics, developers, and other transparency seekers.
https://api.open.fec.gov/developers
Other
483 stars 106 forks source link

[WIP]Fix legal keyword search option #5913

Closed fec-jli closed 3 months ago

fec-jli commented 3 months ago

Summary (required)

This PR will fix the issue that keyword document search by exclude boolean.

Required reviewers

Impacted areas of the application

legal universal search

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:

branch PR
fix/other_pr [link]()
feature/other_pr [link]()

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)

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 22.58065% with 24 lines in your changes missing coverage. Please review.

Project coverage is 86.48%. Comparing base (15370e9) to head (587b3dd). Report is 3 commits behind head on develop.

Files Patch % Lines
webservices/resources/legal.py 22.58% 24 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #5913 +/- ## =========================================== - Coverage 86.68% 86.48% -0.21% =========================================== Files 82 82 Lines 8886 8902 +16 =========================================== - Hits 7703 7699 -4 - Misses 1183 1203 +20 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

fec-jli commented 3 months ago

closed. ref #5916