Right now, the description on TrackLastPatronActivity just says "Do/Don't track last patron activity. Every time a patron will connect, the borrowers.lastseen will be updated with the current time." It should say something about what it specifically records.
I recommend we replace that second sentence with: The first time each day that a patron either logs into the OPAC or connects to Koha via SIP, borrowers.last seen will update with the current date and time.
Created attachment 121540 [details] [review] Bug 28467: Update wording of TrackLastPatronActivity syspref To Test: 1 - look up TrackLastPatronActivity 2 - Read it, be dissatisfied with it 3 - apply patch, restart, reload page 4 - be happier with the new wording
I updated it to read "The first time each day that a patron either logs into the OPAC or connects to Koha via SIP, borrowers.last seen will update with the current date and time."
Created attachment 121576 [details] [review] Bug 28467: Update wording of TrackLastPatronActivity syspref To Test: 1 - look up TrackLastPatronActivity 2 - Read it, be dissatisfied with it 3 - apply patch, restart, reload page 4 - be happier with the new wording Signed-off-by: David Nind <david@davidnind.com>
Created attachment 121581 [details] [review] Bug 28467: Update wording of TrackLastPatronActivity syspref To Test: 1 - look up TrackLastPatronActivity 2 - Read it, be dissatisfied with it 3 - apply patch, restart, reload page 4 - be happier with the new wording Signed-off-by: David Nind <david@davidnind.com>
Created attachment 121582 [details] [review] Bug 28467: Update wording of TrackLastPatronActivity syspref To Test: 1 - look up TrackLastPatronActivity 2 - Read it, be dissatisfied with it 3 - apply patch, restart, reload page 4 - be happier with the new wording Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Current status: Passed QA
Created attachment 121583 [details] [review] Bug 28467: (QA follow-up) Remove errorenous space in borrowers.lastseen Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Is there a reason why we only update that info once a day? But I verified... it would not update after my first login.
Pushed to master for 21.11, thanks to everybody involved!
Pushed to 21.05.x for 21.05.01