**Code Review**
====== Meetings ======

Collaborative Code Review meets Mondays at 2pm Eastern.

  * **November 24, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/2078503|#2078503]] acq: MARC federated search cannot use currency symbol in 020$c
    * [[https://bugs.launchpad.net/evergreen/+bug/1839883|#1839883]] change "request item" to "place hold" in Search for Copies by Barcode - //pullrequest//
    * [[https://bugs.launchpad.net/evergreen/+bug/2115659|#2115659]] OPAC place hold form: validation styles, ARIA for org selector - //signedoff//
    * [[https://bugs.launchpad.net/evergreen/+bug/2115525|#2115525]] account preferences: edit forms need label, ARIA corrections - //signedoff//
    * [[https://bugs.launchpad.net/evergreen/+bug/2115655|#2115655]] hold forms: ARIA for missing email warning - //signedoff//
  * **November 17, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/2110907|#2110907]] Staff Portal Page - Everything Opens in a New Tab - //Fix Committed//
    * [[https://bugs.launchpad.net/evergreen/+bug/1839883|#1839883]] change "request item" to "place hold" in Search for Copies by Barcode - //Followup patch pushed - pullrequest//
    * [[https://bugs.launchpad.net/evergreen/+bug/2131031|#2131031]] Holdings Template Editor: Call Number Fields not available in Grid - //Fix Committed//
    * [[https://bugs.launchpad.net/evergreen/+bug/2107404|#2107404]] updating pickup library from "view holds" doesn't retarget hold - //Fix Committed//
    * [[https://bugs.launchpad.net/evergreen/+bug/2131151|#2131151]] Angular modal dialogs render with unnecessary empty space beneath the last control when content is short - //Fix Committed//
  * **November 10, 2025** - Hack-A-Way
  * **October 27, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/1735857|#1735857]] Consider embedding Accessibility Metadata Project terms in public catalog - //Fix Committed//
    * [[https://bugs.launchpad.net/evergreen/+bug/2115443|#2115443]] OPAC basic search keyword input should not have autofocus - //Feedback Added//
    * [[https://bugs.launchpad.net/evergreen/+bug/2069451|#2069451]] Basic search results too verbose (OPAC) - //Fix Committed//
    * [[https://bugs.launchpad.net/evergreen/+bug/2129807|#2129807]] Github actions builds are failing, they need apt-get update - //Fix Committed//
  * **October 20, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/1949222|#1949222]] Add To Basket not an option in Basket Actions from inside the bib record - //Fix Committed//
    * Getting the nightwatch e2e tests working again!
  * **October 13, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/2119242|#2119242]] Batch user notes
  * **October 6, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/2033101|#2033101]] Port View Hold Shelf / Clear Hold Shelf to Angular 
  * **September 29, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/1175400|#1175400]] ACQ: Possible to Over-allocate from a Funding Source
  * **September 22, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/2121886|#2121886]] SIP2 hold cancellations fail to return a success message when the cancellation is successful - //Fix Committed//
    * [[https://bugs.launchpad.net/evergreen/+bug/787162|#787162]] Admin interface for call number classes - //Feedback provided//
    * [[https://bugs.launchpad.net/evergreen/+bug/2002166|#2002166]] Void lost item billing refunds non-monetary options - //Tested, needs more testing// (there are a lot of billing scenarios to try)
  * **September 15, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/1945366|#1945366]] Preferred name should be used as display name in browser tab - //Fix Committed//
    * [[https://bugs.launchpad.net/evergreen/+bug/2002166|#2002166]] Void lost item billing refunds non-monetary options
    * [[https://bugs.launchpad.net/evergreen/+bug/2122448|#2122448]] Clearing the shelving location in the holdings editor crashes the browser - //Feedback added//
    * [[https://bugs.launchpad.net/evergreen/+bug/2121886|#2121886]] SIP2 hold cancellations fail to return a success message when the cancellation is successful
  * **September 8, 2025**
    * [[https://bugs.launchpad.net/evergreen/+bug/1821093|#1821093]] Patron Self Registration form needs captcha
    * [[https://bugs.launchpad.net/evergreen/+bug/1770289|#1770289]] Wishlist: an acquisitions data dashboard
  * **September 1, 2025** - US Holiday - No Meeting  
  * **August 25, 2025**
    * i18n:
      * [[https://bugs.launchpad.net/evergreen/+bug/2042915|#2042915]] Column headers show as untranslated in Czech staff client [Basic admin component doesn't translate grid column headers] -- //**needs work**//
      * [[https://bugs.launchpad.net/evergreen/+bug/1884261|#1884261]] fm_IDL.xml needs cleanup [adding labels + oils_persist:i18n to text fields]
    * Dark Mode:
      * [[https://bugs.launchpad.net/evergreen/+bug/2097385|#2097385]] Dark Mode z39.50 'View MARC' modal contrast issues - //Fix Committed//
      * [[https://bugs.launchpad.net/evergreen/+bug/2107141|#2107141]] Dark mode with enhanced marc editor looks bad - //Fix Committed//
      * [[https://bugs.launchpad.net/evergreen/+bug/2093225|#2093225]] Dark Mode: Acq Worksheet Column Headers are Light Grey text on Light Grey Background - //Fix Committed//