[Jgnash-tracker] [ jgnash-Patches-1858501 ] import mt940 transactions
jGnash Personal Finance
Brought to you by:
ccavanaugh
From: SourceForge.net <no...@so...> - 2008-01-05 11:11:03
|
Patches item #1858501, was opened at 2007-12-26 11:31 Message generated for change (Comment added) made by ccavanaugh You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=389746&aid=1858501&group_id=27240 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Arnout Engelen (raboofje) Assigned to: Nobody/Anonymous (nobody) Summary: import mt940 transactions Initial Comment: Patch (against jgnash2 trunk HEAD) for importing transactions from mt940-files, as provided by some banks (such as the Dutch ABN AMRO). This is not a ready-to-commit patch, but rather a request for comments and further collaboration. The importing is functional, but the UI for selecting the file to import and the accounts to affect is currently lacking. ---------------------------------------------------------------------- >Comment By: Craig Cavanaugh (ccavanaugh) Date: 2008-01-05 06:10 Message: Logged In: YES user_id=219057 Originator: NO resource files are modified by hand. Looking good! ---------------------------------------------------------------------- Comment By: Arnout Engelen (raboofje) Date: 2008-01-04 06:30 Message: Logged In: YES user_id=392382 Originator: YES I have some more improvements coming up, including generalization of the entire (Ofx)ImportDialog and the searching for duplicate transactions. Should I modify resource_*.properties by hand or are they somehow generated? ---------------------------------------------------------------------- Comment By: Craig Cavanaugh (ccavanaugh) Date: 2008-01-04 04:29 Message: Logged In: YES user_id=219057 Originator: NO Starting to look good. Haven't tried merging yet. Titles, text can probably be cleaned up to be generic. I can handle bits and piece like that. I've got some changes to investment transactions I want to stay focused on for the next week or so and then I will be ready to merge this is you are ready. Thanks! ---------------------------------------------------------------------- Comment By: Arnout Engelen (raboofje) Date: 2008-01-01 18:18 Message: Logged In: YES user_id=392382 Originator: YES Indeed I managed to reuse/generalize a lot of the OFX importing, particularly the wizard. This needs some general cleaning-up, but I'd be much obliged if you could take a look and see if you like the general idea. ---------------------------------------------------------------------- Comment By: Arnout Engelen (raboofje) Date: 2008-01-01 18:16 Message: Logged In: YES user_id=392382 Originator: YES File Added: mt940.patch ---------------------------------------------------------------------- Comment By: Arnout Engelen (raboofje) Date: 2008-01-01 11:45 Message: Logged In: YES user_id=392382 Originator: YES File Added: mt940.patch ---------------------------------------------------------------------- Comment By: Craig Cavanaugh (ccavanaugh) Date: 2007-12-26 13:13 Message: Logged In: YES user_id=219057 Originator: NO Sounds like a good addition. I took a quick look and the code looks clean and sane. The OFX Import wizard will be the best UI to copy / work with for importing the transactions. I still have more work to do to detecting duplicates, but it is stable enough to work with. An option does exist to refactor the OfxTransaction and OfxBank classes into generic import classes that can be used for OFX and the mt940 transactions. Regards, Craig ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=389746&aid=1858501&group_id=27240 |