Bug 41700 - Checkouts note_date has incorrect format in swagger definitions
Summary: Checkouts note_date has incorrect format in swagger definitions
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: REST API (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2026-01-23 19:44 UTC by Nick Clemens (kidclamp)
Modified: 2026-01-23 19:56 UTC (History)
1 user (show)

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Comma delimited list of Sponsors:
Crowdfunding goal: 0
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 41700: Update note_date to datetime (1.17 KB, patch)
2026-01-23 19:55 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Bug 41700: Unit test (1.03 KB, patch)
2026-01-23 19:55 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Bug 41700: Update note_date to datetime (1.22 KB, patch)
2026-01-23 19:55 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Bug 41700: Unit test (1.08 KB, patch)
2026-01-23 19:56 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2026-01-23 19:44:39 UTC
note_date is defined as 'date' in checkout.yaml
in the issues table it is a datetime. This is causing errors

To recreate:
1 - Enable issue notes:  AllowCheckoutNotes preference
2 - Issue an item to a patron
3 - Sign in to opac as that patron, add a note
4 - Have staff client open and a second tab try:
    http://localhost:8081/api/v1/checkouts/
5 - Error "message":"Does not match date format.","path":"\/body\/0\/note_date"
Comment 1 Nick Clemens (kidclamp) 2026-01-23 19:55:35 UTC
Created attachment 191985 [details] [review]
Bug 41700: Update note_date to datetime

This patch updates the swaager definition to have the correct format

To test:
1 - Enable issue notes:  AllowCheckoutNotes preference
2 - Issue an item to a patron
3 - Sign in to opac as that patron, add a note
4 - Have staff client open and a second tab try:
    http://localhost:8081/api/v1/checkouts/
5 - Error "message":"Does not match date format.","path":"\/body\/0\/note_date"
6 - Apply patch, yarn build, restart all
7 - Refresh the page
8 - Success!
Comment 2 Nick Clemens (kidclamp) 2026-01-23 19:55:38 UTC
Created attachment 191986 [details] [review]
Bug 41700: Unit test
Comment 3 Nick Clemens (kidclamp) 2026-01-23 19:55:59 UTC
Created attachment 191987 [details] [review]
Bug 41700: Update note_date to datetime

This patch updates the swaager definition to have the correct format

To test:
1 - Enable issue notes:  AllowCheckoutNotes preference
2 - Issue an item to a patron
3 - Sign in to opac as that patron, add a note
4 - Have staff client open and a second tab try:
    http://localhost:8081/api/v1/checkouts/
5 - Error "message":"Does not match date format.","path":"\/body\/0\/note_date"
6 - Apply patch, yarn build, restart all
7 - Refresh the page
8 - Success!

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 4 Nick Clemens (kidclamp) 2026-01-23 19:56:01 UTC
Created attachment 191988 [details] [review]
Bug 41700: Unit test

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 5 Nick Clemens (kidclamp) 2026-01-23 19:56:22 UTC
Trivial, but far reaching self SO