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

Fix errors on committee profile pages #138

Merged
merged 1 commit into from
May 6, 2015

Conversation

LindsayYoung
Copy link
Contributor

#695

  • fixed mislabeled transfers row
  • removed rows that were not producing real data
  • fixed spelling errors
  • Expanded the explanation when no data is shown

 - removed rows that were not producing real data
 - fixed spelling errors
 - Expanded the explanation when no data is shown
jmcarp added a commit that referenced this pull request May 6, 2015
@jmcarp jmcarp merged commit a58339f into develop May 6, 2015
@jmcarp jmcarp removed the in progress label May 6, 2015
<h3 class="section-header">Detailed Summary</h3>
{% with committee=context() %}

{% set table_data = [
(committee.totals.0.individual_itemized_contributions, 'Itemized Individual Contributions'),
(committee.totals.0.individual_unitemized_contributions, 'Unitemized Individual Contributions'),
(committee.totals.0.transfers_to_affiliated_committee, 'Transfer from a Committee'),
(committee.totals.0.transfers_to_affiliated_committee, 'Transfer to a Committee'),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this field go under disbursements, rather than income? (Actually, it is duplicated in the table now.)

In the nesting refactor branch I changed this to transfers_from_other_authorized_committee, but we need to find an appropriate analogous field to pull from the PacOrParty data for this.

@LindsayYoung LindsayYoung deleted the feature/committee-summary-changes#695 branch May 11, 2015 17:23
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.

None yet

4 participants