[04:41:50] 10Multimedia, 10MediaWiki-Authentication-and-authorization, 10UploadWizard: User get logged out while uploading files with UploadWizard - https://phabricator.wikimedia.org/T216777 (10Extarys) Firefox 64 on Linux. I couldn't upload the rest of the files requested here though. (Just confirming) [10:17:28] 10Multimedia, 10Wikidata, 10Wikidata-Campsite, 10SDC Engineering (Depicts-Crawling): Make sure federation works on beta - https://phabricator.wikimedia.org/T217157 (10WMDE-leszek) I _guess_ this might be a different manifestation of T214544 ? [11:18:44] (03PS1) 10Pablo Grass (WMDE): MediaInfoView: revision support for getContent() [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494695 (https://phabricator.wikimedia.org/T210615) [11:45:13] (03CR) 10Esanders: "recheck" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/494043 (owner: 10Esanders) [11:45:30] (03CR) 10Zfilipin: "> Patch Set 1:" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/492710 (https://phabricator.wikimedia.org/T214480) (owner: 10Zfilipin) [11:52:49] (03CR) 10jerkins-bot: [V: 04-1] eslint: Enable reportUnusedDisableDirectives & cache [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/494043 (owner: 10Esanders) [11:59:02] (03CR) 10jerkins-bot: [V: 04-1] MediaInfoView: revision support for getContent() [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494695 (https://phabricator.wikimedia.org/T210615) (owner: 10Pablo Grass (WMDE)) [12:08:44] (03PS2) 10Pablo Grass (WMDE): MediaInfoView: revision support for getContent() [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494695 (https://phabricator.wikimedia.org/T210615) [12:17:13] (03CR) 10jerkins-bot: [V: 04-1] MediaInfoView: revision support for getContent() [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494695 (https://phabricator.wikimedia.org/T210615) (owner: 10Pablo Grass (WMDE)) [13:12:05] 10Multimedia, 10Wikidata, 10Wikidata-Campsite, 10SDC Engineering (Depicts-Crawling): Make sure federation works on beta - https://phabricator.wikimedia.org/T217157 (10Addshore) After talking with cormac on IRC, as far as I can tell federation is working just fine. However search (backed by cirrus) does not... [14:38:08] (03CR) 10Jforrester: [C: 03+1] "Leaving to Hana's judgement as to whether to merge." (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/492976 (https://phabricator.wikimedia.org/T216454) (owner: 10Matthias Mullie) [14:39:15] (03CR) 10Ha78na: [C: 03+2] Only enable submit button when there have been changes [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/492976 (https://phabricator.wikimedia.org/T216454) (owner: 10Matthias Mullie) [14:41:36] (03PS6) 10Jforrester: Drop old wikibase.mediainfo.filepagestyles module [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/491472 [14:42:34] (03CR) 10Jforrester: [C: 04-2] "Target: wmf.22." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/491472 (owner: 10Jforrester) [14:50:14] (03CR) 10Jforrester: [C: 03+2] "recheck" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/494043 (owner: 10Esanders) [14:50:25] 10Multimedia, 10Wikidata, 10Wikidata-Campsite, 10SDC Engineering (Depicts-Crawling): Make sure federation works on beta - https://phabricator.wikimedia.org/T217157 (10Cparle) Erm ... 'working fine' might be over-egging it a bit. We haven't definitively shown it's *not* working, because some of the searchin... [14:56:34] (03Merged) 10jenkins-bot: Only enable submit button when there have been changes [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/492976 (https://phabricator.wikimedia.org/T216454) (owner: 10Matthias Mullie) [14:56:49] (03CR) 10jerkins-bot: [V: 04-1] Drop old wikibase.mediainfo.filepagestyles module [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/491472 (owner: 10Jforrester) [14:59:20] (03CR) 10Pablo Grass (WMDE): "@leszek This is now failing for the reasons discussed (compatibility is with I5fe8ede0b9d69de6b988f8846baa2ccc79d7453b)" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494695 (https://phabricator.wikimedia.org/T210615) (owner: 10Pablo Grass (WMDE)) [15:04:10] (03CR) 10jerkins-bot: [V: 04-1] eslint: Enable reportUnusedDisableDirectives & cache [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/494043 (owner: 10Esanders) [15:11:13] (03CR) 10jerkins-bot: [V: 04-1] eslint: Enable reportUnusedDisableDirectives & cache [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/494043 (owner: 10Esanders) [15:19:22] (03PS2) 10Ha78na: Add install/setup instructions to mw.org Extension:WikibaseMediaInfo [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494283 [15:22:13] (03CR) 10Matthias Mullie: Only enable submit button when there have been changes (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/492976 (https://phabricator.wikimedia.org/T216454) (owner: 10Matthias Mullie) [15:27:37] (03CR) 10jerkins-bot: [V: 04-1] Add install/setup instructions to mw.org Extension:WikibaseMediaInfo [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494283 (owner: 10Ha78na) [15:35:43] (03PS11) 10Matthias Mullie: Add descriptive text for depicts [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/492661 (https://phabricator.wikimedia.org/T216757) [15:35:50] (03CR) 10Matthias Mullie: Add descriptive text for depicts (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/492661 (https://phabricator.wikimedia.org/T216757) (owner: 10Matthias Mullie) [15:36:34] (03CR) 10Matthias Mullie: "recheck" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/492279 (https://phabricator.wikimedia.org/T216763) (owner: 10Matthias Mullie) [16:10:27] (03PS1) 10Eric Gardner: Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) [16:10:33] (03CR) 10jerkins-bot: [V: 04-1] Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [16:15:41] (03PS2) 10Eric Gardner: Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) [16:15:50] (03CR) 10jerkins-bot: [V: 04-1] Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [16:25:27] (03CR) 10Matthias Mullie: "Some of the feedback might be too technical to include in the README, but I just wanted to (try to) give accurate information :p" (0312 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494283 (owner: 10Ha78na) [16:31:07] (03CR) 10Pablo Grass (WMDE): [V: 04-1] "For I5fe8ede0b9d69de6b988f8846baa2ccc79d7453b" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494695 (https://phabricator.wikimedia.org/T210615) (owner: 10Pablo Grass (WMDE)) [16:31:33] (03CR) 10Pablo Grass (WMDE): [C: 04-1] MediaInfoView: revision support for getContent() [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494695 (https://phabricator.wikimedia.org/T210615) (owner: 10Pablo Grass (WMDE)) [16:32:38] (03PS3) 10Eric Gardner: Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) [16:34:36] 10Multimedia, 10Wikidata, 10Wikidata-Campsite, 10SDC Engineering (Depicts-Crawling): Make sure federation works on beta - https://phabricator.wikimedia.org/T217157 (10Cparle) Ok for now what I'm gonna try is have the File page call wbsearchentities on wikidata instead of on commons. We shall see ... [16:37:46] (03CR) 10jerkins-bot: [V: 04-1] Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [16:39:00] (03PS4) 10Eric Gardner: Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) [16:43:29] (03CR) 10Matthias Mullie: [C: 04-1] "This change is ready for review." (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [16:47:17] (03CR) 10jerkins-bot: [V: 04-1] Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [16:48:03] (03CR) 10Matthias Mullie: "recheck" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/491472 (owner: 10Jforrester) [16:49:26] (03CR) 10Matthias Mullie: [C: 03+1] "LGTM, merge at will when time is right." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/491472 (owner: 10Jforrester) [17:29:19] (03PS5) 10Eric Gardner: Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) [17:33:54] (03CR) 10jerkins-bot: [V: 04-1] Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [17:35:57] (03PS6) 10Eric Gardner: Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) [18:09:03] (03CR) 10Eric Gardner: Add install/setup instructions to mw.org Extension:WikibaseMediaInfo (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494283 (owner: 10Ha78na) [19:04:15] (03CR) 10Jforrester: "Looks pretty good. Code suggestion inline." (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [20:32:13] (03PS7) 10Eric Gardner: Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) [20:37:45] (03CR) 10Eric Gardner: "James, I think I was able to address all your suggestions. This is ready for final review." (032 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [20:48:46] (03CR) 10jerkins-bot: [V: 04-1] Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [20:52:02] (03CR) 10Eric Gardner: "recheck" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner) [21:11:35] (03PS3) 10Ha78na: Update install/setup instructions in README [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494283 [21:24:38] (03CR) 10Ha78na: Update install/setup instructions in README (033 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494283 (owner: 10Ha78na) [21:46:23] (03CR) 10Ha78na: [C: 03+1] Store user CC0 license agreement in localstorage or userprefs [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/494766 (https://phabricator.wikimedia.org/T215959) (owner: 10Eric Gardner)