-
Notifications
You must be signed in to change notification settings - Fork 40
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
6445 MUR citation filters #6588
base: develop
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #6588 +/- ##
===========================================
- Coverage 79.72% 79.64% -0.08%
===========================================
Files 235 235
Lines 5159 5164 +5
===========================================
Hits 4113 4113
- Misses 1046 1051 +5 ☔ View full report in Codecov by Sentry. |
<input type="radio" id="citation_require_all_false" name="case_citation_require_all" value="false"{% if not case_citation_require_all %}checked{% endif %}> | ||
<label for="citation_require_all_false">Show cases citing any of them</label> | ||
<input type="radio" id="citation_require_all_true" name="case_citation_require_all" value="true"{% if case_citation_require_all %}checked{% endif %}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The boolean for case_citation_require_all seems to always default to "Show cases citing all of them" regardless of which one the user chooses.
Summary
Adding the citations filters for MURs searches
ALSO fixed a related oversight for AOs search
Required reviewers
Impacted areas of the application
Screenshots
Updated filter order:
The document date tooltip
Related PRs
None
How to test
npm run build