User:Victor Grousset - tuxayo/Community work/Reports/2024-04

QA

Notes:

• An attempt results in feedback provided on the bug. To ask if I correctly followed the test plan due to results that I couldn't interpret. Or if there is indeed an issue. Or questions to clarify something in the code.
• Partial QA is a way to contribute to a ticket that I can't QA alone but for which I can confidently review only part of the code or aren't sure enough about understanding the functional need and change.

Hackfest: Input on "Stricter naming in schema + code" and try to summarize which changes should be acceptable

The opposition on renaming tables and columns made us forgot about the rest that can be done in all the code. Which looked doable. So I went asking most people that were at the discussion in the next days to reconstitute the rest of the picture besides table renaming. Plus get additional thoughts to complete the only brief moments these other subtopics were able to be discussed.

Hackfest: Provide guidance to someone trying to implement record locking to avoid conflicting edits

On community workflow and ideas for the problem in itself. Bug 23705 - Add a syspref to enable record locking, so that when one record is opened to be edited, it's read only for other users

Help giving feedback about how to handle total count in paginated patron checkout list

Oral, I think related to Bug 22613 - Add /patrons/patron_id/checkouts endpoints

Hackfest: Input on: general community process issues discussion

Time not counted as QA when having nothing to contribute to a discussion, so most Hackfest items aren't as much time as they look.

Version 2 of: Bug 36118 - ILL request log does not display patron information

Patches were redone significantly.

Note for the people funding my work and the rest of the community for transparency purposes: QA of an enhancement from PTFS Europe. For QA of enhancements I'm asked to ideally focus on one monthly.

Partial: Bug 25408 - CanBookBeReserved & CanItemBeReserved should check "opacitemholds" policy

Quite a tough one to try fully review, thanks to another partial review, we were able to pass QA.

Found issues: Bug 35659 - OAI Harvester

Note for the people funding my work and the rest of the community for transparency purposes: QA of an enhancement from BibLibre. For QA of enhancements I'm asked to ideally focus on one monthly.

Found issue: Bug 35197 - Expose additional_field definitions through REST API

Note for the people funding my work and the rest of the community for transparency purposes: QA of an enhancement from PTFS Europe. For QA of enhancements I'm asked to ideally focus on one monthly. It's the 2nd and will be kept track of to rebalance next months.

Bug 31345 - Add ability to exit process_message_queue.pl early if any plugin before_send_messages hook fails

Note for the people funding my work and the rest of the community for transparency purposes: QA of an enhancement from ByWater Solutions. For QA of enhancements I'm asked to ideally focus on one monthly.

Signoffs (testing submitted patches so they can move on to be integrated)

Note: an attempt usually results in feedback provided on the bug. To ask if I correctly followed the test plan due to results that I couldn't interpret. Or if there is indeed an issue.

Retest to confirm it's been fixed in another ticket: Bug 35591 - bulkmarcimport.pl overwrites data if controlnumber eq authid

Making the patch not needed.

Bug 36552 - Update record 'date entered on file' when duplicating a record

Minor, not counted in billable time.

New tickets opened

i.e. Found something wrong and documented how to reproduce it. Or suggest an enhancement (usually for dev tools)