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

Feature/add purpose filter #346

Merged
merged 2 commits into from
Jul 21, 2015
Merged

Conversation

jmcarp
Copy link
Contributor

@jmcarp jmcarp commented Jul 21, 2015

Add disbursement purpose filter and column to Schedule B data table. @noahmanger: the original issue only mentioned the filter, but I think it's helpful to include filtered columns in the table if possible so users can understand how the filters work. I can take out the extra column if it's distracting though.

@msecret
Copy link
Contributor

msecret commented Jul 21, 2015

I see the purpose filter on the filters tab and the correct purpose in the table, but when I type something into the purpose filter, it doesn't seem to actual filter anything. Is that expected at this point?

@noahmanger
Copy link
Contributor

Yep, you're right. Good catch adding the column.

On Tue, Jul 21, 2015 at 9:17 AM, Marco Segreto [email protected]
wrote:

I see the purpose filter on the filters tab and the correct purpose in the
table, but when I type something into the purpose filter, it doesn't seem
to actual filter anything. Is that expected at this point?


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

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

@jmcarp
Copy link
Contributor Author

jmcarp commented Jul 21, 2015

@msecret this won't work using the dev API until https://github.com/18F/openFEC/pull/1076 is merged. If you're running your own API server and pointing at the production RDS, it should work. Sorry, I should've mentioned this before.

msecret pushed a commit that referenced this pull request Jul 21, 2015
@msecret msecret merged commit a8ab02d into fecgov:develop Jul 21, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants