We are seeing older requests that have no date placed break the display of the table
Created attachment 152737 [details] [review] Bug 34130: Make requested_date nullable in api definition
Created attachment 152741 [details] [review] Bug 34130: Make requested_date nullable in api definition This patch allows the requested_date for an ILL request to be NULL to accomodate older data To test: 1 - Install the Koha 2 Koha ILL plugin: https://gitlab.com/koha-community/plugins/koha-plugin-ill-koha 2 - Enable the ILL system preference 3 - Force an ILL request with minimal data from backend: INSERT INTO illrequests (borrowernumber,biblio_id,branchcode,backend,status) VALUES (5,3,'CPL','Koha','placed'); 4 - View the ILL table 5 - Error: {"message":"Expected string - got null.","path":"\/body\/0\/requested_date"} 6 - Apply patch 7 - Table loads successfully Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Trivial change, passing QA to allow RM to agree (or not)
Pushed to master for 23.11. Nice work everyone, thanks!
Thanks for all the hard work! Pushed to 23.05.x for the next release
Nice work everyone! Pushed to 22.11.x for next release