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

/filings/ endpoint marking more than one most_recent Form 1 (Statement of Organization) #3096

Closed
3 of 4 tasks
lbeaufort opened this issue May 1, 2018 · 2 comments
Closed
3 of 4 tasks

Comments

@lbeaufort
Copy link
Member

lbeaufort commented May 1, 2018

  • Add a testing ticket

So that committees and the public can see the most recent version of the Form 1 (Statement of Organization), we need to update the logic to handle when a committee files multiple Form 1's as new and only show the newest as "Current Version" and the others as "Past Version."

@patphongs discovered this issue.

Example: (https://www.fec.gov/data/committee/C00639013/?tab=filings)

8-31-2017 3-35-01 pm

Background: Using FECFile and other 3rd party vendors, committees can amend their Statement of Organization (Form 1) by filing it as new. Then, an overnight process updates it to show as amended. As a result, there is no amendment number associated with the file that was submitted as new and now displays as amended.

Potential fix: Add to /filings/ "amendment chain" logic to handle multiple F1Ns for the same committee. If the amendment chain logic fails (>1 current version for the same filing type - for statements of org. ) then use the most recently filed one as the current version and mark the others as past version.

Existing amendment chain view is complicated and not a good place to fix the logic for F1's and F2's. We should investigate where else we can fix the logic upstream.

@lbeaufort lbeaufort changed the title Committee filing pages displaying more than one "Current" Form 1 (Statement of Organization) /filings/ endpoint marking more than one most_recent Form 1 (Statement of Organization) May 1, 2018
@lbeaufort lbeaufort self-assigned this May 1, 2018
@jwchumley jwchumley modified the milestones: Sprint 6.3, Sprint 6.4 Jun 25, 2018
@PaulClark2 PaulClark2 added the Bug label Jul 3, 2018
@lbeaufort lbeaufort modified the milestones: Sprint 6.4, Sprint 6.5 Jul 10, 2018
@lbeaufort lbeaufort removed their assignment Jul 24, 2018
@lbeaufort lbeaufort modified the milestones: Sprint 6.5, Sprint 6.6 Jul 24, 2018
@lbeaufort lbeaufort self-assigned this Aug 17, 2018
@vrajmohan vrajmohan self-assigned this Aug 22, 2018
@vrajmohan
Copy link
Contributor

After talking to @lbeaufort, we should ideally fix this at the API level to benefit all users. If no easy solution is possible, we can fix this on the CMS side.

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

5 participants