[00:03:16] (03Merged) 10jenkins-bot: build: Upgrade eslint-config-wikimedia 0.12.0, drop grunt-jsonlint [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/508669 (owner: 10Jforrester) [00:09:56] (03CR) 10jenkins-bot: build: Upgrade eslint-config-wikimedia 0.12.0, drop grunt-jsonlint [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/508669 (owner: 10Jforrester) [00:16:55] (03Merged) 10jenkins-bot: build: Upgrade eslint-config-wikimedia 0.12.0, drop grunt-jsonlint [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/508663 (owner: 10Jforrester) [00:20:56] (03CR) 10jenkins-bot: build: Upgrade eslint-config-wikimedia 0.12.0, drop grunt-jsonlint [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/508663 (owner: 10Jforrester) [00:26:01] (03CR) 10MaxSem: [C: 03+2] build: Upgrade eslint-config-wikimedia 0.12.0, drop grunt-jsonlint [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/508659 (owner: 10Jforrester) [00:49:41] (03Merged) 10jenkins-bot: build: Upgrade eslint-config-wikimedia 0.12.0, drop grunt-jsonlint [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/508659 (owner: 10Jforrester) [08:41:29] 10Multimedia, 10SDC Design, 10SDC Engineering, 10Multimedia-Current-Work, 10SDC-Statements (Depicts-Walking): Captions on UW doesn't warn you more than once if your input is over 250 - https://phabricator.wikimedia.org/T222483 (10matthiasmullie) The actual issue here is that UW was configured to allow 25... [09:11:38] (03PS1) 10Matthias Mullie: Use Wikibase's actual max length for label [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509013 (https://phabricator.wikimedia.org/T222483) [09:11:40] (03PS1) 10Matthias Mullie: Allow moving on to next step if captions fail [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509014 (https://phabricator.wikimedia.org/T222481) [09:12:17] 10Multimedia, 10SDC Design, 10SDC Engineering, 10Multimedia-Current-Work, and 2 others: Captions on UW doesn't warn you more than once if your input is over 250 - https://phabricator.wikimedia.org/T222483 (10matthiasmullie) a:03matthiasmullie [09:15:06] 10Multimedia, 10SDC Design, 10SDC Engineering, 10MW-1.34-notes (1.34.0-wmf.5; 2019-05-14), and 3 others: Publish data button should be disabled if nothing has been added yet - https://phabricator.wikimedia.org/T222482 (10matthiasmullie) a:05matthiasmullie→03Edtadros [09:15:19] 10Multimedia, 10SDC Design, 10SDC Engineering, 10UploadWizard, and 4 others: Increase padding between statement blocks on UW - https://phabricator.wikimedia.org/T222485 (10matthiasmullie) a:05matthiasmullie→03Edtadros [09:15:33] 10Multimedia, 10SDC Design, 10UploadWizard, 10MW-1.34-notes (1.34.0-wmf.3; 2019-04-30), and 2 others: Update UploadWizard describe step with text explaining what "publish" actually does - https://phabricator.wikimedia.org/T220429 (10matthiasmullie) a:05PDrouin-WMF→03Edtadros [10:00:25] 10Multimedia, 10SDC Engineering, 10I18n, 10Multimedia-Current-Work, 10RTL: In RTL languages, default instructional text for statement auto-suggest search field overlaps - https://phabricator.wikimedia.org/T222283 (10Volker_E) @Ramsey-WMF Without having tested myself, Farsi `fa` should be in the list I th... [10:05:42] 10Multimedia, 10SDC Engineering, 10I18n, 10Multimedia-Current-Work, 10RTL: Captions Panel alignment and overlapping issues in RTL languages - https://phabricator.wikimedia.org/T222867 (10Volker_E) [10:10:10] 10Multimedia, 10SDC Engineering, 10I18n, 10Multimedia-Current-Work, 10RTL: Captions Panel alignment and overlapping issues in RTL languages - https://phabricator.wikimedia.org/T222867 (10Volker_E) [10:10:30] 10Multimedia, 10SDC Engineering, 10I18n, 10Multimedia-Current-Work, 10RTL: Captions Panel alignment and overlapping issues in RTL languages - https://phabricator.wikimedia.org/T222867 (10Volker_E) [10:11:38] 10Multimedia, 10SDC Design, 10I18n, 10Multimedia-Current-Work, 10RTL: Inconsistent handling of RTL languages in Structured Data UI elements - https://phabricator.wikimedia.org/T221109 (10Volker_E) [10:12:09] 10Multimedia, 10SDC Design, 10I18n, 10Multimedia-Current-Work, 10RTL: Inconsistent handling of RTL languages in Structured Data UI elements - https://phabricator.wikimedia.org/T221109 (10Volker_E) [10:21:06] 10Multimedia, 10SDC Design, 10I18n, 10Multimedia-Current-Work, 10RTL: Inconsistent handling of RTL languages in Structured Data UI elements - https://phabricator.wikimedia.org/T221109 (10Volker_E) When a language is chosen (in the examples for problem #1 of the task description Hebrew), it becomes the pr... [10:40:53] 10Multimedia, 10SDC Design, 10SDC Engineering, 10UploadWizard, and 4 others: Increase padding between statement blocks on UW - https://phabricator.wikimedia.org/T222485 (10Cparle) [10:42:02] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review: [SDOC] There is no link to Wikidata item in depicts: when there is no label in the currently used language - https://phabricator.wikimedia.org/T221676 (10Cparle) [10:45:22] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review, 10SDC-Statements (Other-statements-running): Basic depicts support for campaigns in UploadWizard - https://phabricator.wikimedia.org/T219368 (10Cparle) @Ramsey-WMF this is verified working on labs - do we need to verify it on... [10:46:19] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review, 10SDC-Statements (Depicts-on-training-wheels): [Alpha/Labs] OOUI element/font size is too large - https://phabricator.wikimedia.org/T218614 (10Cparle) @PDrouin-WMF could you check this on production now, and close the ticket... [10:47:55] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review, 10SDC-Statements (Depicts-on-training-wheels): [Alpha/Labs] OOUI element/font size is too large - https://phabricator.wikimedia.org/T218614 (10Cparle) Gah sorry @PDrouin-WMF I misread the ticket ... this is *not* testable on... [10:52:08] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10SDC-Statements (Statement-and-keyword-search-power-walking): Make sure that the order of statements on the File page is correct - https://phabricator.wikimedia.org/T220213 (10Cparle) Cannot reasonably be verified until other statements goes live [10:55:14] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review: Add php integration tests for Depicts - https://phabricator.wikimedia.org/T220113 (10Cparle) [10:59:23] 10Multimedia, 10SDC Engineering, 10SDC-Statements (Statement-and-keyword-search-power-walking): [DO THIS FIRST] Auto-suggest widget that returns properties limited to 1 or more data types - https://phabricator.wikimedia.org/T219378 (10Lucas_Werkmeister_WMDE) [11:02:37] 10Multimedia, 10SDC Engineering, 10Wikidata, 10MW-1.34-notes (1.34.0-wmf.4; 2019-05-07), and 3 others: interwiki items are formatted as NonExistingEntityId in text/html when no label exists in requested language - https://phabricator.wikimedia.org/T222138 (10WMDE-leszek) @matthiasmullie that code make it i... [11:33:44] 10Multimedia, 10SDC Engineering, 10StructuredDataOnCommons, 10Wikidata: Wikibase Repo api modules and special pages should be conditionally loaded based on entity types enabled (or at least, not crash) - https://phabricator.wikimedia.org/T207683 (10hoo) [11:46:01] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review: [SDOC] There is no link to Wikidata item in depicts: when there is no label in the currently used language - https://phabricator.wikimedia.org/T221676 (10matthiasmullie) 05Open→03Resolved [11:46:19] 10Multimedia, 10SDC Engineering, 10Wikidata, 10MW-1.34-notes (1.34.0-wmf.4; 2019-05-07), and 3 others: interwiki items are formatted as NonExistingEntityId in text/html when no label exists in requested language - https://phabricator.wikimedia.org/T222138 (10matthiasmullie) 05Open→03Resolved a:03matth... [14:13:59] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review, 10SDC-Statements (Depicts-on-training-wheels): [Alpha/Labs] OOUI element/font size is too large - https://phabricator.wikimedia.org/T218614 (10PDrouin-WMF) makes sense to me, @Cparle [14:14:47] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review, 10SDC-Statements (Depicts-on-training-wheels): [Alpha/Labs] OOUI element/font size is too large - https://phabricator.wikimedia.org/T218614 (10Cparle) 05Open→03Resolved a:03Cparle [15:01:32] (03PS7) 10Matthias Mullie: Require statement exports from other modules [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/507567 [15:07:57] (03PS3) 10Matthias Mullie: Remove mw.mediaInfo.statements [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/507568 [15:20:46] (03CR) 10jerkins-bot: [V: 04-1] Require statement exports from other modules [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/507567 (owner: 10Matthias Mullie) [15:29:16] (03CR) 10jerkins-bot: [V: 04-1] Remove mw.mediaInfo.statements [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/507568 (owner: 10Matthias Mullie) [15:49:31] (03CR) 10Jforrester: [C: 03+2] Use Wikibase's actual max length for label [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509013 (https://phabricator.wikimedia.org/T222483) (owner: 10Matthias Mullie) [15:55:49] 10Multimedia, 10SDC Engineering, 10Multimedia-Current-Work, 10Patch-For-Review, 10SDC-Statements (Other-statements-running): Basic depicts support for campaigns in UploadWizard - https://phabricator.wikimedia.org/T219368 (10Ramsey-WMF) Yes. >>! In T219368#5169669, @Cparle wrote: > @Ramsey-WMF this is ve... [16:07:53] 10Multimedia: [Spike] investigate how much effort would be involved in wrapping wikibase.mediainfo.statements in an Adapter - https://phabricator.wikimedia.org/T218803 (10Ramsey-WMF) p:05Triage→03Normal We'll pick this up after Other Statements release but before Annotations and machine vision. [16:11:02] 10Multimedia, 10SDC Engineering: [Spike] investigate how much effort would be involved in wrapping wikibase.mediainfo.statements in an Adapter - https://phabricator.wikimedia.org/T218803 (10Ramsey-WMF) [16:57:44] 10Multimedia, 10SDC Engineering, 10Wikidata, 10Patch-For-Review: If a MediaInfo items depicts something that has its own 'depicts' statements, add those to the search index too - https://phabricator.wikimedia.org/T199241 (10Cparle) [17:00:55] 10Multimedia, 10SDC Engineering, 10Wikidata, 10Patch-For-Review: If a MediaInfo items depicts something that has its own 'depicts' statements, add those to the search index too - https://phabricator.wikimedia.org/T199241 (10Cparle) @Ramsey-WMF moving out of current work, because we're not currently working... [17:01:55] 10Multimedia, 10SDC Engineering, 10Patch-For-Review: Basic implementation of virtual statements in MediaInfo using DerivedPropertyValueSnak - https://phabricator.wikimedia.org/T207863 (10Cparle) [17:02:18] 10Multimedia, 10SDC Engineering, 10Patch-For-Review: Basic implementation of virtual statements in MediaInfo using DerivedPropertyValueSnak - https://phabricator.wikimedia.org/T207863 (10Cparle) @Ramsey-WMF moving out of current work, because we're not currently working on it [17:27:28] 10Multimedia, 10SDC Engineering, 10I18n, 10Multimedia-Current-Work, 10RTL: In RTL languages, default instructional text for statement auto-suggest search field overlaps - https://phabricator.wikimedia.org/T222283 (10Ramsey-WMF) [17:28:28] 10Multimedia, 10SDC Engineering, 10I18n, 10Multimedia-Current-Work, 10RTL: In RTL languages, default instructional text for statement auto-suggest search field overlaps - https://phabricator.wikimedia.org/T222283 (10Ramsey-WMF) [17:29:02] 10Multimedia, 10SDC Engineering, 10I18n, 10Multimedia-Current-Work, 10RTL: In RTL languages, default instructional text for statement auto-suggest search field overlaps - https://phabricator.wikimedia.org/T222283 (10Ramsey-WMF) >>! In T222283#5169561, @Volker_E wrote: > @Ramsey-WMF Without having tested... [18:18:56] (03PS2) 10Jforrester: Use Wikibase's actual max length for label [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509013 (https://phabricator.wikimedia.org/T222483) (owner: 10Matthias Mullie) [18:19:02] (03CR) 10Jforrester: [C: 03+2] "…" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509013 (https://phabricator.wikimedia.org/T222483) (owner: 10Matthias Mullie) [18:19:10] (03PS2) 10Jforrester: Allow moving on to next step if captions fail [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509014 (https://phabricator.wikimedia.org/T222481) (owner: 10Matthias Mullie) [18:19:14] (03CR) 10Jforrester: [C: 03+2] Allow moving on to next step if captions fail [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509014 (https://phabricator.wikimedia.org/T222481) (owner: 10Matthias Mullie) [18:19:44] (03PS2) 10Jforrester: Move UploadWizardHooks.php into includes folder [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/507557 (owner: 10Umherirrender) [18:21:45] (03CR) 10jerkins-bot: [V: 04-1] Move UploadWizardHooks.php into includes folder [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/507557 (owner: 10Umherirrender) [18:24:36] (03PS3) 10Jforrester: Move UploadWizardHooks.php into includes folder [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/507557 (owner: 10Umherirrender) [18:28:35] (03Merged) 10jenkins-bot: Use Wikibase's actual max length for label [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509013 (https://phabricator.wikimedia.org/T222483) (owner: 10Matthias Mullie) [18:30:30] (03CR) 10jerkins-bot: [V: 04-1] Move UploadWizardHooks.php into includes folder [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/507557 (owner: 10Umherirrender) [18:35:23] (03CR) 10jenkins-bot: Use Wikibase's actual max length for label [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509013 (https://phabricator.wikimedia.org/T222483) (owner: 10Matthias Mullie) [18:36:35] (03Merged) 10jenkins-bot: Allow moving on to next step if captions fail [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509014 (https://phabricator.wikimedia.org/T222481) (owner: 10Matthias Mullie) [18:37:37] (03PS4) 10Umherirrender: Move UploadWizardHooks.php into includes folder [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/507557 [18:41:17] (03CR) 10jenkins-bot: Allow moving on to next step if captions fail [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/509014 (https://phabricator.wikimedia.org/T222481) (owner: 10Matthias Mullie) [19:32:42] (03CR) 10Jforrester: [C: 03+2] Move UploadWizardHooks.php into includes folder [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/507557 (owner: 10Umherirrender) [19:41:12] (03Merged) 10jenkins-bot: Move UploadWizardHooks.php into includes folder [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/507557 (owner: 10Umherirrender) [19:44:50] (03CR) 10jenkins-bot: Move UploadWizardHooks.php into includes folder [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/507557 (owner: 10Umherirrender) [21:10:31] 10Multimedia, 10Commons, 10MediaWiki-File-management, 10SDC General, 10Wikidata: Provide a way to curate and fetch titles (short descriptions) for media files - https://phabricator.wikimedia.org/T122381 (10Abit) I feel like SDC captions satisfies the use cases in the task description. Can we close this... [21:58:02] 10Multimedia, 10Commons, 10MediaWiki-File-management, 10SDC General, 10Wikidata: Provide a way to curate and fetch titles (short descriptions) for media files - https://phabricator.wikimedia.org/T122381 (10Tgr) 05Open→03Resolved a:03Abit Yay! (For those not familiar with SDC, see [[https://www.med... [22:46:49] 10Multimedia, 10MediaWiki-Uploading, 10Patch-For-Review, 10Wikimedia-production-error: Upload failed with fatal error: Call to load() on a non-object in UploadBase.php - https://phabricator.wikimedia.org/T208539 (10Krinkle) Thanks. I'm landing Umherirrender patch and will circle back to see if this rare is... [23:00:42] (03CR) 10Krinkle: "Hm.. would it not still be dependant on skin context in some way? Looks like it's not compatible with Minerva for example." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/508406 (owner: 10Krinkle) [23:00:46] (03CR) 10Krinkle: "(an unrelated issue I suppose)" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/508406 (owner: 10Krinkle) [23:01:05] (03CR) 10Krinkle: "Will break off the clean up from the switch, agreed not worth it if that's indeed the plan." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/508406 (owner: 10Krinkle) [23:02:09] (03PS2) 10Krinkle: wikibase.mediainfo.search: Simplify widget UI insertion code [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/508406 [23:51:46] 10Multimedia, 10Discovery-Search, 10GeoData, 10GlobalUsage, and 5 others: Wikimedia\Rdbms\LBFactory::getEmptyTransactionTicket: GeoData\Hooks::doLinksUpdate does not have outer scope - https://phabricator.wikimedia.org/T221577 (10Krinkle) >>! In T221577#5163318, @aaron wrote: > It's the new use of Deferred...