Prior to bug 30070 we showed the most recent edifact messages first: > order_by => { -desc => 'transfer_date' }, We need to restore this functionality when loading the messages via the api
Created attachment 177841 [details] [review] Bug 38957: edifactmsgs data table sorts on date This trivial patch addresses a missing order-by on the edifact messages datatable since moving to API based fetching TO TEST: a) Read in a load of EDIFact messages in k.t.d b) Notice on the EDIFact messages table the order is seemingly arbitrary c) APPLY PATCH d) Notice the sorting is now by transfer date, descending
Created attachment 177954 [details] [review] Bug 38957: edifactmsgs data table sorts on date Worked as expected This trivial patch addresses a missing order-by on the edifact messages datatable since moving to API based fetching TO TEST: a) Read in a load of EDIFact messages in k.t.d b) Notice on the EDIFact messages table the order is seemingly arbitrary c) APPLY PATCH d) Notice the sorting is now by transfer date, descending Signed-off-by: Lisette Scheer <lisette@bywatersolutions.com>
Created attachment 178086 [details] [review] Bug 38957: edifactmsgs data table sorts on date This trivial patch addresses a missing order-by on the edifact messages datatable since moving to API based fetching TO TEST: a) Read in a load of EDIFact messages in k.t.d b) Notice on the EDIFact messages table the order is seemingly arbitrary c) APPLY PATCH d) Notice the sorting is now by transfer date, descending Signed-off-by: Lisette Scheer <lisette@bywatersolutions.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Pushed for 25.05! Well done everyone, thank you!
Nice work everyone! Pushed to 24.11.x for 24.11.03