[08:57:59] 6Multimedia, 10MediaWiki-Uploading: Thumbnail request for a log image name throws: Error generating thumbnail - https://phabricator.wikimedia.org/T113458#1665547 (10santhosh) 3NEW [09:00:46] 6Multimedia, 6Commons, 6Editing-Department, 10Possible-Tech-Projects, and 2 others: Allow uploading of 3D files to Wikimedia Commons - https://phabricator.wikimedia.org/T3790#1665557 (10Qgil) @marktraceur @jeanfred this project is still featured at #Possible-Tech-Projects. Do you want to propose it for #Ou... [09:04:24] 6Multimedia, 6Commons, 6Editing-Department, 10GlobalUsage, and 3 others: Show edits made on WikimediaCommons on the Watchlist - https://phabricator.wikimedia.org/T91192#1665560 (10Qgil) This task is listed under "Missing mentors" at #Possible-Tech-Projects. Are there mentors willing to propose it for #Out... [09:13:47] 6Multimedia, 10MediaWiki-File-management, 10Possible-Tech-Projects, 7Testme: Embed image author, description, and copyright data in file metadata fields - https://phabricator.wikimedia.org/T5361#1665641 (10Qgil) This is a message posted to all tasks under "Backlog" at #Possible-Tech-Projects. #Outreachy-Ro... [09:13:49] 6Multimedia, 6Commons, 6Community-Tech, 10MediaWiki-Uploading, 10Possible-Tech-Projects: Add support for KML/KMZ filetype - https://phabricator.wikimedia.org/T28059#1665640 (10Qgil) This is a message posted to all tasks under "Backlog" at #Possible-Tech-Projects. #Outreachy-Round-11 is around the corner.... [10:43:39] 6Multimedia, 6Commons, 10Traffic, 6operations: Commons API fails (413 error) to upload file within 100MB threshold - https://phabricator.wikimedia.org/T86436#1665790 (10Raymond) Same problem while overwriting an [[ https://commons.wikimedia.org/wiki/File:Portr%C3%A4t_von_Diederick_Hoeufft,_Kapit%C3%A4n_der... [10:45:36] 6Multimedia, 10MediaWiki-Uploading: Thumbnail request for a log image name throws: Error generating thumbnail - https://phabricator.wikimedia.org/T113458#1665793 (10Aklapper) https://commons.wikimedia.org/wiki/File:Coat_of_arms_of_the_16th_Mechanized_Infantry_Regiment_Castilla.svg is 11MB. Interesting that lar... [10:45:59] 6Multimedia, 10MediaWiki-Uploading: Specific thumbnail request for a SVG log image name throws: Error generating thumbnail - https://phabricator.wikimedia.org/T113458#1665795 (10Aklapper) [10:46:14] 6Multimedia, 10MediaWiki-Uploading: Specific thumbnail request for a SVG log image name throws: Error generating thumbnail - https://phabricator.wikimedia.org/T113458#1665547 (10Aklapper) [10:56:18] 6Multimedia: Improve mw.widgets.CategorySelector so that it can be used in something like HotCat - https://phabricator.wikimedia.org/T113462#1665823 (10Prtksxna) 3NEW a:3Prtksxna [10:56:40] 6Multimedia, 10UI-Standardization, 5MW-1.26-release, 5Patch-For-Review, 5WMF-deploy-2015-09-15_(1.26wmf23): Create mw.widgets.CategorySelector - https://phabricator.wikimedia.org/T111791#1665832 (10Prtksxna) >>! In T111791#1662968, @MarkTraceur wrote: > We changed our mind about this bug...filing a new b... [10:57:08] 6Multimedia: Improve mw.widgets.CategorySelector so that it can be used in something like HotCat - https://phabricator.wikimedia.org/T113462#1665823 (10Prtksxna) [11:54:37] edsanders: Why do we need a separate panelNameSet event when StackLayout already has the set event? [11:54:54] set emits with a panel reference [11:55:01] so you'd have to do a messy lookup [11:55:32] it might be better to use a booklet layout instead of a stack layout, that stores pages by name [11:56:59] edsanders: Ah got it. Are there any other differences between Stack and Booklet layouts, apart from the outline of course. [11:57:09] I'll look into it [11:57:12] Do we want to use that instead, or will this suffice for now? [11:58:27] I'll have a go at rewriting it with one [11:59:35] edsanders: Alright, I'll hold the FSUD changes till then (https://gerrit.wikimedia.org/r/#/c/235431/) [11:59:43] ok [12:06:02] marktraceur: Also, legal text. :-) [12:19:05] prtksxna, done [12:19:34] edsanders: Cool, I'll take a look in a bit [12:29:32] 6Multimedia, 10MediaWiki-Uploading: Specific thumbnail request for a SVG log image name throws: Error generating thumbnail - https://phabricator.wikimedia.org/T113458#1666107 (10matmarex) [12:58:28] James_F|Away: legal text is drafted, I'll tweak in CR, so will slaporte [12:59:09] hi. [12:59:10] Unless prtksxna was just making a placeholder [12:59:14] Hi MatmaRex [13:00:42] are we gerrit-cleaning-upping? [13:01:27] Ah yes. I suppose so. [13:12:41] I even hope so. In any case I'd appreciate it. :) [13:38:43] marktraceur: Hm? [13:43:48] prtksxna: Legal text for the upload dialog - was your text just a placeholder or should we tweak from it? [13:43:56] marktraceur: Placeholder, yes. [13:44:03] Ah, K. I should probably write a new one then. [13:44:07] AFTER GERRIT CLEANUP [13:44:08] Argh. [13:44:19] But andre__ there's so much to dooooooo [13:54:27] (03Abandoned) 10Reedy: Updating EmbedPlayer [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/144484 (owner: 10Paladox) [13:54:52] (03Abandoned) 10Reedy: Update TimedText [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/144675 (owner: 10Paladox) [13:55:20] (03CR) 10Bartosz Dziewoński: "I wonder if we could merge this." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/99178 (owner: 10MarkTraceur) [13:57:15] Oh, holy shit, that didn't get merged? [13:57:34] I wonder how many PDFs got rejected in the past two years [13:57:54] (03CR) 10MarkTraceur: "I'd say that's A Good Idea™" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/99178 (owner: 10MarkTraceur) [14:01:47] marktraceur: You will like the feeling of having a smaller backlog at the end of a day, plus having given feedback to contributors. There's always so much to do hence please break out from it once for a few hours. :) [14:01:56] Or just blame me if you miss some deadlines. :P [14:02:05] Sold [14:02:51] (03Abandoned) 10Bartosz Dziewoński: Fix a conditional that causes errors sometimes [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/99178 (owner: 10MarkTraceur) [14:03:42] one down, a few hundred more to go [14:03:50] Heh. [14:03:59] MatmaRex: Not merging? [14:04:05] Or did it get fixed elsewhere? [14:04:15] marktraceur: that fix is already in master. [14:04:24] Oh. Weird. [14:04:41] from T89011 [14:05:35] Yeah, a Gilles patch. Good work, Gilles. [14:05:52] bawolff patches for Flickr stuff... [14:06:39] (03PS2) 10MarkTraceur: Update for I51f7966add: load oojs-ui.styles.icons [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/238309 (owner: 10Catrope) [14:06:53] (03CR) 10MarkTraceur: [C: 032] Update for I51f7966add: load oojs-ui.styles.icons [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/238309 (owner: 10Catrope) [14:08:48] (03Abandoned) 10MarkTraceur: WIP Use TemplateData to build upload forms [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/76356 (owner: 10Rasel160) [14:10:04] (03CR) 10jenkins-bot: [V: 04-1] Update for I51f7966add: load oojs-ui.styles.icons [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/238309 (owner: 10Catrope) [14:11:06] tgr|away: https://www.mediawiki.org/wiki/UploadWizard_OSM_map_embedding <-- this is clearly inactive but is it held back for some reason? There are like 8 patches sitting in review for it [14:14:21] (03Abandoned) 10MarkTraceur: [Very WIP] Only use one file input, use Upload objects to represent upload process [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192809 (https://phabricator.wikimedia.org/T96916) (owner: 10MarkTraceur) [14:21:09] (03CR) 10Bartosz Dziewoński: [C: 04-1] "There were so many changes made since this was done, it now really should be redone from scratch. Unless this is suddenly high priority, I" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/187471 (https://phabricator.wikimedia.org/T87988) (owner: 10Gilles) [14:21:37] (03Abandoned) 10Bartosz Dziewoński: Add extension.json, empty PHP entry point [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/187471 (https://phabricator.wikimedia.org/T87988) (owner: 10Gilles) [14:22:11] 6Multimedia, 10UploadWizard: Convert UploadWizard to use extension registration - https://phabricator.wikimedia.org/T87988#1666392 (10matmarex) [14:26:08] I might have found an older patch than ones I have open... [14:28:34] (03Abandoned) 10Bartosz Dziewoński: Use promises for getting image info [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/146604 (https://phabricator.wikimedia.org/T51988) (owner: 10MarkTraceur) [14:28:47] 6Multimedia, 10UploadWizard, 7JavaScript: Use Deferred/Promise and generally async-friendly code paths instead of jquery.pubsub - https://phabricator.wikimedia.org/T51988#1666404 (10matmarex) a:5MarkTraceur>3None [14:32:12] 6Multimedia, 10UploadWizard, 7JavaScript: Use Deferred/Promise and generally async-friendly code paths instead of jquery.pubsub - https://phabricator.wikimedia.org/T51988#1666415 (10matmarex) The work done so far: * {3590770bf9754ba8eba021a88c33a0bc36395ebc} (which got rid of the simpler half of pubsub usag... [14:32:29] hmm, maybe we could actually kill pubsub now. [14:32:41] if anybody likes upload code in MW Core, https://gerrit.wikimedia.org/r/#/c/139696 is also an ancient patch with less than 10 lines, awaiting review [14:32:43] Maybe!?!?!? [14:32:47] * andre__ looking at some ancient patchsets [14:33:06] andre__: I'll look at it after I'm done screwing around with another ancient patch [14:33:14] 8786. [14:33:20] four digits. whou. [14:33:24] :-*! :-) [14:33:27] marktraceur: we just killed the part of it that was breaking my brain the most, together with the preview dialog [14:33:51] MatmaRex: Yay [14:34:01] I guess the previews were the worst part, yeah [14:37:50] andre__: that patch looks straightforward, i'll test it and see. [14:39:02] marktraceur: could you instead look at https://gerrit.wikimedia.org/r/#/c/224600/ and amend whatever is wrong with it? i really don't get flickr upload code, and that looks pretty close (and the submitter seemed like a smart person when we last interacted). (and it still merges, unlike most pending patches to UW :P) [14:40:03] MatmaRex: Pretty surprising that a smart person worked on UploadWizard but working on it [14:40:13] heheh [14:40:51] ehehe [14:50:45] Hm. Did the date widget patch not add a stanza for copying the date? [14:53:43] marktraceur: it did? [14:54:08] marktraceur: it's short, since it's a lot simpler with OOUI widgets. search for 'oouiCopy' helper function [14:55:29] Right. [14:55:53] (03PS2) 10MarkTraceur: Flickr: Added support for cc-zero & public domain mark [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/224600 (https://phabricator.wikimedia.org/T105629) (owner: 10Zhuyifei1999) [14:56:19] (03CR) 10MarkTraceur: Flickr: Added support for cc-zero & public domain mark (033 comments) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/224600 (https://phabricator.wikimedia.org/T105629) (owner: 10Zhuyifei1999) [14:56:55] prtksxna: Were you going to merge https://gerrit.wikimedia.org/r/#/c/239990/ ? [14:57:20] MatmaRex: Not sure if that's against the rules of Gerrit Cleanup Day, though. :P [14:57:37] (03CR) 10jenkins-bot: [V: 04-1] Flickr: Added support for cc-zero & public domain mark [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/224600 (https://phabricator.wikimedia.org/T105629) (owner: 10Zhuyifei1999) [14:57:46] FFS. [14:58:09] CI got fucked up again, it seems. [14:58:36] No, that's a real failure. [14:58:44] although that's a different failure, yes. [14:58:53] Yeah [14:58:59] but https://gerrit.wikimedia.org/r/#/c/240342/ just failed on me, for example. [14:59:50] MatmaRex: Oh, that's just the normal can't-run-headless-Chrome-tests-in-series issue. :-( [14:59:53] Not new. [15:00:22] (03PS3) 10MarkTraceur: Flickr: Added support for cc-zero & public domain mark [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/224600 (https://phabricator.wikimedia.org/T105629) (owner: 10Zhuyifei1999) [15:06:01] (03CR) 10Zhuyifei1999: "Thanks for fixing those." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/224600 (https://phabricator.wikimedia.org/T105629) (owner: 10Zhuyifei1999) [15:11:06] James_F: Yes, when it was a StackLayout, I haven't seen the Booklet layout changes yet. Will merge today (~2hrs) [15:11:21] prtksxna: Gah, slowness. ;-) [15:23:00] 6Multimedia, 10MediaWiki-Uploading, 7Easy: upload by url of something that's not a url doesn't give error - https://phabricator.wikimedia.org/T34361#1666553 (10matmarex) 5Open>3Resolved [15:23:17] marktraceur: https://gerrit.wikimedia.org/r/#/c/224600/ good to go, you think? i'll review and merge if so [15:25:39] (03PS11) 10Paladox: Move hooks from hooks file to Main php file and also some updates to sql [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/229470 [15:26:36] MatmaRex: Should be. Tested and all. Helper function is a bit of a backwards-bending-over, but it works. [15:27:08] (03PS12) 10Paladox: Move hooks from hooks file to Main php file and also some updates to sql [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/229470 [15:32:58] Are we meeting or just cleaning [15:33:29] I can monologue with the best of them but maybe I shouldn't do it in a Hangout [15:34:00] Happy to skip? [15:34:20] oh, heh, a meeting. [15:35:10] Should I join? [15:35:22] Sure [15:35:49] OK. [15:37:50] So sorry to have inconvenienced you James_F :P [15:38:08] * James_F grins. [15:38:56] * prtksxna was too late :( [15:43:02] prtksxna: The general consensus seems to be that gerrit cleanup is not worth discussing anyway [15:43:18] o_o [15:52:03] (03CR) 10Jdlrobson: [C: 04-1] "not sure if still relevant but needs a rebase if it is" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/178219 (https://phabricator.wikimedia.org/T110579) (owner: 10Gergő Tisza) [15:53:51] (03CR) 10Bartosz Dziewoński: [C: 032] Flickr: Added support for cc-zero & public domain mark [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/224600 (https://phabricator.wikimedia.org/T105629) (owner: 10Zhuyifei1999) [15:54:59] (03Merged) 10jenkins-bot: Flickr: Added support for cc-zero & public domain mark [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/224600 (https://phabricator.wikimedia.org/T105629) (owner: 10Zhuyifei1999) [15:56:43] 6Multimedia, 6Commons, 10UploadWizard, 7Easy: Use {{PD-USGov-USGS}} if from USGS Bee Inventory and Monitoring Lab on Flickr - https://phabricator.wikimedia.org/T106269#1666650 (10matmarex) [15:56:46] 6Multimedia, 6Commons, 10UploadWizard, 7Tracking: UploadWizard Flickr (tracking) - https://phabricator.wikimedia.org/T45450#1666651 (10matmarex) [15:56:49] 6Multimedia, 6Commons, 10UploadWizard, 5Patch-For-Review: Transferring PD-images from Flickr becomes 'undefined'-license - https://phabricator.wikimedia.org/T105629#1666648 (10matmarex) 5Open>3Resolved Thanks for the patch, and sorry you had to wait so long. [16:04:13] (03Abandoned) 10Bartosz Dziewoński: Add a redlink to dead duplicate file links [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/50760 (https://bugzilla.wikimedia.org/32606) (owner: 10Nischayn22) [16:05:19] (03Abandoned) 10Bartosz Dziewoński: Making all handlers return $.Promise objects [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/84344 (owner: 10Rahul21) [16:07:22] My first +2 in core, I think [16:07:24] :S [16:08:54] (03Abandoned) 10Bartosz Dziewoński: Catch error thrown by unparseable titles. [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/49280 (https://bugzilla.wikimedia.org/39642) (owner: 10Nischayn22) [16:09:43] 6Multimedia, 10UploadWizard: blank title throws error: "mw.Title: Could not parse title ".jpg"" - https://phabricator.wikimedia.org/T41642#1666683 (10matmarex) 5Open>3Resolved a:3matmarex Can not reproduce. [16:09:55] prtksxna: yay :o [16:10:47] MatmaRex: UI for what? (https://gerrit.wikimedia.org/r/#/c/237609/) [16:11:00] MatmaRex: :) [16:11:26] prtksxna: i mean, some sort of a dropdown to choose the search type dynamically [16:11:42] MatmaRex: Hmm, I don't think that should be part of the CategorySelector [16:11:53] right. i agree [16:12:27] MatmaRex: It should be easy enough to hook it up with a dropdownWidget for a HotCat-like extension [16:13:29] indeed [16:20:10] prtksxna: Did you remove your +2 on https://gerrit.wikimedia.org/r/#/c/239990/ or did gerrit break? [16:22:28] James_F: I didn't remove my +2 [16:23:56] * prtksxna hmm http://www.downforeveryoneorjustme.com/www.mediawiki.org/wiki/Reading/Strategy_Process/Q%26A [16:25:26] OK… [16:25:46] James_F: Should I +2 again? [16:25:59] (03CR) 10Hashar: "recheck" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/238309 (owner: 10Catrope) [16:26:23] prtksxna: Yeah. [16:26:56] James_F: I had +2'd PS7, sorry [16:28:21] Oh, hah. [16:28:23] (How is that possible?) [16:28:40] I suppose the new Gerrit interface lets you do it [16:29:01] Ah, mediawiki is back up :D [16:29:04] That was quick [16:31:11] (03PS35) 10Paladox: Add extensions.json, empty PHP entry point [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/210176 (https://phabricator.wikimedia.org/T87981) [16:44:37] 6Multimedia, 10VisualEditor, 10VisualEditor-MediaWiki, 10VisualEditor-MediaWiki-Media, and 2 others: Button in VisualEditor to initiate Commons file uploading, and insert image on completion - https://phabricator.wikimedia.org/T40030#1666750 (10Jdforrester-WMF) [16:45:05] (03PS11) 10Bartosz Dziewoński: Add a link to Commons:Geocoding page in location entry tooltip [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/46960 (https://phabricator.wikimedia.org/T36336) (owner: 10Rjain) [16:45:30] (03CR) 10Bartosz Dziewoński: [C: 031] "Rebased and tweaked." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/46960 (https://phabricator.wikimedia.org/T36336) (owner: 10Rjain) [16:46:49] marktraceur: bonk that? old patch that mostly needed a rebase. [16:46:51] off to dinner [16:50:34] Looking [16:58:56] (03CR) 10MarkTraceur: [C: 032] Add a link to Commons:Geocoding page in location entry tooltip [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/46960 (https://phabricator.wikimedia.org/T36336) (owner: 10Rjain) [17:01:30] (03Merged) 10jenkins-bot: Add a link to Commons:Geocoding page in location entry tooltip [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/46960 (https://phabricator.wikimedia.org/T36336) (owner: 10Rjain) [17:11:08] 6Multimedia, 10UploadWizard: location entry tooltip should link to [[commons:Commons:Geocoding]] - https://phabricator.wikimedia.org/T36336#1666856 (10matmarex) 5Open>3Resolved a:3Rjain [17:21:15] (03Abandoned) 10Bartosz Dziewoński: Will allow dynamic form generation in the details step [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/66561 (https://phabricator.wikimedia.org/T49561) (owner: 10Rasel160) [17:21:57] 6Multimedia, 6Commons, 10UploadWizard: UW support for other templates than {{Information}} (eg {{Artwork}}) - https://phabricator.wikimedia.org/T51443#1666899 (10matmarex) [17:22:15] 6Multimedia, 6Commons, 10UploadWizard, 6Wikisource: Book upload customization - https://phabricator.wikimedia.org/T49561#513009 (10matmarex) [17:27:06] 6Multimedia, 10UploadWizard: UploadWizard Flickr on Wikimedia Commons should use {{Flickr source}} and {{Flickr author}} templates - https://phabricator.wikimedia.org/T45408#1666927 (10matmarex) [17:29:33] 6Multimedia, 5Patch-For-Review: Improve mw.widgets.CategorySelector so that it can be used in something like HotCat - https://phabricator.wikimedia.org/T113462#1666933 (10Jdforrester-WMF) p:5Triage>3Normal [17:56:05] (03PS6) 10Bartosz Dziewoński: Unbreak confirmation of no categories being set [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/239418 (https://phabricator.wikimedia.org/T51710) [17:57:50] marktraceur: does https://commons.wikimedia.org/wiki/Commons:Village_pump/Proposals look like a good place to ask if they still want https://gerrit.wikimedia.org/r/239418 ? [17:59:32] MatmaRex: Yeah [17:59:50] marktraceur: …do you feel like asking about it? :P [18:01:37] 6Multimedia, 10MediaWiki-General-or-Unknown: Improve mw.widgets.CategorySelector so that it can be used in something like HotCat - https://phabricator.wikimedia.org/T113462#1667090 (10matmarex) 5Open>3Resolved [18:01:57] 6Multimedia, 10MediaWiki-General-or-Unknown, 10UI-Standardization, 5MW-1.26-release, and 2 others: Create mw.widgets.CategorySelector - https://phabricator.wikimedia.org/T111791#1667094 (10matmarex) [18:05:28] MatmaRex: Siigh sure [18:06:13] MatmaRex: It doesn't actually make them required, IIRC, just makes them "look" required? [18:06:56] marktraceur: it pops up the confirmation dialog. [18:07:01] let me screenshot it real quick. [18:07:04] Ah, right, and you can still override [18:10:58] marktraceur: hold on, i'm poking at things [18:11:16] Righto [18:11:27] marktraceur: the text in the dialogs is a bit tiny, but there's actually an option to make it bigger. i think we should use it [18:14:21] marktraceur: http://i.imgur.com/KiDVo4R.png http://i.imgur.com/mUrMXSY.png [18:15:04] i'll put them up on commons [18:17:50] Perfect. [18:18:10] MatmaRex: Looks nice. [18:24:53] marktraceur: https://commons.wikimedia.org/wiki/File:UploadWizard_category_warning.png https://commons.wikimedia.org/wiki/File:UploadWizard_category_warning_(in_context).png [18:26:08] MatmaRex: Hmm, bad i18n. [18:26:43] MatmaRex: Options should be "Let me fix" and "Skip" or something, I believe. [18:29:11] James_F: meh. [18:29:43] :_) [18:29:50] MatmaRex: Posted. [18:29:55] https://commons.wikimedia.org/wiki/Commons:Village_pump/Proposals#Categories_.22looking_more_required.22_in_UploadWizard [18:30:16] now let me submit the patch which actually makes it look like that. [18:31:47] (03PS7) 10Bartosz Dziewoński: Unbreak confirmation of no categories being set [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/239418 (https://phabricator.wikimedia.org/T51710) [18:31:49] (03PS1) 10Bartosz Dziewoński: Pass 'verbose: true' to all MessageDialogs to get bigger font size [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/240441 [18:32:10] thanks marktraceur [18:33:55] 6Multimedia, 10UploadWizard, 5Patch-For-Review, 7user-notice: Make categories more required-looking - https://phabricator.wikimedia.org/T51710#1667265 (10matmarex) https://commons.wikimedia.org/wiki/Commons:Village_pump/Proposals#Categories_.22looking_more_required.22_in_UploadWizard [18:44:16] MatmaRex: Bah, details. [18:54:19] (03PS1) 10Brion VIBBER: Re-enable 2-pass Theora encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/240446 (https://phabricator.wikimedia.org/T109658) [18:56:28] (03CR) 10Paladox: [C: 031] Re-enable 2-pass Theora encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/240446 (https://phabricator.wikimedia.org/T109658) (owner: 10Brion VIBBER) [19:55:18] (03PS6) 10Bartosz Dziewoński: Allow displaying more than 50 Flickr images at once [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/45951 (https://phabricator.wikimedia.org/T44979) (owner: 10Nischayn22) [19:55:28] i spent way too much time restoring that. [20:03:46] (03CR) 10jenkins-bot: [V: 04-1] Allow displaying more than 50 Flickr images at once [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/45951 (https://phabricator.wikimedia.org/T44979) (owner: 10Nischayn22) [20:29:16] MatmaRex: Is that the pagination patch? [20:29:56] marktraceur: kind of? it does lazy-loading rather than paginate [20:35:17] Hrm. [20:35:28] I think the TOS says something like "only ever show fifty images at a time" [20:35:41] Or at least it did when we tried that patch before [20:41:11] marktraceur: yeah, that's discussed on the bug [20:41:24] technically, your screen is probably not big enough to display 50 of them at once… [20:41:59] anyway. i wasn't planning to work on that today. :P [20:43:19] AOK. [20:43:35] MatmaRex: I think a technicality is hardly a good way to comply with the TOS [20:43:48] But I also feel like we talked to Flickr and they were like "yeah that's probably fine" [20:44:18] that bug says that [20:44:23] but it was also like two years ago [20:44:40] so [20:44:41] i dunno [21:08:04] (03PS1) 10Bartosz Dziewoński: [WIP] CategorySelector [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/240586 (https://phabricator.wikimedia.org/T112764) [21:08:08] (03CR) 10jenkins-bot: [V: 04-1] [WIP] CategorySelector [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/240586 (https://phabricator.wikimedia.org/T112764) (owner: 10Bartosz Dziewoński) [21:08:25] marktraceur: in other news, CategorySelector mostly works. https://gerrit.wikimedia.org/r/240586 [21:08:51] needs some cleanup work, but i think it's fully functional. [21:10:09] marktraceur: re UW-OSM, there are some blockers to enabling it as a beta feature, mainly UX and legal [21:10:12] 6Multimedia, 10UploadWizard, 7Epic: Migrate UploadWizard UI elements to OOjs UI - https://phabricator.wikimedia.org/T96914#1667974 (10matmarex) [21:10:14] 6Multimedia, 10UploadWizard: Move UI creation into JavaScript - https://phabricator.wikimedia.org/T96918#1667973 (10matmarex) [21:10:20] 6Multimedia, 10UploadWizard, 5Patch-For-Review: Replace CoolCats with mw.widgets.CategorySelector - https://phabricator.wikimedia.org/T112764#1667975 (10matmarex) [21:10:21] 6Multimedia, 10UploadWizard, 7Epic: Migrate UploadWizard UI elements to OOjs UI - https://phabricator.wikimedia.org/T96914#1229248 (10matmarex) [21:10:45] 6Multimedia, 10UploadWizard, 5Patch-For-Review: Replace CoolCats with mw.widgets.CategorySelector - https://phabricator.wikimedia.org/T112764#1645254 (10matmarex) [21:10:47] 6Multimedia, 10UploadWizard, 7Epic: Migrate UploadWizard UI elements to OOjs UI - https://phabricator.wikimedia.org/T96914#1229248 (10matmarex) [21:11:04] specifically, it would be troublesome with an external tileserver, and now that we are about to get our own, it's probably better to wait for that [21:11:10] tgr: marktraceur: hold on, don't we have out own map thingy? [21:11:13] our* [21:11:34] not yet on Commons [21:12:02] it's only enabled for Wikivoyage, I suppose they are scaling it up [21:13:31] I would like to merge the branch once that happens, although I'm not sure how much of the code is still relevant [21:13:40] tgr: i find it hard to imagine that the traffic to all of Wikivoyage and commons.wm.o/wiki/Special:UploadWizard is even on the same order of magnitude. that shouldn't be a technical blocker. [21:15:24] that's true, but in any case non-Wikivoyage traffic is filtered out for now [21:15:48] right [21:16:55] that reminds me, is the multimedia team interested about https://phabricator.wikimedia.org/T91649 ? [21:17:32] after being blocked on code review for a quarter or two, Sentry has finally been merged [21:17:46] personally i don't even know where, what, and why we log in UW :/ [21:18:35] this probably wouldn't involve any code changes in UW [21:19:33] but there is no point to doing it unless you intend to use the error logs [21:21:12] as for the old logging, https://www.mediawiki.org/wiki/UploadWizard/Funnel_analysis is an only mildly outdate summary [21:21:26] but I think most of that doesn't work anymore [21:23:34] Krinkle filed a task the other day that most of the events we log are being dropped, i don't even know where to start debugging that [21:25:44] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Transcode isn't happening - https://phabricator.wikimedia.org/T113532#1668073 (10Krenair) [21:28:04] (03CR) 10Gergő Tisza: "Still relevant but broken. See comments above." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/178219 (https://phabricator.wikimedia.org/T110579) (owner: 10Gergő Tisza) [21:31:17] MatmaRex: EL drops anything above 1000 characters, so just make sure your events are not longer than that (json-serialized and urlencoded) [21:33:21] it's probably the UploadWizardErrorFlowEvent schema as that contains error messages which can be long [21:33:44] that schema was just a weird temporary placeholder for Sentry anyway [21:57:03] (03CR) 10Gergő Tisza: [C: 04-1] "Needs rebase/rewrite, code has heavily changed when the MultiLightbox class was killed." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/107023 (https://bugzilla.wikimedia.org/58848) (owner: 10Apsdehal) [22:12:21] (03CR) 10Jdlrobson: [C: 04-1] "needs a rebase!" [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/173627 (https://bugzilla.wikimedia.org/56666) (owner: 10Gergő Tisza) [23:14:46] (03PS36) 10Kaldari: Add extensions.json, empty PHP entry point [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/210176 (https://phabricator.wikimedia.org/T87981) (owner: 10Paladox) [23:15:28] (03CR) 10jenkins-bot: [V: 04-1] Add extensions.json, empty PHP entry point [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/210176 (https://phabricator.wikimedia.org/T87981) (owner: 10Paladox) [23:25:47] (03CR) 10Kaldari: "Jenkins is showing lots of Notices like..." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/210176 (https://phabricator.wikimedia.org/T87981) (owner: 10Paladox) [23:37:52] (03CR) 10Bartosz Dziewoński: [C: 032] Add scrollbar to list of available transcodes [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/219886 (https://phabricator.wikimedia.org/T87129) (owner: 10Paladox) [23:38:18] (03Merged) 10jenkins-bot: Add scrollbar to list of available transcodes [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/219886 (https://phabricator.wikimedia.org/T87129) (owner: 10Paladox) [23:38:43] (03CR) 10Paladox: "I am not sure if using one of those new mergin config will fix errors configs I am talking about are 2d array and the others. But for now " [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/210176 (https://phabricator.wikimedia.org/T87981) (owner: 10Paladox) [23:41:46] (03PS37) 10Paladox: Add extensions.json, empty PHP entry point [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/210176 (https://phabricator.wikimedia.org/T87981) [23:44:34] (03CR) 10Paladox: "Thanks." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/219886 (https://phabricator.wikimedia.org/T87129) (owner: 10Paladox)