Re: [Kmymoney-devel] Review Request 120260: CSV import wizard does not appear to be working.

2014-09-18 Thread Cristian Oneț
> On Sept. 18, 2014, 6:57 p.m., Cristian Oneț wrote: > > It fixes the reported problem. Don't forget to add the 'BUG: NR' and > > 'REVIEW: NR' markers in the commit message (each on it's own line). > > Allan Anderson wrote: > Good. Did you notice specifically the behaviour of the radio but

[Kmymoney-devel] [kmymoney4] [Bug 337208] Security names after a security with a missing trading symbol are replaced by that symbol.

2014-09-18 Thread george
https://bugs.kde.org/show_bug.cgi?id=337208 --- Comment #4 from geo...@wildturkeyranch.net --- (In reply to allan from comment #1) > In case there are any other features in your file that influence the > reported behaviour, it would be very helpful if you could provide a simple > file, obviously w

[Kmymoney-devel] [kmymoney4] [Bug 339192] New: OFX download leaves "Missing assignment" Edit requries change to Enter.

2014-09-18 Thread george
https://bugs.kde.org/show_bug.cgi?id=339192 Bug ID: 339192 Summary: OFX download leaves "Missing assignment" Edit requries change to Enter. Product: kmymoney4 Version: git master Platform: Fedora RPMs OS: Li

Re: [Kmymoney-devel] Question on new investment account

2014-09-18 Thread Jack
But the only balance in an investment account is number of shares, not dollars. I can see allowing a starting number of shares when creating a security or adding a security to an investment account. Cristian Oneț wrote: >I guess that by allowing an opening balance a user can start tracking >an

Re: [Kmymoney-devel] Review Request 120260: CSV import wizard does not appear to be working.

2014-09-18 Thread Allan Anderson
> On Sept. 18, 2014, 6:57 p.m., Cristian Oneț wrote: > > It fixes the reported problem. Don't forget to add the 'BUG: NR' and > > 'REVIEW: NR' markers in the commit message (each on it's own line). Good. Did you notice specifically the behaviour of the radio buttons? - Allan --

Re: [Kmymoney-devel] Review Request 120260: CSV import wizard does not appear to be working.

2014-09-18 Thread Cristian Oneț
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120260/#review66860 --- Ship it! It fixes the reported problem. Don't forget to add t

Re: [Kmymoney-devel] Question on new investment account

2014-09-18 Thread Cristian Oneț
I guess that by allowing an opening balance a user can start tracking an already existing account without caring about historic transactions. Regards, Cristian 2014-09-18 19:04 GMT+03:00 Jack : > For some of the testing I've done recently, I created a new Investment > account, and noticed that it

[Kmymoney-devel] Question on new investment account

2014-09-18 Thread Jack
For some of the testing I've done recently, I created a new Investment account, and noticed that it is possible to specify an opening balance. This is the Investment account, not the Brokerage account. Since the only content of an Investment account should be Securities, this seems wrong

Re: [Kmymoney-devel] Review Request 120153: Remove the OFX investment transactions amount sign inversion hack.

2014-09-18 Thread Jack Ostroff
> On Sept. 12, 2014, 12:10 p.m., Allan Anderson wrote: > > I've confirmed that when the OFX file (produced from the logfile) is > > imported into a checking account, the sign now is correct. > > I've also imported a number of my own non-OFX files and they too are OK. > > I'm not an OFX user, and

Re: [Kmymoney-devel] Review Request 120153: Remove the OFX investment transactions amount sign inversion hack.

2014-09-18 Thread Jack
On 2014.09.12 08:16, Cristian Oneț wrote: > On Sept. 12, 2014, 12:10 p.m., Allan Anderson wrote: I've confirmed that when the OFX file (produced from the logfile) is imported into a checking account, the sign now is correct. I've also imported a number of my own non-OFX files and they too are

[Kmymoney-devel] [kmymoney4] [Bug 339175] New: Multiple selection of transactions for "matching" not allowed, and transactions don't match automatically on OFX direct download

2014-09-18 Thread Mike Wolfe
https://bugs.kde.org/show_bug.cgi?id=339175 Bug ID: 339175 Summary: Multiple selection of transactions for "matching" not allowed, and transactions don't match automatically on OFX direct download Product: kmymoney4

[Kmymoney-devel] Review Request 120260: CSV import wizard does not appear to be working.

2014-09-18 Thread Allan Anderson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120260/ --- Review request for KMymoney. Repository: kmymoney Description ---

Re: [Kmymoney-devel] Fwd: Re: [kmymoney4] [Bug 339044] CSV import wizard does not appear to be working.

2014-09-18 Thread Allan
On 18/09/14 12:08, Cristian Oneț wrote: OK, but where is the patch? Please post a review request. Regards, Cristian I'd been hoping to have it tested first, before submitting it, as I can't test on XP, but it looks like that's a non-starter. This is a pain. I'll do a review request later.

Re: [Kmymoney-devel] Fwd: Re: [kmymoney4] [Bug 339044] CSV import wizard does not appear to be working.

2014-09-18 Thread Cristian Oneț
OK, but where is the patch? Please post a review request. Regards, Cristian 2014-09-18 13:22 GMT+03:00 Allan : > I'd intended to cc this to the list, in case anyone can offer > comment/explanation. > > The patch I refer to is to address the incorrect display of the CSV import > plugin, when on Wi

[Kmymoney-devel] Fwd: Re: [kmymoney4] [Bug 339044] CSV import wizard does not appear to be working.

2014-09-18 Thread Allan
I'd intended to cc this to the list, in case anyone can offer comment/explanation. The patch I refer to is to address the incorrect display of the CSV import plugin, when on Windows and Ubuntu. I'm only able to test on Ubuntu, where all now looks fine. It consists mainly of modified UI file