transaction index is not supported with non-journal files [..|2..] #394

Closed
simonmichael opened this Issue Aug 14, 2016 · 0 comments

Projects

None yet

1 participant

@simonmichael
Owner
simonmichael commented Aug 14, 2016 edited

Transactions in journal files have a 1-based index, which can be seen eg with hledger print -O csv.
When reading from a csv or timeclock or timedot file, all transaction indexes are 0. This can be seen on hledger-ui's transaction screen.

@simonmichael simonmichael referenced this issue Aug 14, 2016
Open

hledger-ui unfinished tasks [16] #378

4 of 11 tasks complete
@simonmichael simonmichael added a commit that referenced this issue Aug 14, 2016
@simonmichael lib: non-journal formats now produce transaction ids #394
Transactions are now numbered consistently during journal finalisation,
rather than just in the journal reader. Also transaction knot-tying has been
moved out of journalBalanceTransactions.
72c3947
@simonmichael simonmichael added a commit that referenced this issue Aug 14, 2016
@simonmichael timeclock: fix order of transaction indexes #394
Timeclock transaction ids now count up rather than down.

Also, remove old code for appending timeclock transactions to journal transactions,
a holdover from the days when both were allowed in one file.
4ceda8f
@simonmichael simonmichael changed the title from transaction index is not supported with non-journal files to transaction index is not supported with non-journal files [..|2..] Aug 14, 2016
@mstksg mstksg added a commit to mstksg/hledger that referenced this issue Feb 3, 2017
@simonmichael @mstksg + mstksg lib: non-journal formats now produce transaction ids #394
Transactions are now numbered consistently during journal finalisation,
rather than just in the journal reader. Also transaction knot-tying has been
moved out of journalBalanceTransactions.
5027a8c
@mstksg mstksg added a commit to mstksg/hledger that referenced this issue Feb 3, 2017
@simonmichael @mstksg + mstksg timeclock: fix order of transaction indexes #394
Timeclock transaction ids now count up rather than down.

Also, remove old code for appending timeclock transactions to journal transactions,
a holdover from the days when both were allowed in one file.
3a86fcc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment