Several enhancements in the queue related to the patron display. We should first have Cypress test for main before starting adding more changes.
Created attachment 187837 [details] [review] Bug 41003: Remove extra spaces
Created attachment 187838 [details] [review] Bug 41003: Add Cypress tests to cover patron formatting
Created attachment 187840 [details] [review] Bug 41003: Add Cypress tests to cover patron formatting
No one tested this?
(In reply to Lucas Gass (lukeg) from comment #4) > No one tested this? I didn't think it was needed for several reasons: * This patch adds tests for "main", nothing else (the "Remove extra spaces" change is not something visible on the interface, only to make the tests pass) * It has been added as blocker of other bugs and I didn't want to block them because of tests (especially bug 40245 that is not from me) * Follow-up bugs will extend those tests, and so testers and QAers will confirm they work anyway But feel free to ask for a QA stamp if you are not comfortable with the patches!
Created attachment 188802 [details] [review] Bug 41003: Add Cypress tests to cover patron formatting Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Conflicts from bug 30568 - I don't believe first patch was needed any longer - dropped
Nice work everyone! Pushed to main for 25.11