[15:23:44] (CR) Yaron Koren: [C: +2] Used wgExtensionAssetsPath and removed variable [extensions/PageForms] - https://gerrit.wikimedia.org/r/676575 (https://phabricator.wikimedia.org/T275404) (owner: Anjali041) [15:23:55] (CR) Yaron Koren: [C: +2] "This looks good - thank you!" [extensions/PageForms] - https://gerrit.wikimedia.org/r/676575 (https://phabricator.wikimedia.org/T275404) (owner: Anjali041) [15:28:31] (CR) jerkins-bot: [V: -1] Used wgExtensionAssetsPath and removed variable [extensions/PageForms] - https://gerrit.wikimedia.org/r/676575 (https://phabricator.wikimedia.org/T275404) (owner: Anjali041) [17:21:39] (CR) Yaron Koren: "This is good, but on second thought, I think it would actually be better to have PFTemplateInForm::$mFields always be an array - and just " [extensions/PageForms] - https://gerrit.wikimedia.org/r/676573 (owner: Yashvarshney02)