Re: Review Request 128511: Move displayLine to CSVWizard

2016-07-24 Thread Łukasz Wojniłowicz
> On July 24, 2016, 12:54 p.m., Thomas Baumgart wrote: > > kmymoney/plugins/csvimport/csvwizard.cpp, line 514 > > > > > > Remove if not needed Ups! that shouldn't be commented out, but works anyhow :). > On Ju

Re: Review Request 128511: Move displayLine to CSVWizard

2016-07-24 Thread Allan Anderson
> On July 24, 2016, 10:54 a.m., Thomas Baumgart wrote: > > Again, just visual inspection. Looks good to me besides the two little > > nitpicks. Someone with more knowledge about the functionality should check > > as well. If that includes me, then I agree. - Allan -

Re: Review Request 128511: Move displayLine to CSVWizard

2016-07-24 Thread Thomas Baumgart
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128511/#review97792 --- Again, just visual inspection. Looks good to me besides the t

Review Request 128511: Move displayLine to CSVWizard

2016-07-24 Thread Łukasz Wojniłowicz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128511/ --- Review request for KMymoney. Repository: kmymoney Description ---