See bug 15504
Note that bug 35027 adds holds.
Created attachment 156955 [details] [review] Bug 35030: Add 'article' to patron activity triggers This patch adds 'article' to the list of triggers available for tracking patron activity. Test plan 1) Select 'Placing an article request TrackPatronLastActivityTriggers system preference 2) As a staff member, place a hold on any item for a test user 3) Confirm that the borrowers.lastseen field is updated for that test borrower
Could you please extend the unit test ?
Created attachment 156961 [details] [review] Bug 35030: Add 'article' to patron activity triggers This patch adds 'article' to the list of triggers available for tracking patron activity. Test plan 1) Select 'Placing an article request TrackPatronLastActivityTriggers system preference 2) As a staff member, place a hold on any item for a test user 3) Confirm that the borrowers.lastseen field is updated for that test borrower
Tests added :)
Created attachment 156964 [details] [review] Bug 35030: Add 'article' to patron activity triggers This patch adds 'article' to the list of triggers available for tracking patron activity. Test plan 1) Select 'Placing an article request TrackPatronLastActivityTriggers system preference 2) As a staff member, place a hold on any item for a test user 3) Confirm that the borrowers.lastseen field is updated for that test borrower Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Moving to PQA, combining with SO
Marking as BLOCKED until bug 15504 is PQA. Please adjust dependencies if this is wrong.
Pushed to master for 23.11. Nice work everyone, thanks!
Depends on Bug 15504 not in 23.05.x