On 08-02-13 11:48, Mike Evans wrote:
On Fri, 08 Feb 2013 10:13:01 +0100
Geert Janssens <[email protected]> wrote:
Last item is on our Feature Schedule [1]: "Reversed sort order by
date for transaction", a feature Mike Evans has been working on. As
far as I can see it's mostly done. I reviewed his patch and gave my
feedback. I don't know what kept Mike from completing it. Even as is,
the patch is usable and useful already.
Re ^: This currently doesn't save the reverse sort setting between runs,
and I really didn't like the empty_split being at the bottom when
reverse sorting. It is functional for people wanting to compare their
on-screen, reverse-sorted bank accounts though. I just ran out of
time, enthusiasm, money... I'll commit it as is but subsequent commits
mean that the patch is obsolete and I'll have to revisit the code again.
I also marked Bug 682732 as a blocker because it simply doesn't work
for me. I think we either revert to the old CSV importer or fix the new
version. Perhaps others could test it and update the bug.
Mike Evans
Hi Mike,
Thanks for the feedback. I notice you have committed your revisited code
in the meantime already. Good ! Though not fully finished, I don't feel
the state of the reverse sort is now blocking for feature freeze.
The bug you refer to isn't either (for feature freeze). It probably is
for release and we will have to look into it before that time.
Geert
_______________________________________________
gnucash-devel mailing list
[email protected]
https://lists.gnucash.org/mailman/listinfo/gnucash-devel
