[08:05:03] (03CR) 10Itamar Givon: [C: 03+1] Update WikibaseClient::getSettings() calls [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666188 (https://phabricator.wikimedia.org/T275412) (owner: 10Lucas Werkmeister (WMDE)) [08:47:09] (03CR) 10WMDE-leszek: "Only after submitting I've realized there is already patch submitted, addressing the similar issue but a bit differently: I18cf9a2a2467b05" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666530 (https://phabricator.wikimedia.org/T267363) (owner: 10WMDE-leszek) [08:58:05] (03CR) 10WMDE-leszek: "This change is ready for review." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666531 (https://phabricator.wikimedia.org/T267363) (owner: 10WMDE-leszek) [11:04:57] (03CR) 10Itamar Givon: [C: 03+1] Update getStatementGuidParser() call [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/656453 (https://phabricator.wikimedia.org/T272166) (owner: 10Lucas Werkmeister (WMDE)) [11:33:59] (03CR) 10Lucas Werkmeister (WMDE): [C: 03+2] "merging, since it already has +1s from both WMF and WMDE people" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/656453 (https://phabricator.wikimedia.org/T272166) (owner: 10Lucas Werkmeister (WMDE)) [12:05:17] (03CR) 10Lucas Werkmeister (WMDE): "I’m not convinced this code belongs in WikibaseMediaInfo. The whole slot / MCR handling code is in Wikibase itself, as far as I’m aware – " [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666531 (https://phabricator.wikimedia.org/T267363) (owner: 10WMDE-leszek) [12:09:30] (03CR) 10Matthias Mullie: [C: 04-1] "AIUI, requirements have changed a bit yesterday and there now still needs to be a padding-top (a smaller one)" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666385 (https://phabricator.wikimedia.org/T271440) (owner: 10Seddon) [12:11:16] (03CR) 10Lucas Werkmeister (WMDE): Added NoMediainfoSlotHandlingRevisionLookup (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666531 (https://phabricator.wikimedia.org/T267363) (owner: 10WMDE-leszek) [12:21:03] (03Merged) 10jenkins-bot: Update getStatementGuidParser() call [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/656453 (https://phabricator.wikimedia.org/T272166) (owner: 10Lucas Werkmeister (WMDE)) [14:20:45] (03CR) 10Anne Tomasevich: "I tested the updated version locally - there's a noticeable improvement for search terms that specifically benefit from the addition of al" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/643388 (https://phabricator.wikimedia.org/T258053) (owner: 10Anne Tomasevich) [14:25:56] (03PS1) 10Lucas Werkmeister (WMDE): Update WikibaseRepo::getLogger() call [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666641 (https://phabricator.wikimedia.org/T275623) [16:52:43] (03PS2) 10Seddon: Media Search: Decrease padding below filters [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666385 (https://phabricator.wikimedia.org/T271440) [22:24:38] (03PS1) 10Anne Tomasevich: [WIP, need to implement in PHP] [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666745 (https://phabricator.wikimedia.org/T271174) [22:36:42] (03CR) 10Anne Tomasevich: "I think we still want the 16px margin-bottom - could you add that back in and just set the margin-top to 6px?" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/666385 (https://phabricator.wikimedia.org/T271440) (owner: 10Seddon) [22:44:31] (03PS6) 10Eric Gardner: Get license groups from community-defined wikipage [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/665458 (https://phabricator.wikimedia.org/T273166) [23:38:32] 10Multimedia, 10Commons, 10MediaWiki-File-management, 10Platform Engineering Roadmap Decision Making, and 2 others: Migrate file tables to a modern layout (image/oldimage; file/file_revision; add primary keys) - https://phabricator.wikimedia.org/T28741 (10CCicalese_WMF)