[04:51:04] (03CR) 10Eric Gardner: "Looks good. A few questions in the comments below (I want to make sure I actually understand everything that's going on here)." (034 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/554846 (owner: 10Matthias Mullie) [04:52:12] (03PS9) 10Eric Gardner: Consolidate ItemInputWidget and EntityInputWidget [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/554846 (owner: 10Matthias Mullie) [04:54:12] (03CR) 10Eric Gardner: [C: 03+1] "Looks good – made some formatting changes in the latest patchset." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/554846 (owner: 10Matthias Mullie) [04:58:42] (03CR) 10SonarQube Bot: "❌ Quality gate failed" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/554846 (owner: 10Matthias Mullie) [09:14:21] (03CR) 10Matthias Mullie: Consolidate ItemInputWidget and EntityInputWidget (034 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/554846 (owner: 10Matthias Mullie) [09:14:32] (03PS10) 10Matthias Mullie: Consolidate ItemInputWidget and EntityInputWidget [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/554846 [09:16:28] (03CR) 10Matthias Mullie: [C: 03+1] "LGTM btw, but will need Eric to verify the map sizing now works fine on his end too (and review the other changes)" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/556305 (https://phabricator.wikimedia.org/T227116) (owner: 10Eric Gardner) [09:21:20] (03CR) 10SonarQube Bot: "❌ Quality gate failed" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/554846 (owner: 10Matthias Mullie) [10:36:03] 10Multimedia, 10Core Platform Team Legacy (Watching / External), 10Services (watching): Explore moving the Panoviewer gadget/Tool Labs tool into production - https://phabricator.wikimedia.org/T138933 (10TheDJ) I've now got a PanoViewer extension with Pannellum, and using the new XMP and iPhone metadata. TOD... [15:49:59] (03CR) 10Cparle: "recheck" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/558592 (https://phabricator.wikimedia.org/T240966) (owner: 10Cparle) [16:01:18] 10Multimedia, 10Core Platform Team Legacy (Watching / External), 10Services (watching): Explore moving the Panoviewer gadget/Tool Labs tool into production - https://phabricator.wikimedia.org/T138933 (10dschwen) Just as a quick comment, I think multires tiles are absolutely necessary for the stitched panos t... [18:49:40] (03CR) 10Eric Gardner: [C: 03+2] "This all looks good to me, thanks for answering those questions. I'll look out for a follow-up patch for some of the minor changes discuss" (033 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/554846 (owner: 10Matthias Mullie) [19:12:30] (03Merged) 10jenkins-bot: Consolidate ItemInputWidget and EntityInputWidget [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/554846 (owner: 10Matthias Mullie) [19:28:28] (03PS16) 10Eric Gardner: Allow users to input coordinates using map UI [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/556305 (https://phabricator.wikimedia.org/T227116) [19:34:01] (03CR) 10Eric Gardner: [C: 03+1] "This small update fixes the bug in the maps that I've been seeing locally by re-introducing the second MutationObserver. Everything else l" (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/556305 (https://phabricator.wikimedia.org/T227116) (owner: 10Eric Gardner) [19:35:07] (03CR) 10SonarQube Bot: "❌ Quality gate failed" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/556305 (https://phabricator.wikimedia.org/T227116) (owner: 10Eric Gardner) [20:12:52] (03PS4) 10Anne Tomasevich: Show wbparsevalue errors in the WBMI UI [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/558218 (https://phabricator.wikimedia.org/T239857) [20:15:02] (03PS3) 10Anne Tomasevich: Update error UX on submit [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/559211 (https://phabricator.wikimedia.org/T233051) [20:32:10] (03CR) 10jerkins-bot: [V: 04-1] Show wbparsevalue errors in the WBMI UI [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/558218 (https://phabricator.wikimedia.org/T239857) (owner: 10Anne Tomasevich) [20:36:49] (03CR) 10jerkins-bot: [V: 04-1] Update error UX on submit [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/559211 (https://phabricator.wikimedia.org/T233051) (owner: 10Anne Tomasevich) [20:40:42] (03PS5) 10Anne Tomasevich: Show wbparsevalue errors in the WBMI UI [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/558218 (https://phabricator.wikimedia.org/T239857) [20:49:05] (03CR) 10SonarQube Bot: "❌ Quality gate failed" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/558218 (https://phabricator.wikimedia.org/T239857) (owner: 10Anne Tomasevich) [21:07:54] (03PS4) 10Anne Tomasevich: Update error UX on submit [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/559211 (https://phabricator.wikimedia.org/T233051) [21:14:31] (03CR) 10SonarQube Bot: "❌ Quality gate failed" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/559211 (https://phabricator.wikimedia.org/T233051) (owner: 10Anne Tomasevich) [21:17:39] (03CR) 10Anne Tomasevich: "This change is ready for review." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/558218 (https://phabricator.wikimedia.org/T239857) (owner: 10Anne Tomasevich) [21:18:55] (03CR) 10Anne Tomasevich: "This change is ready for review." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/559211 (https://phabricator.wikimedia.org/T233051) (owner: 10Anne Tomasevich)