Summary: | Add ILL as option to TrackLastPatronActivityTriggers | ||
---|---|---|---|
Product: | Koha | Reporter: | Magnus Enger <magnus> |
Component: | ILL | Assignee: | Bugs List <koha-bugs> |
Status: | NEW --- | QA Contact: | |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | pedro.amorim, tomascohen |
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: |
Description
Magnus Enger
2024-06-26 12:04:59 UTC
Would simply having "Place an ILL request" as an option satisfy this use case? I suggest this be handled in core sub backend_create. It would update a patron's lastseen regardless of the type of request, or if it results in a loan or not. It would also be in line with current options (very specific, single, actions), and future use cases can also be considered to be implemented the same way e.g. "Cancelling an ILL request". (In reply to Magnus Enger from comment #0) > - Would it be OK to have "ill" listed as an activity for > TrackLastPatronActivityTriggers even if the ILL module is not activated? It would be okay imo (not critical), but I think ideally it shouldn't list the option if ILLModule is disabled. Although I can see the following happening: 1) "Place an ILL request" option is enabled in TrackLastPatronActivityTriggers 2) ILLModule gets disabled 3) TrackLastPatronActivityTriggers sys pref is updated ("Place an ILL request" is now gone because it wasn't listed thus not submitted) 4) ILLModule gets enabled 5) TrackLastPatronActivityTriggers does not have the "Place an ILL request" enabled. Not very relevant or critical, but something to keep in mind if this is a first (having conditional list options in a sys pref) > Would simply having "Place an ILL request" as an option satisfy this use case? I suggest this be handled in core sub backend_create. It would update a patron's lastseen regardless of the type of request, or if it results in a loan or not.
Yeah, that sounds like an excellent idea.
|