flake8
passed.
JSHint
passed.
Review Request #12821 — Created Feb. 1, 2023 and submitted
We have some pretty old 3rd party code for our autocomplete UI that has an
improperly declared variable in one of itsfor
loops, which breaks our
autocomplete UI. This didn't cause any problems in previous Review Board
versions, but with our recent JavaScript upgrades we now bundle in our 3rd
party code by importing straight fromnode_modules
, rather than manually
copying it into our tree. This is most likely what exposed the bug. While we
should ultimately move away from this autocomplete code and replace it with
something more modern, for now we'll just patch it. This change fixes the
improperly declared variable.
Manually tested the autocomplete UI.
Summary | ID |
---|---|
485aa1f75c2940eedbea4bc850008f9a1a3e95c9 |