Bug 28127

Summary: POS register details tables should have a transaction date
Product: Koha Reporter: Lucas Gass (lukeg) <lucas>
Component: Fines and feesAssignee: Lucas Gass (lukeg) <lucas>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: andrew, cbrannon, fridolin.somers, gmcharlt, jonathan.druart, martin.renvoize
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This patch adds the accountline timestamp to transactions on the register detail page.
Version(s) released in:
21.05.00
Circulation function:
Bug Depends on:    
Bug Blocks: 28390    
Attachments: Bug 28127: add transaction timestampt to register details tables
Bug 28127: add transaction timestampt to register details tables
Bug 28127: Add transaction timestamp to register details tables
Bug 28127: [20.05.x] Add transaction timestamp to register details tables

Description Lucas Gass (lukeg) 2021-04-10 15:46:51 UTC
Some librarians would fine it useful to include the transaction timestamp when viewing the two register detail tables (sales and past_sales). 

The tables are a bit in different than others in Koha, transaction occupy two table rows and the description spans 2 columns. This leaves some table cells unused. Rather than adding a new column to these tables I purpose adding the transaction timestamp into the unused <td> in the Transaction column.
Comment 1 Lucas Gass (lukeg) 2021-04-10 15:55:43 UTC
Created attachment 119435 [details] [review]
Bug 28127: add transaction timestampt to register details tables

To Test:
-Enable POS and setup some registers
-Make some transactions
-Go to Point of Sale > Register details and look at the two tables, sales and past sales
-No timestamp or transaction date
-Apply patch and look again
-See Tranactions dates in the Transaction column
Comment 2 David Nind 2021-04-12 10:05:18 UTC
Created attachment 119447 [details] [review]
Bug 28127: add transaction timestampt to register details tables

To Test:
-Enable POS and setup some registers
-Make some transactions
-Go to Point of Sale > Register details and look at the two tables, sales and past sales
-No timestamp or transaction date
-Apply patch and look again
-See Tranactions dates in the Transaction column

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Katrin Fischer 2021-04-17 23:47:25 UTC
Created attachment 119809 [details] [review]
Bug 28127: Add transaction timestamp to register details tables

To Test:
-Enable POS and setup some registers
-Make some transactions
-Go to Point of Sale > Register details and look at the two tables, sales and past sales
-No timestamp or transaction date
-Apply patch and look again
-See Tranactions dates in the Transaction column

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Jonathan Druart 2021-04-21 13:30:29 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 5 Fridolin Somers 2021-04-29 13:18:31 UTC
(In reply to Katrin Fischer from comment #3)
> Created attachment 119809 [details] [review] [review]
> Bug 28127: Add transaction timestamp to register details tables
> 
> To Test:
> -Enable POS and setup some registers
> -Make some transactions
> -Go to Point of Sale > Register details and look at the two tables, sales
> and past sales
> -No timestamp or transaction date
> -Apply patch and look again
> -See Tranactions dates in the Transaction column
> 
> Signed-off-by: David Nind <david@davidnind.com>
> 
> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Ah now there are 2 patches, older with "add" and newer with "Add".

But the older has been pushed to master :
https://git.koha-community.org/Koha-community/Koha/commit/572c0981c3f678fbb32476b8060ca79f0e642016

How fix this ?
Comment 6 Andrew Fuerste-Henry 2021-06-18 15:01:50 UTC
I'd love to see this backported to 20.05 if possible. Thanks!
Comment 7 Lucas Gass (lukeg) 2021-06-18 16:12:31 UTC
Created attachment 122166 [details] [review]
Bug 28127: [20.05.x] Add transaction timestamp to register details tables
Comment 8 Martin Renvoize (ashimema) 2021-08-04 07:49:28 UTC
Personally.. I think if this is backported then bug 28390 should be too in quick succession as it fixes a bug with this implimentation.
Comment 9 Fridolin Somers 2021-08-04 08:02:42 UTC
Comment on attachment 119809 [details] [review]
Bug 28127: Add transaction timestamp to register details tables

Patch with "add" was pushed to master so I obsolete patch with "Add".
It will be missing "Signed-off-by: Katrin Fischer" but we know it ;)
Comment 10 Fridolin Somers 2021-08-04 08:04:45 UTC
OK so we wait on Bug 28390