-
Notifications
You must be signed in to change notification settings - Fork 41
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
Make a plan for importing various "Reports about the FEC" #779
Comments
Progress update on a suggested approach over in this PR https://github.com/18F/fec-cms/pull/804 |
@patphongs and @emileighoutlaw I made a new spreadsheet 🔒 that we can use for organizing these reports. This has columns for all of the fields we need for the import script, it just needs to be filled out. I set up the category column to validate based on the sub-categories available for each parent category. Can I hand this off to one of you to fill take from here? |
Want to divvy this up between the two of us, @patphongs ? |
@emileighoutlaw Sure let's look at this together. Let me talk to you about it. |
tl;dr I've been working with the spreadsheet of report content 🔒. I think we may need to adjust our design. Current design pattern:
🎉 Yay! So awesome! But with real content, the fields are regularly repetitive, a lá:
😭 Not awesome. My first fix for this repetition was to tailor the link field to avoid duplication:
🎉 Yay again!
So! That leads me to my questions.
If yes and yes, this seems like a good solution. We can stop here. If no ⬇️ : We should probably tweak the design. These feeds are going to house the content folks are least willing and able to adjust by hand. If we set folks up with multiple fields to fill out, we're paving the way for future duplication like this (h/t to Corey M. for sharing this case study from her California work): That first link to salary scales is a repetition nightmare. Basically, I looked at a lot of ways that we could tailor this content to make it more friendly to the design. But I think anything that has to be done by hand is going to be a mistake. Instead, I'd recommend we move to a design with fewer fields. Like this one: The benefit of this one is that we have only:
That's a giant bucket full of thoughts I just dumped here. So I'll give you time to digest @noahmanger. Also tagging @jenniferthibault, because this has to do with design. We can discuss in person, if we don't want to keep talking in GitHub. |
@emileighoutlaw yeah, those solution's aren't tricky at all. We'd just have one field for document title and could include it twice in the template. I think I'd prefer the simpler approach that just shows the title with some indicator that it's a PDF. Though just noting that we won't have the "size" value unless you add it by hand. |
Our spreadsheet should be totally filled with content now, except for the few documents that we couldn't find dates for! |
Forgot to ping @noahmanger in ^^ |
Plan made and a new issue made for importing these #828 . I'll make the small change to the layout in a PR. |
The "Reports about the FEC" section in "About" calls for several different pages, each of which acts as a feed of the various reports about the agency (IG reports, budget docs, etc.). Many of these are PDFs, with an occasional HTML page.
I'd like to use this issue to:
The text was updated successfully, but these errors were encountered: