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

Statutory citation filter broken #3271

Closed
patphongs opened this issue Jul 9, 2018 · 3 comments
Closed

Statutory citation filter broken #3271

patphongs opened this issue Jul 9, 2018 · 3 comments

Comments

@patphongs
Copy link
Member

When performing a filter for statutory citations, the API returns a 500 error. Citation filters were just added back into the AO search here: https://fec-dev-proxy.app.cloud.gov/data/legal/search/advisory-opinions/?type=advisory_opinions&ao_category=F

Here's an example of the API error when attempting to filter by statutory citation: https://fec-dev-api.app.cloud.gov/v1/legal/search/?type=advisory_opinions&ao_category=F&ao_statutory_citation=52+U.S.C.+%C2%A730101

@patphongs patphongs added the Bug label Jul 9, 2018
@PaulClark2 PaulClark2 added this to the Sprint 7.1 milestone Aug 31, 2018
@patphongs patphongs changed the title Statutory citation filter broken Research: Statutory citation filter broken Sep 4, 2018
@patphongs patphongs modified the milestones: Sprint 7.1, Sprint 7.2 Sep 5, 2018
@vrajmohan vrajmohan modified the milestones: Sprint 7.2, Sprint 7.1 Sep 5, 2018
@vrajmohan vrajmohan self-assigned this Sep 5, 2018
vrajmohan pushed a commit that referenced this issue Sep 5, 2018
@vrajmohan vrajmohan changed the title Research: Statutory citation filter broken Statutory citation filter broken Sep 5, 2018
@vrajmohan
Copy link
Contributor

This is an easy fix. @AmyKort, should we expedite the fix so that it makes the release slated for Sep 11?

@AmyKort
Copy link

AmyKort commented Sep 5, 2018

That would be perfect if we can make it work @vrajmohan Thank you so much!

@vrajmohan
Copy link
Contributor

Done. Thanks, @lbeaufort, for retargeting my PR.

lbeaufort pushed a commit that referenced this issue Sep 6, 2018
lbeaufort pushed a commit that referenced this issue Sep 6, 2018
lbeaufort pushed a commit that referenced this issue Sep 11, 2018
lbeaufort pushed a commit that referenced this issue Sep 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants