Skip to content
This repository has been archived by the owner on Dec 23, 2017. It is now read-only.

Deal with Cycle select on committee filings tab #612

Closed
LindsayYoung opened this issue Sep 9, 2015 · 3 comments
Closed

Deal with Cycle select on committee filings tab #612

LindsayYoung opened this issue Sep 9, 2015 · 3 comments

Comments

@LindsayYoung
Copy link
Contributor

The cycle filter changes the url and is not connected to the form. I have only thought of hacky ways to deal with this. Any ideas?

The last remaining issue on 18F/openFEC#1028

@noahmanger
Copy link
Contributor

This currently works for me. Updating the cycle updates the reports and URL. Am I missing something?

@jmcarp
Copy link
Contributor

jmcarp commented Sep 10, 2015

@noahmanger the issue is the page refresh. Changing inputs inside the filter form (amendment indicator, election type, etc.) loads new filings data without refreshing the page. Changing the cycle select refreshes the page. So yes, changing the cycle select gives the expected data, but it's kind of clunky.

@noahmanger
Copy link
Contributor

Ah. Gotcha. Considering that we're likely to move to rendering more data
client side on these pages, would it make sense to wait to do a universal
cycle select update then? I'd like to at least keep the behavior of the
cycle select the same for now.

On Thu, Sep 10, 2015 at 6:41 AM, Joshua Carp [email protected]
wrote:

@noahmanger https://github.com/noahmanger the issue is the page
refresh. Changing inputs inside the filter form (amendment indicator,
election type, etc.) loads new filings data without refreshing the page.
Changing the cycle select refreshes the page. So yes, changing the cycle
select gives the expected data, but it's kind of clunky.


Reply to this email directly or view it on GitHub
#612 (comment)
.

Noah Manger
18F http://18f.gsa.gov | Work: 415-696-6146

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants