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

Feature/committee dates noah #170

Merged
merged 27 commits into from
May 13, 2015
Merged

Feature/committee dates noah #170

merged 27 commits into from
May 13, 2015

Conversation

noahmanger
Copy link
Contributor

While trying to do clean up locally accidentally deleted the origin branch, thus closing the last PR.

Apologies for this becoming the pull request from 🔥

The last few commits respond to the comments @jmcarp and @msecret made on the last one:

  • Adding tests for the new filters
  • Creating a more succinct last_n_characters filter
  • Replacing jQuery with $

LindsayYoung and others added 27 commits May 6, 2015 16:36
To make it correct it needs to be calculated from cycle instead of year.
so it would be "reports filed 2007-2008" but when I tried
{{ cycle - 1 }} - {{ cycle }}
It couldn't do the math. So I am removing the wrong label and leaving a note.
New filter necessary to get the last three characters from the
beginning_image_number to build the url
Conflicts:
	__init__.py
	templates/committees-single.html
  - adapting date formating to new style
  - getting rid of extra {% with committee=context() %}
  - made formatting clearer on recent reports
 (We still need to go over what fields we are showing)
assert app.date_filter_md(None) == ''


def test_last_n_characters():
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

msecret pushed a commit that referenced this pull request May 13, 2015
@msecret msecret merged commit 43e0d09 into develop May 13, 2015
@LindsayYoung LindsayYoung deleted the feature/committee-dates-noah branch June 2, 2015 21:45
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

5 participants