Code review comment for lp:~hbrunn/banking-addons/lp-1106831-handle-P

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Good catch, but did you check whether this makes for a valid Clieop file? The most recent Clieop specifications don't mention it, but the specifications from March 2006 rule it out explicitely in page 15 of

It may be bank specific as well. Rabobank rules it out in page 12 of

So does ING in page 10 of

All of this seems to indicate that a better approach would be to replace the P by a zero. What do you think?

review: Needs Information

« Back to merge proposal