[01:49:27] (03CR) 10Reedy: "recheck" [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590454 (owner: 10Libraryupgrader) [01:50:41] (03PS2) 10Gergő Tisza: Move rest of API code into GrowthTasksApi [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/589714 [01:50:43] (03PS1) 10Gergő Tisza: [WIP] Post-edit dialog [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/590790 (https://phabricator.wikimedia.org/T245790) [02:00:35] (03CR) 10jerkins-bot: [V: 04-1] [WIP] Post-edit dialog [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/590790 (https://phabricator.wikimedia.org/T245790) (owner: 10Gergő Tisza) [02:17:03] (03PS1) 10Reedy: Fix some excessive indenting [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590804 [02:21:33] (03CR) 10jerkins-bot: [V: 04-1] Fix some excessive indenting [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590804 (owner: 10Reedy) [02:26:59] 10Growth-Team, 10PageCuration: PageTriage ArticleMetadataTest::testValidatePageIds failing - https://phabricator.wikimedia.org/T250643 (10Reedy) [02:28:00] (03CR) 10Reedy: "Failure is T250643" [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590454 (owner: 10Libraryupgrader) [02:28:28] (03Abandoned) 10Reedy: Test [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/584668 (owner: 10DannyS712) [02:29:10] 10Growth-Team, 10PageCuration, 10ci-test-error (WMF-deployed Build Failure): PageTriage ArticleMetadataTest::testValidatePageIds failing - https://phabricator.wikimedia.org/T250643 (10Reedy) [02:47:09] (03PS1) 10DannyS712: Check something [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 [02:47:59] (03PS2) 10DannyS712: Check something [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 [02:51:51] (03CR) 10Reedy: Check something (031 comment) [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 (owner: 10DannyS712) [02:52:06] (03CR) 10jerkins-bot: [V: 04-1] Check something [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 (owner: 10DannyS712) [02:52:25] (03PS3) 10DannyS712: Check something [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 [02:53:20] (03CR) 10DannyS712: Check something (031 comment) [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 (owner: 10DannyS712) [02:54:15] (03PS2) 10Reedy: Fix some excessive indenting [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590804 [02:56:45] (03CR) 10jerkins-bot: [V: 04-1] Check something [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 (owner: 10DannyS712) [02:57:40] 10Growth-Team, 10PageCuration, 10ci-test-error (WMF-deployed Build Failure): PageTriage ArticleMetadataTest::testValidatePageIds failing - https://phabricator.wikimedia.org/T250643 (10DannyS712) [02:58:26] (03PS4) 10DannyS712: ArticleMetadataTest::testValidatePageIds starts with invalid page ids [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 [02:58:29] (03CR) 10jerkins-bot: [V: 04-1] Fix some excessive indenting [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590804 (owner: 10Reedy) [02:58:49] 10Growth-Team, 10PageCuration, 10ci-test-error (WMF-deployed Build Failure): PageTriage ArticleMetadataTest::testValidatePageIds failing - https://phabricator.wikimedia.org/T250643 (10DannyS712) See https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageTriage/+/590816/ for a demonstration of the immedi... [03:01:54] (03CR) 10DannyS712: [C: 03+1] "Noticed that too; LGTM (once the tests are fixed)" [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590804 (owner: 10Reedy) [03:02:51] (03CR) 10jerkins-bot: [V: 04-1] ArticleMetadataTest::testValidatePageIds starts with invalid page ids [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 (owner: 10DannyS712) [03:14:00] (03PS5) 10DannyS712: ArticleMetadataTest::testValidatePageIds starts with invalid page ids [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 [03:15:26] (03CR) 10jerkins-bot: [V: 04-1] ArticleMetadataTest::testValidatePageIds starts with invalid page ids [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 (owner: 10DannyS712) [03:16:53] (03PS6) 10DannyS712: ArticleMetadataTest::testValidatePageIds starts with invalid page ids [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 [03:21:23] (03CR) 10jerkins-bot: [V: 04-1] ArticleMetadataTest::testValidatePageIds starts with invalid page ids [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 (owner: 10DannyS712) [03:22:32] (03PS7) 10DannyS712: ArticleMetadataTest::testValidatePageIds starts with invalid page ids [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 [03:27:34] (03CR) 10jerkins-bot: [V: 04-1] ArticleMetadataTest::testValidatePageIds starts with invalid page ids [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/590816 (owner: 10DannyS712) [03:52:08] (03CR) 10DannyS712: For Action::__construct() use Article instead of Page (031 comment) [extensions/Flow] - 10https://gerrit.wikimedia.org/r/588517 (https://phabricator.wikimedia.org/T249146) (owner: 10Art-Baltai) [08:12:22] (03CR) 10Urbanecm: "recheck" [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/590573 (https://phabricator.wikimedia.org/T250554) (owner: 10Urbanecm) [09:47:21] (03PS3) 10Gergő Tisza: Move rest of API code into GrowthTasksApi [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/589714 (https://phabricator.wikimedia.org/T245790) [09:47:23] (03PS2) 10Gergő Tisza: [WIP] Post-edit dialog [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/590790 (https://phabricator.wikimedia.org/T245790) [09:57:04] 10Growth-Team, 10Notifications, 10MediaWiki-Page-editing, 10User-DannyS712: Duplicate notifications for my first edit - https://phabricator.wikimedia.org/T250668 (10DannyS712) [09:57:17] 10Growth-Team, 10Notifications, 10MediaWiki-Page-editing, 10User-DannyS712: Duplicate notifications for my first edit - https://phabricator.wikimedia.org/T250668 (10DannyS712) [09:57:41] 10Growth-Team, 10Notifications, 10MediaWiki-Page-editing, 10User-DannyS712: Duplicate notifications for my first edit - https://phabricator.wikimedia.org/T250668 (10DannyS712) I would guess it was caused by the first 2 rollbacks happening so quickly together [10:29:08] (03CR) 10DannyS712: HomepageHooks: Fix talk tab destination on user subpages (031 comment) [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/590573 (https://phabricator.wikimedia.org/T250554) (owner: 10Urbanecm) [10:30:20] 10Growth-Team, 10Notifications: Wikipedia interface: add "Muted pages" option to Preferences > Notifications - like the existing "Muted users" - https://phabricator.wikimedia.org/T250673 (10DannyS712) Muting users was part of #anti-harassment, but I don't think muting pages would be. Just in case, cc @Niharika [10:40:28] (03PS2) 10Urbanecm: HomepageHooks: Fix talk tab destination on user subpages [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/590573 (https://phabricator.wikimedia.org/T250554) [10:41:40] (03CR) 10Urbanecm: HomepageHooks: Fix talk tab destination on user subpages (031 comment) [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/590573 (https://phabricator.wikimedia.org/T250554) (owner: 10Urbanecm) [10:48:43] 10Growth-Team, 10Notifications: Wikipedia interface: add "Muted pages" option to Preferences > Notifications - like the existing "Muted users" - https://phabricator.wikimedia.org/T250673 (10Aklapper) @Neolexx: Could you explain why you do not drop that page from the watch list? [10:49:09] 10Growth-Team, 10Notifications: Add "Muted pages" option to Preferences > Notifications - like existing "Muted users" - https://phabricator.wikimedia.org/T250673 (10Aklapper) [11:00:44] (03CR) 10Gergő Tisza: HomepageHooks: Fix talk tab destination on user subpages (031 comment) [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/590573 (https://phabricator.wikimedia.org/T250554) (owner: 10Urbanecm) [11:11:00] 10Growth-Team, 10Notifications: Add "Muted pages" option to Preferences > Notifications - like existing "Muted users" - https://phabricator.wikimedia.org/T250673 (10Neolexx) >>! In T250673#6070890, @Aklapper wrote: > @Neolexx: Could you explain why you do not drop that page from the watch list? To be honnest I... [11:18:06] 10Growth-Team, 10Notifications, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10Ammarpad) [11:18:08] 10Growth-Team, 10Notifications: Add "Muted pages" option to Preferences > Notifications - like existing "Muted users" - https://phabricator.wikimedia.org/T250673 (10Ammarpad) [12:29:01] 10Growth-Team, 10Notifications, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10Neolexx) I am a bit confused by reading https://www.mediawiki.org/wiki/Help:Notifications/Types#Page_links Is it possible - or even default behavior that 1) t... [12:38:30] (03CR) 10Kosta Harlan: "postscript: Writing to this.$foot could be considered a hack, since OOUI wants to use that for the values set in actions. But since we don" [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/589590 (https://phabricator.wikimedia.org/T244541) (owner: 10Kosta Harlan) [13:01:21] (03CR) 10Kosta Harlan: "I guess it's OK but I wonder if we should try to address the fixme's instead of merging this patch as is. However at the moment I don't ha" (032 comments) [extensions/Flow] - 10https://gerrit.wikimedia.org/r/585118 (https://phabricator.wikimedia.org/T160813) (owner: 10Aaron Schulz) [13:09:23] (03CR) 10Zfilipin: Upgrade WebdriverIO version to v5 (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/590284 (https://phabricator.wikimedia.org/T250610) (owner: 10Gaura87) [14:09:43] (03PS5) 10Gaura87: Upgrade WebdriverIO version to v5 [extensions/Echo] - 10https://gerrit.wikimedia.org/r/590284 (https://phabricator.wikimedia.org/T250610) [14:23:59] (03CR) 10jerkins-bot: [V: 04-1] Upgrade WebdriverIO version to v5 [extensions/Echo] - 10https://gerrit.wikimedia.org/r/590284 (https://phabricator.wikimedia.org/T250610) (owner: 10Gaura87) [14:48:56] 10Growth-Team, 10Growth-Scaling, 10GrowthExperiments-Help panel: Scale: support various help desk archiving systems - https://phabricator.wikimedia.org/T248976 (10Trizek-WMF) [14:50:02] 10Growth-Team, 10Growth-Scaling, 10GrowthExperiments-Help panel: Scale: support various help desk archiving systems - https://phabricator.wikimedia.org/T248976 (10Trizek-WMF) 05Open→03Resolved Even In the case we don't choose to go with {T250235}, I created the list. Honestly, we have too much cases to c... [14:50:46] 10Growth-Team, 10Cognate, 10ContentTranslation, 10DBA, and 10 others: Restart extension1 (x1) database primary master (db1120) - https://phabricator.wikimedia.org/T250701 (10Marostegui) [15:25:54] 10Growth-Team, 10Notifications, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10Quiddity) @Neolexx Yes, that is correct. Unfortunately, it is currently only possible to uncheck //all// "Page link" notifications. There is no relationship b... [15:44:52] 10Growth-Team, 10Growth-Scaling, 10Chinese-Sites: [Growth scaling] Group 1: community engagement for deployment on fr, pt, pl, fa, sv, da, it, id, zh Wikipedias - https://phabricator.wikimedia.org/T250481 (10Trizek-WMF) p:05Triage→03High Given the pandemic, we have decided to target one on these wikis fi... [16:47:34] (03CR) 10Gaura87: "> Patch Set 5: Verified-1" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/590284 (https://phabricator.wikimedia.org/T250610) (owner: 10Gaura87) [16:52:11] (03CR) 10Jforrester: [C: 03+2] Move one class to AutoloadNamespaces [extensions/Echo] - 10https://gerrit.wikimedia.org/r/590009 (https://phabricator.wikimedia.org/T187154) (owner: 10Reedy) [17:04:56] 10Growth-Team, 10GrowthExperiments: Suggested edit cards should show display title, not title - https://phabricator.wikimedia.org/T250719 (10Tgr) [17:06:37] (03CR) 10jerkins-bot: [V: 04-1] Move one class to AutoloadNamespaces [extensions/Echo] - 10https://gerrit.wikimedia.org/r/590009 (https://phabricator.wikimedia.org/T187154) (owner: 10Reedy) [17:06:53] 10Growth-Team, 10CommRel-Specialists-Support (Apr-Jun-2020), 10Goal, 10User-Johan: Interim management of the Growth ambassadors - https://phabricator.wikimedia.org/T190311 (10Johan) [17:11:52] 10Growth-Team, 10Notifications, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10Neolexx) >>! In T46787#6071872, @Quiddity wrote: > @Neolexx Yes, that is correct. Unfortunately, it is currently only possible to uncheck //all// "Page link"... [17:12:59] 10Growth-Team, 10MediaWiki-Recent-changes, 10Advanced Mobile Contributions, 10Mobile: Deletion summary in Special:RecentChanges is shown incorrectly - https://phabricator.wikimedia.org/T250308 (10Etonkovidova) @Zoranzoki21 yes, #advanced_mobile_contributions can be removed for this task - thanks! [17:16:14] 10Growth-Team, 10MediaWiki-Recent-changes, 10Mobile: Deletion summary in Special:RecentChanges is shown incorrectly - https://phabricator.wikimedia.org/T250308 (10Zoranzoki21) >>! In T250308#6072535, @Etonkovidova wrote: > @Zoranzoki21 yes, #advanced_mobile_contributions can be removed for this task - thank... [17:20:15] (03CR) 10Reedy: "T250728 filed for the Wikibase selenium fail" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/590009 (https://phabricator.wikimedia.org/T187154) (owner: 10Reedy) [17:42:01] 10Growth-Team, 10Notifications, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10Quiddity) I'm not a developer at all, but maybe @kostajh can give some quick insights? p.s. Thank you for being a Tech Ambassador from ru-wiki! [17:59:35] (03CR) 10Gaura87: "recheck" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/590284 (https://phabricator.wikimedia.org/T250610) (owner: 10Gaura87) [19:35:08] 10Growth-Team, 10AntiSpoof, 10PageCuration, 10ProofreadPage, and 5 others: Wikimedia\Rdbms\Database::normalizeUpsertKeys called with deprecated parameter style: the unique key array should be a string or array of string arrays generating 2 million warnings in 24 ho... - https://phabricator.wikimedia.org/T248147 [19:57:21] (03PS1) 10Kosta Harlan: Add page linked event title blacklist [extensions/Echo] - 10https://gerrit.wikimedia.org/r/591154 (https://phabricator.wikimedia.org/T46787) [20:04:11] 10Edit-Review-Improvements-RC-Page, 10Growth-Team, 10MediaWiki-Recent-changes, 10MediaWiki-extensions-WikibaseClient, 10Wikidata: Bot edits on Wikidata should be hidden in other project's recent changes - https://phabricator.wikimedia.org/T246746 (10Jarekt) I am using QuickStatements on Commons to add st... [20:04:21] 10Growth-Team, 10Notifications, 10Patch-For-Review, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10kostajh) no, a JavaScript hook isn't really going to work for this, it has to be dealt with server side. I've pushed a patch that adds... [20:26:51] 10Growth-Team, 10Notifications, 10Patch-For-Review, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10Quiddity) I //think// this solution will solve the majority of the problems, for the majority of users. Can another editor or two, plea... [20:27:52] 10Growth-Team, 10Notifications, 10Patch-For-Review, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10Neolexx) >>! In T46787#6073227, @kostajh wrote: > Is this going to satisfy the use cases that have been written up here? Yes, that would... [20:34:51] 10Growth-Team, 10Notifications, 10Patch-For-Review, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10Levivich) >>! In T46787#6073256, @Quiddity wrote: > Can another editor or two, please confirm, with your thoughts? Speaking for myself,... [20:46:18] 10Growth-Team, 10StructuredDiscussions, 10Accessibility: Keyboard focus doesn't go on list items of topic three dots - https://phabricator.wikimedia.org/T250749 (10Bkudiess-msft) [20:51:49] (03PS1) 10Bkudiess-msft: Adds focus feedback to dropdown list of actions [extensions/Flow] - 10https://gerrit.wikimedia.org/r/591176 [21:04:23] 10Growth-Team, 10CommRel-Specialists-Support (Jan-Mar-2020), 10Goal: [goal] Deploy Growth features on wikis requesting them - https://phabricator.wikimedia.org/T228980 (10Etonkovidova) [21:13:41] 10Growth-Team, 10Notifications, 10Accessibility: Badge icon focus not visible - https://phabricator.wikimedia.org/T250753 (10Bkudiess-msft) [21:24:20] 10Growth-Team, 10MediaWiki-Maintenance-system, 10Wikidata, 10Core Platform Team Workboards (Clinic Duty Team), and 3 others: DatabaseUpdater: protect methods for direct database modification - https://phabricator.wikimedia.org/T249603 (10daniel) 05Open→03Resolved [21:24:30] 10Growth-Team, 10MediaWiki-Maintenance-system, 10Performance-Team, 10StructuredDiscussions, and 5 others: sql.php runs LoadExtensionSchemaUpdates - https://phabricator.wikimedia.org/T157651 (10daniel) [21:50:26] 10Growth-Team, 10Notifications, 10User-DannyS712: Duplicate notifications for my first edit - https://phabricator.wikimedia.org/T250668 (10matmarex) [22:17:01] 10Growth-Team, 10JsonConfig, 10MassMessage, 10MediaWiki-Revision-backend, and 8 others: DifferenceEngine $mNewRev and $mOldRev are Revision objects - https://phabricator.wikimedia.org/T250761 (10DannyS712) [22:17:10] 10Growth-Team, 10JsonConfig, 10MassMessage, 10MediaWiki-Revision-backend, and 8 others: DifferenceEngine $mNewRev and $mOldRev are Revision objects - https://phabricator.wikimedia.org/T250761 (10DannyS712) p:05Triage→03Medium [22:19:27] 10Growth-Team, 10JsonConfig, 10MassMessage, 10MediaWiki-Revision-backend, and 8 others: DifferenceEngine $mNewRev and $mOldRev are Revision objects - https://phabricator.wikimedia.org/T250761 (10DannyS712) [22:30:50] 10Growth-Team, 10Thanks, 10Anti-Harassment, 10MediaWiki-Partial-blocks, and 2 others: Allow a user to be blocked from sending thanks - https://phabricator.wikimedia.org/T242785 (10DannyS712) p:05Triage→03Low a:03DannyS712 Tagging #anti-harassment for product approval before proceeding [22:53:27] (03CR) 10Ppchelko: [C: 03+1] "This is indeed achieving it's goal of removing the deprecate method. However, in theory there should be no need at all for Flow to be expl" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/587313 (https://phabricator.wikimedia.org/T246284) (owner: 10DannyS712) [22:55:53] 10Growth-Team, 10Notifications, 10Patch-For-Review, 10patch-welcome: Allow excluding pages from the page links notifications - https://phabricator.wikimedia.org/T46787 (10Thryduulf) I think this idea would be good, and would be good for me. I think if you're wanting to exclude very many pages that you'd be... [22:58:14] 10Growth-Team, 10MinervaNeue, 10StructuredDiscussions, 10Design, and 4 others: Consolidate MinervaNeue and Flow on one WikimediaUI icon for ToC - https://phabricator.wikimedia.org/T232383 (10Jdlrobson) [22:58:27] 10Growth-Team, 10MinervaNeue, 10StructuredDiscussions, 10Design, and 4 others: Consolidate MinervaNeue and Flow on one WikimediaUI icon for ToC - https://phabricator.wikimedia.org/T232383 (10Jdlrobson) This code appears to be in production. [23:03:56] (03PS1) 10DannyS712: Replace use of `ArticleRollbackComplete` hook with `RollbackComplete` [extensions/Echo] - 10https://gerrit.wikimedia.org/r/591215 (https://phabricator.wikimedia.org/T250543) [23:06:12] (03PS2) 10DannyS712: Replace use of `ArticleRollbackComplete` hook with `RollbackComplete` [extensions/Echo] - 10https://gerrit.wikimedia.org/r/591215 (https://phabricator.wikimedia.org/T250543) [23:15:06] (03PS1) 10Reedy: Use namespaced PageImages [extensions/GrowthExperiments] - 10https://gerrit.wikimedia.org/r/591223 [23:21:05] 10Growth-Team, 10Notifications, 10MachineVision, 10MediaWiki-Revision-backend, and 9 others: Replace `ArticleRollbackComplete` hook - https://phabricator.wikimedia.org/T250543 (10DannyS712) @Mholloway for #machinevision and #wikimediaeditortasks is it okay to bump the required version? Currently they suppo... [23:32:48] 10Growth-Team, 10Notifications, 10MediaWiki-Revision-backend, 10MediaWiki-extensions-WikibaseRepository, and 8 others: Hard deprecate Article::getRevision and WikiPage::getRevision - https://phabricator.wikimedia.org/T250532 (10DannyS712) 05Open→03Resolved