Merge lp:~wolfer7/wxbanker/bugfix-lp-534923 into lp:wxbanker
Proposed by
Michael Rooney
Status: | Merged |
---|---|
Merged at revision: | not available |
Proposed branch: | lp:~wolfer7/wxbanker/bugfix-lp-534923 |
Merge into: | lp:wxbanker |
Diff against target: | 0 lines |
To merge this branch: | bzr merge lp:~wolfer7/wxbanker/bugfix-lp-534923 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Rooney | Approve | ||
Review via email: mp+21107@code.launchpad.net |
This proposal supersedes a proposal from 2010-03-09.
Description of the change
changed csv-importer. previously one could only skip the first line, now it is possible to exactly specify how many rows should be skipped at the beginning of a file.
also added 'comdirect' to the shipped csv profiles.
To post a comment you must log in.
Looks good, thanks for this feature and merge request! I can happily merge this after a few quick things. I would have just fixed the first two myself and pointed them out for your reference but the third one requires data from you.
- It looks like there is a typo: "Linex to skip" 'linesToSkip' ]>linesSkipped: linesSkipped+=1", I would remove that logic and just change the for to "for row in csvReader[ settings[ 'linesToSkip' ]:]:" ertests. py for test examples, and you'll want to put an example CSV with a few transactions in the data/fixtures directory with the other .csv files.
- Instead of doing the counting logic like "if settings[
- All the included currencies should have tests with a small fixture CSV. It is fairly straightforward, see tests/csvimport
Let me know if you have any questions and I look forward to merging this, thanks again!