fecgov / openFEC-web-app

DEPRECATED See https://github.com/18F/fec-cms for fec.gov's code
Other
43 stars 31 forks source link

Change default election year to most recent only #2084

Closed PaulClark2 closed 7 years ago

PaulClark2 commented 7 years ago

This seems to be causing some confusion. Let's change this to default to the most recent election year. I don't think that surprises folks.

image

: https://www.fec.gov/data/candidates/?election_year=2014&election_year=2016&election_year=2018&has_raised_funds=true

jwchumley commented 7 years ago

I think we should throw it over to Nicky. My thought is to remove the default year entirely since this is the advanced search. Without the defaults the current behavior would be that it would find everybody and when you click on one you automatically get the most recent activity year for that committee.

rhoughfec commented 7 years ago

I think this will be a common issue for enforcement attorneys and commission offices since a lot of their cases are 4-6 years behind the current election cycle. I agree Jeff, I think if we can remove the default entirely that would be great. Thanks guys!

PaulClark2 commented 7 years ago

@nickykrause can you take a look at this?

nickykrause commented 7 years ago

Sure! I probably won't be able to 🤔 on it until later today, after usability testing/retro, but I am happy to look at it.

noahmanger commented 7 years ago

Easy enough fix. Removing the default probably does make sense in this case.

rhoughfec commented 7 years ago

Thanks @noahmanger !

nickykrause commented 7 years ago

@noahmanger - I just looked at it, and I agree. I wondered if there was a reason that we decided to have a default? Did it have anything to do with trying to add some limit to the number of results right up front? If so, is that something we actually need to be concerned about, or are we cool with removing the default and letting all the results appear (paginated, of course)?

noahmanger commented 7 years ago

IIRC (and this was very long ago) it was an attempt to just limit the universe to a more relevant set of results be default, but that feels less important now.

noahmanger commented 7 years ago

I assume we want to do the same for /committees?

noahmanger commented 7 years ago

Will be fixed in https://github.com/18F/openFEC-web-app/pull/2083

nickykrause commented 7 years ago

Makes sense to me