Bug 33647

Summary: Display borrowers.lastseen in patron record
Product: Koha Reporter: Klas Blomberg <klas.blomberg>
Component: PatronsAssignee: Lari Strand <lmstrand>
Status: Needs Signoff --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: bibliwho, Chip.Halvorsen, cj.lynce, george, gmcharlt, hebah, kyle, lmstrand, rcoert
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 33647: Display borrowers.lastseen on patronbriefinfo
Bug 33647: Display borrowers.lastseen on patronbriefinfo
Bug 33647: (follow-up) Don't try to show last seen info if there is none
Bug 33647: (follow-up) Add UI improvements

Description Klas Blomberg 2023-05-02 09:31:07 UTC
It would be nice if borrower.lastseen could be displayed at Patron details / Library use.

Sometimes it gives a clue why a borrower who doesn't circulate anything is still active
Comment 1 Chip Halvorsen 2025-08-11 20:28:00 UTC
Circulation Staff would really like to see this. lastseen has much more value to them than Last Updated.
Comment 2 George Williams (NEKLS) 2025-08-12 15:17:52 UTC
+1 - this would be a nice feature
Comment 3 Lari Strand 2025-08-14 11:11:32 UTC
Created attachment 185382 [details] [review]
Bug 33647: Display borrowers.lastseen on patronbriefinfo

To test:

1. Apply patch
2. Check the top left corner patron info for a new element after Updated on field called patronlastseenon, note that if patron has no lastseen date, atfer the text Last seen on there is no date/time data visible
Comment 4 Owen Leonard 2025-08-14 16:02:56 UTC
Created attachment 185402 [details] [review]
Bug 33647: Display borrowers.lastseen on patronbriefinfo

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 5 Owen Leonard 2025-08-14 16:02:59 UTC
Created attachment 185403 [details] [review]
Bug 33647: (follow-up) Don't try to show last seen info if there is none
Comment 6 Lari Strand 2025-08-15 08:17:18 UTC
In my patch the idea was to let the user know when patron has never used the account in a visual way in the element. What if we showed this somehow like:
"Last seen on: never" or something like that instead? Maybe use : syntax with updated on too for uniformity.
Comment 7 Owen Leonard 2025-08-15 11:21:54 UTC
(In reply to Lari Strand from comment #6)
> "Last seen on: never" or something like that instead?

That makes sense to me
Comment 8 Chip Halvorsen 2025-08-15 14:06:28 UTC
(In reply to Owen Leonard from comment #7)
> (In reply to Lari Strand from comment #6)
> > "Last seen on: never" or something like that instead?
> 
> That makes sense to me

Same here. Circulation staff would know instantly that there's been no activity, instead of possibly wondering why the field isn't there.
Comment 9 Lari Strand 2025-08-19 09:10:23 UTC
Created attachment 185535 [details] [review]
Bug 33647: (follow-up) Add UI improvements

Use colon for Updated on and Last seen on texts
If lastseen value doesn't exist, show the text 'Never' in UI
Comment 10 Katrin Fischer 2025-08-19 12:26:14 UTC
Just wondering about older databases or databases where the feature is turned off. 

2 suggestions:

1) Could we use "not recorded" or similar when there is no date?
2) Should we make the display of the data dependent on the feature being turned on? (at least one trigger active)