Project

General

Profile

Spent time

Filters

Apply Clear

Hours: 344.53

Created Date User Activity Issue Comment Hours
27.01.2022 22:05 26.01.2022 Lari Taskula Communication Support #761: koha-illbackend-libris enhancements Requesting information from Kalle. 0.25 Actions
24.01.2022 14:43 24.01.2022 Lari Taskula Learning Support #761: koha-illbackend-libris enhancements Getting started. Reading task specification for our first tasks and checking out related resources. Installing koha-illbackend-libris. Running into some issues while configurating the module together with @samu.heiskanen that prevent creation of new requests. The issue is that branchcode does not get passed into the CGI-script. Is it missing from template? Debugging and checking out git history for related changes. It appears a recent commit has caused a bug. Contacting author of the module. 2.00 Actions
22.02.2022 14:27 22.02.2022 Lari Taskula Development Support #762: Edit patron information before registration of arrival Reading an email from Magnus. He had reported a bug on Bug 22321 that I had forgotten. Creating an unit test to cover the bug and implementing a fix. Pushing patches to Bugzilla and reporting back to Magnus. 0.75 Actions
14.02.2022 11:44 14.02.2022 Lari Taskula Development Support #762: Edit patron information before registration of arrival Designing this feature. It was requested that reserves.borrowernumber should also update when illrequests.borrowernumber is updated. It could be implemented in various ways, but deciding to extend Koha::Illrequest->store() with logic to replace borrowernumber also in reserves-table. Writing unit tests. Reading plenty of DBIx documentation. Figuring out how to retrieve the before-store() value of "borrowernumber" so that it could be compared with the borrowernumber used in "set" (or $illreq->borrowernumber(xxx)), but apprarently it's not possible without fetching the whole Koha-object again (or unless you somehow store it locally in the object). Managed to make it work by fetching the whole object again if borrowernumber is updated according to "get_dirty_columns". Adding an editable input field to ILL-request edit screen and writing a test plan. Pushing patches to Bugzilla. 4.50 Actions
14.02.2022 07:50 14.02.2022 Lari Taskula Communication Support #762: Edit patron information before registration of arrival Received a response from Magnus. Reading it. 0.25 Actions
10.02.2022 01:17 10.02.2022 Lari Taskula Learning Support #762: Edit patron information before registration of arrival Reading specification and trying to understand it. Browsing related views in Koha. Asking for more information. 1.00 Actions
24.01.2022 15:46 24.01.2022 Lari Taskula Development Support #762: Edit patron information before registration of arrival Creating a mock Libris response so that we can start development without Libris API keys. Using a dummy response from Libris documentation. Running into configuration issues. partner_code seems to not get populated for $ill_config. It is defined in $KOHA_CONF but it looks like the code is attempting to read it from config.yaml. Debugging. 1.00 Actions
11.02.2022 13:11 11.02.2022 Lari Taskula Development Support #776: “Details” under patrons in the staff interface a way to separate ordinary reservations from inter library loan reservations Continuing where Samu left off. Reading specification description, observing Koha's current capabilities for displaying item type, adding item type as new column, creating a new Bug, translating item type code to item type description and sending a new patch to Bugzilla. 1.00 Actions
20.04.2022 14:37 20.04.2022 Lari Taskula Maintenance Support #779: Bug 24239 - Let the ILL module set ad hoc hard due dates Rebasing 0.25 Actions
09.02.2022 23:56 10.02.2022 Lari Taskula Development Support #779: Bug 24239 - Let the ILL module set ad hoc hard due dates Reading Magnus's response. Creating a configuration variable that allows configuring which field will be used for due date. Adding business logic behind it. Testing. Creating a pull request. 1.25 Actions
02.02.2022 00:21 02.02.2022 Lari Taskula Development Support #779: Bug 24239 - Let the ILL module set ad hoc hard due dates Studying the specification doc for task a. Checking Bugzilla Bug 24239 and studying Magnus's patch. Reading discussion related to the bug. It was suggested that instead of custom attribute, a date_due column could be added to illrequests table. Implementing this and writing tests for this. 3.50 Actions
09.02.2022 23:54 10.02.2022 Lari Taskula Development Support #780: Charge patron a fee at receive Noticed a bug that causes logs to get spammed by a warning. Fixing it and sending a PR. 0.50 Actions
02.02.2022 01:45 02.02.2022 Lari Taskula Development Support #780: Charge patron a fee at receive Reading specification for step b. Implementing changes. 1.25 Actions
17.02.2022 20:24 16.02.2022 Lari Taskula Development Feature #786: Cron script reminder for unretrieved material Last fixes. illrequests.date_due was not updating so investigating the issue and sending a new commit to PR. 0.50 Actions
15.02.2022 19:48 15.02.2022 Lari Taskula Development Feature #786: Cron script reminder for unretrieved material Making fixes according to Magnus's feedback. Removed faulty logic that message transport types would be based on the messages sent at receival step. Changed it so that now we attempt to find all transport types defined for that specific letter code and send notifications to all of them. Pushing changes to GitHub PR. 1.50 Actions
14.02.2022 10:47 14.02.2022 Lari Taskula Documenting Feature #786: Cron script reminder for unretrieved material Improving documentation. Adding pod2usage and author & license. 0.25 Actions
14.02.2022 10:34 14.02.2022 Lari Taskula Development Feature #786: Cron script reminder for unretrieved material Some fixes were required after Magnus's reviewing. Removed useless query, removed a statement that overrided a configuration variable that eventually led into a bug on further iterations of the reminder loop, added some print statements for better logging, and fixed a bug in array size comparison that made the lastly configured letter to be skipped. 1.00 Actions
09.02.2022 22:59 10.02.2022 Lari Taskula Development Feature #786: Cron script reminder for unretrieved material Making improvements according to author's feedback. 0.75 Actions
03.02.2022 08:59 03.02.2022 Lari Taskula Development Feature #786: Cron script reminder for unretrieved material Studying module's already existing cronjob. Designing a flexible config.yaml configuration for this feature. Developing the feature and sending a PR to upstream repository. 6.00 Actions
15.02.2022 13:25 15.02.2022 Lari Taskula Development Support #820: Allow renewals for ILLs Bugfixes. Pushing new commits to PR. 0.50 Actions
14.02.2022 07:41 14.02.2022 Lari Taskula Development Support #820: Allow renewals for ILLs More testing and making bug fixes. Creating a new PR to upstream, ready for review. 0.75 Actions
11.02.2022 16:32 11.02.2022 Lari Taskula Development Support #820: Allow renewals for ILLs Developing this feature. Studying and experimenting how ILL backend statuses behave. It seems that it's enough to create a new placeholder status "IN_UTL_RENEW" that's not an actual status, but just a placeholder to add a button to illview screen. Its actual status (stored in the database) is still "IN_UTL". Using code from the receive step and adjusting it for the needs of renewals. Testing. Pushing changes to GitHub. 3.50 Actions
11.02.2022 16:29 11.02.2022 Lari Taskula Design Support #820: Allow renewals for ILLs Received a response from Magnus. Reading his more detailed specification and looking at current implementations. There seems to be something regarding renewals already. Deciding to use it. 0.75 Actions
11.02.2022 13:33 10.02.2022 Lari Taskula Communication Support #820: Allow renewals for ILLs Reading specs and trying to understand it. Confused about workflow, why isn't Koha's own renewal features enough? Asking Magnus for more information. 0.50 Actions
14.02.2022 12:10 14.02.2022 Lari Taskula Development Support #840: Don't update borrowernumber if old illrequest exists This was one of the tasks that we requested more information for. Reading Magnus's response to it and looking at the code. It seems we should override $borrower with the pre-existing borrower found in illrequests.borrowernumber if an old illrequests exists. Implementing this and sending a PR to upstream. 0.75 Actions
(51-75/182) Per page: 25, 50, 100, 500

Also available in: Atom CSV