-
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
Implement content for downloadable data accordions in Adv. data tabs #1400
Comments
II went through the data catalog S3 location spreadsheet to define links by year. This blocker is mostly resolved, links by year are included in the spreadsheet. I also highlighted the URL patterns and when those patterns don't work. Two small issues remain:
Still in progress - QCing that the data that would be downloaded matches classic. Method: use the data catalog on classic to perform the search, see how many results come up. Then, download the new file, compare the number of results in excel (use line number, minus 1 line for the header row). The number of results should be the same for both searches. The QCing process has been started, but not finished. Let me know if you have any questions! Big thanks to @jenniferthibault for showing me how to browse the S3 bucket! |
Continued quality checking the data to make sure that the new downloads match what is available on the classic site. In all but one instance, the results match exactly. For ECs, there's a slight difference in output for 2018. This may be that the new bulk download's output (showing 4 lines of results) is in a different format from the one on classic (showing 1 result). Details: The 1 result is on both outputs. 2 of the results may be descriptor lines for the result above. A result labeled for the 2016 election is also showing up. I don't have time to dive deeper today. Will try to look later this week, but may need to wait for Paul to help explain. |
Thank you so much for double checking this data @llienfec and unblocking this issue and the PR! Connecting the spreadsheet to this issue: @johnnyporkchops looks like this can move #1603 mostly forward, but we need to check with @PaulClark2 on Lauren's questions above to find the right files or see if these are all we have on the following categories:
Lauren has put comments in the 📝 Google Doc of accordion content + in the spreadsheet linked just above. Between the two, if questions come up as you go along please ask! |
Thanks @jenniferthibault and @llienfec for pulling this together! |
where are we with this issue? |
@johnnyporkchops This work is finished and will be in the next release, right? |
After setting up the new Adv data page and accordion styles in https://github.com/18f/fec-cms/issues/1309, add the accordions to the tabs and fill them with real content and links.
Designs from fecgov/openFEC-web-app#2318
🖥 Invision prototype
Click into each accordion to see the different content and layouts
Basic design templates to implement:
Using:
Completion criteria:
The text was updated successfully, but these errors were encountered: