-
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
Temporarily Remove SSF/Leadership PACS options from Disbursements datatable #2979
Temporarily Remove SSF/Leadership PACS options from Disbursements datatable #2979
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we're also removing Leadership PACs from the dropdown too.
Codecov Report
@@ Coverage Diff @@
## release/public-20190625 #2979 +/- ##
========================================================
Coverage 74.83% 74.83%
========================================================
Files 119 119
Lines 7126 7126
Branches 615 615
========================================================
Hits 5333 5333
Misses 1793 1793 Continue to review full report at Codecov.
|
@JonellaCulmer , I added exclusion for Leadership PACs as requested in your review. |
Thanks @johnnyporkchops! Per our discussion, can you:
|
With this PR:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks @johnnyporkchops!
Summary
Remove
Separate segregated funds (SSF)
andLeadership PAC
options from Disbursements datatable > Filters > Spender Committee Type > PACS dropdownRelated PR: Remove SSFs from receipts and individual contributions datatable #2941
Changed spender-committee-types.jinja partial to macro, added ssf conditional option.
It may have been possible to reuse the commitee-types.jinja macro here since currently the options are the same for both rasing and spending datatables-- But, this macro should remain separate in case we ever need to have different filter options for Disbursements VS. raising datatables( Receipts , Individual Contributions) .
Impacted areas of the application
renamed: data/templates/partials/filters/spender-committee-types.jinja -> data/templates/macros/filters/spender-committee-types.jinja
modified: data/templates/partials/disbursements-filter.jinja
How to test
Filters > Spender Committee Type > PACS dropdown
PAC with non-contribution account - qualified