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

Test candidate filings tab by 9/20/2018 #2339

Closed
6 tasks done
Tracked by #137
patphongs opened this issue Sep 7, 2018 · 9 comments
Closed
6 tasks done
Tracked by #137

Test candidate filings tab by 9/20/2018 #2339

patphongs opened this issue Sep 7, 2018 · 9 comments
Assignees
Milestone

Comments

@patphongs
Copy link
Member

patphongs commented Sep 7, 2018

A filings tab has been implemented on all candidate pages in order to display their Statement of candidacy and other documents. We need to test to make sure everything looks good before release. This feature will be released on 9/25/2018. All testing must be completed by 9/20/2018 in order to catch any major dealbreakers before release.

For reference, original issue is #2154 and the PR that was merged is #2310

Filings tab can be tested on develop. Here's an example URL of a candidate profile page: https://fec-dev-proxy.app.cloud.gov/data/candidate/S6AZ00019/?tab=filings

Completion criteria:

  • Ensure filings tab and sub items of the filings tab exists on candidate profile page left hand nav
  • For current candidates, ensure that the "About this candidate" tab lists only the candidate's current version of their Form 2 Statement of candidacy. Example: https://fec-dev-proxy.app.cloud.gov/data/candidate/S2MA00170/?tab=about-candidate
  • Check the filings tab to make sure that within the Statement of Candidacy (SOC) table all versions of the Form 2 is present including the current version
  • Check the SOC table to make sure that only F2s and RQ-5s display here.
  • Check that the Other documents table within the filings tab has all the necessary docs and are accurate. There should be no RFAIs listed in this table.
  • Check that the SOC table has columns in this order: Document > Version > Date filed > Image number > Pages
@patphongs patphongs added this to the Sprint 7.1 milestone Sep 7, 2018
@johnnyporkchops
Copy link
Contributor

This candidate 404s John S. McCaine for President:
https://fec-dev-proxy.app.cloud.gov/data/candidate/P80002801/
Works in prod:
https://www.fec.gov/data/candidate/P80002801/
The other examples of 404 errors that @lbeaufort fixed in PR #2320 are working ,so hopefully this is just an outlier.
Example of previous candidate that got 404 (now working)
https://fec-dev-proxy.app.cloud.gov/data/candidate/P40003832/

@patphongs
Copy link
Member Author

It seems to 404 on past candidates who no longer have financial activity because it is trying to call to the 2018 cycle, but that candidate does not have data in 2018, so it 404s.

Below is an example of another candidate that 404s, George Bush:

https://fec-dev-proxy.app.cloud.gov/data/candidate/P00000455

API Call:

ERR INFO:data.api_caller: load_single_type: https://fec-dev-api.app.cloud.gov/v1/candidate/P00000455/history/2018/?per_page=1&election_full=False

@lbeaufort
Copy link
Member

@patphongs @johnnyporkchops thanks for finding this. It looks like it's an issue on stage, so most likely a "feature" of the candidate refactoring/404 bug fix work. We should fix this before the release tomorrow. I can't find a testing issue for testing the candidate 404's - it may have been missed.

@llienfec
Copy link
Contributor

Tested a handful of committees. Only saw one strange case where a Form 2 that is a letter should come up as the most recent F2 under "About this candidate," but doesn't. The weird part is that it's first in the statement of candidacy list under "Filings."

H2CA42023 – most recent F2 on file is a letter from 11/28/2006. It shows up in the doc feed but not on the first page.

Tested candidates are below. I plan to look at a couple more senates and some presidentials tomorrow. Also need to find some examples where people have received RQ-5s to make sure those are showing up correctly. MS-Hs (disavowal notices) are showing up correctly in the other documents section.

H0LA03018 – good
H4TX02082 – good
H4LA03069 - good
H8TX16109 – good
S2MN00019 – good
S4KS00010 – good
S8VT00059 – good

@llienfec
Copy link
Contributor

Potential issue I'm looking at the candidate filing tab and it looks like it's not responsive like the committee filing tab for the same candidate. I used S2TX00106 and C00369033:

Candidate filing tab

At half width on my Surface - can't see the entire table and can't scroll:
image

Committee filing tab - responsive

image

@llienfec
Copy link
Contributor

llienfec commented Sep 14, 2018

More tested committees:
S6OH00163 – works
S2TX00106 – works
S6IN00191 - works
P80003353 – works
P80001571 – works
P00006825 - works

Candidates with RQ-5s - testing that they show up with the Form 2s and not with "other documents"
H8CA39257 - works
H8CA4316 - works
H8OK68016 - works

@llienfec
Copy link
Contributor

@patphongs I'm finished with testing. The only real problem is the data chart's responsiveness mentioned above. The testing items from the ticket above all passed. This round of testing is finished. We can either close this ticket and make a new one for the responsiveness issue, or we can add it on here. Whichever works best for you. Thanks!

@patphongs
Copy link
Member Author

Closing out this testing ticket as it has been completed. Opened 2 new follow-up issues to take care of the responsive and recent F2 issue:

#2381
#2382

@patphongs
Copy link
Member Author

There were no dealbreakers in the testing and we can proceed with deployment of the Candidate filings tab

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