Fix double last entry #17

Merged
merged 3 commits into from Mar 30, 2013

Conversation

Projects
None yet
2 participants
Contributor

jodell commented Mar 29, 2013

First of all, thanks to @jm81 and @barmstrong for putting time and energy into this library. We love it.

This pull request deals with a bug we noticed with parsing that duplicated the last line entry. I'm not entirely sure if there isn't some case that warrants the line that got removed, but this pull request adds the failing test and fix for it (and works for us when we parse and display sent ACH files now).

@jm81 jm81 added a commit that referenced this pull request Mar 30, 2013

@jm81 jm81 Merge pull request #17 from avantcredit/fix-double-last-entry
Fix double last entry
7c97201

@jm81 jm81 merged commit 7c97201 into jm81:master Mar 30, 2013

Owner

jm81 commented Mar 30, 2013

I've pushed an updated gem. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment