[02:23:46] 10VisualEditor, 10VisualEditor-DataModel, 10User-Ryasmeen: ve.ui.MWWikitextStringTransferHandler uses bare content - https://phabricator.wikimedia.org/T232944 (10dchan) [06:46:40] 10VisualEditor, 10BlueSpice, 10translatewiki.net, 10Patch-For-Review, and 2 others: PHP Notice: Key 8:visualeditor-linkcontext-label-label already belongs to ext-bluespicevisualeditorconnector, conflict with mwgerrit-visualeditor-ve - https://phabricator.wikimedia.org/T232892 (10Zoranzoki21) 05Open→03Re... [06:58:20] 10VisualEditor, 10BlueSpice, 10translatewiki.net, 10Patch-For-Review, and 2 others: PHP Notice: Key 8:visualeditor-linkcontext-label-label already belongs to ext-bluespicevisualeditorconnector, conflict with mwgerrit-visualeditor-ve - https://phabricator.wikimedia.org/T232892 (10Raymond) >>! In T232892#549... [07:34:50] (03PS2) 10Divec: TreeModifier: delete text nodes emptied by moveText operations [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/536754 (https://phabricator.wikimedia.org/T232942) [08:06:19] (03CR) 10jerkins-bot: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/536944 (owner: 10L10n-bot) [08:31:57] (03CR) 10Raimond Spekking: [C: 03+2] "false positive" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/536944 (owner: 10L10n-bot) [08:57:59] (03CR) 10jenkins-bot: Localisation updates from https://translatewiki.net. [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/536944 (owner: 10L10n-bot) [09:25:40] (03CR) 10jenkins-bot: Localisation updates from https://translatewiki.net. [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/536913 (owner: 10L10n-bot) [09:52:17] 10VisualEditor: 404 error when using VisualEditor: apierror-visualeditor-docserver-http - https://phabricator.wikimedia.org/T230272 (10IndyJr) The problem I reported still go on, on random pages, and is not reproducible. The only workaround I found to don't lose job until then, is to copy and paste into a new wi... [09:55:30] 10Citoid, 10Core Platform Team Workboards (Clinic Duty Team), 10Patch-For-Review: In cite journal, do not add URL expanded from existing identifier like DOI - https://phabricator.wikimedia.org/T232771 (10mobrovac) [10:02:59] 10Citoid, 10Core Platform Team Workboards (Clinic Duty Team), 10Patch-For-Review: In cite journal, do not add URL expanded from existing identifier like DOI - https://phabricator.wikimedia.org/T232771 (10Mvolz) Dupe of T190850, I think. [10:06:27] 10TemplateData, 10VisualEditor: Inconsistent detection of JSON errors in templatedata - https://phabricator.wikimedia.org/T232954 (10Mvolz) >>! In T232954#5494396, @matmarex wrote: > I remember that at some point in the migration //to// HHVM we discovered that HHVM's JSON parser was less strict than PHP's, and... [10:07:08] 10TemplateData, 10VisualEditor: Inconsistent detection of JSON errors in templatedata - https://phabricator.wikimedia.org/T232954 (10Mvolz) [10:07:12] 10TemplateData, 10VisualEditor, 10HHVM: TemplateData's PHP JSON validation isn't strict enough because WMF cluster's HHVM allows trailing commas - https://phabricator.wikimedia.org/T128029 (10Mvolz) [10:44:02] (03CR) 10Esanders: [C: 03+2] TreeModifier: delete text nodes emptied by moveText operations [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/536754 (https://phabricator.wikimedia.org/T232942) (owner: 10Divec) [10:51:07] (03Merged) 10jenkins-bot: TreeModifier: delete text nodes emptied by moveText operations [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/536754 (https://phabricator.wikimedia.org/T232942) (owner: 10Divec) [10:52:23] 10TemplateData, 10VisualEditor, 10HHVM: TemplateData's PHP JSON validation isn't strict enough because WMF cluster's HHVM allows trailing commas - https://phabricator.wikimedia.org/T128029 (10Mvolz) Another complaint about this here: https://www.mediawiki.org/wiki/Topic:V6u2mpy11qiu9705 [11:22:57] (03CR) 10jenkins-bot: TreeModifier: delete text nodes emptied by moveText operations [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/536754 (https://phabricator.wikimedia.org/T232942) (owner: 10Divec) [11:34:25] 10Citoid, 10User-Josve05a: Do not include "PMC" part of the PMC identifier - https://phabricator.wikimedia.org/T155887 (10Mvolz) [11:34:28] 10Citoid, 10VisualEditor, 10User-Ryasmeen: Visual Editor adds erroneous text "PMC" to citation template pmc= parameter - https://phabricator.wikimedia.org/T157152 (10Mvolz) [11:36:26] 10Citoid, 10VisualEditor, 10User-Ryasmeen: Visual Editor adds erroneous text "PMC" to citation template pmc= parameter - https://phabricator.wikimedia.org/T157152 (10Mvolz) https://doi.org/10.1186/1471-2148-7-97 still includes the PMC, re-opening. [11:38:42] 10Citoid, 10VisualEditor, 10User-Ryasmeen: Visual Editor adds erroneous text "PMC" to citation template pmc= parameter - https://phabricator.wikimedia.org/T157152 (10Mvolz) 05Resolved→03Open [11:39:05] 10Citoid, 10VisualEditor, 10User-Ryasmeen: Visual Editor adds erroneous text "PMC" to citation template pmc= parameter - https://phabricator.wikimedia.org/T157152 (10Mvolz) Re-opening as some of them still have the PMC and this is inconsistent, i.e. 10.1186/1471-2148-7-97 [12:15:00] 10VisualEditor-DataModel, 10User-Ryasmeen, 10VisualEditor (Current work): ve.ui.MWWikitextStringTransferHandler uses bare content - https://phabricator.wikimedia.org/T232944 (10Esanders) [12:15:12] 10VisualEditor-DataModel, 10VisualEditor (Current work): TreeModifier#applyTreeOperation doesn't delete text nodes emptied by moveText operations - https://phabricator.wikimedia.org/T232942 (10Esanders) [12:18:50] 10VisualEditor: Potential API Errors Associated with VE on Meta-Wiki - https://phabricator.wikimedia.org/T232530 (10Esanders) Can you still reproduce this error? It could have been a transient server issue. [12:21:39] 10VisualEditor: Potential API Errors Associated with VE on Meta-Wiki - https://phabricator.wikimedia.org/T232530 (10Ammarpad) [12:21:44] 10VisualEditor: 404 error when using VisualEditor: apierror-visualeditor-docserver-http - https://phabricator.wikimedia.org/T230272 (10Ammarpad) [12:26:34] 10VisualEditor, 10VisualEditor-ContentEditable, 10VisualEditor-CopyPaste: Pasting multiple list items in VisualEditor in Firefox results in incorrect display and saving - https://phabricator.wikimedia.org/T232528 (10Esanders) This happens when the selection starts in the slug above the list. Such a selection... [13:00:21] 10WikiEditor: Selection of text misses starting character(s) for complex scripts like Telugu - https://phabricator.wikimedia.org/T228611 (10Aklapper) After choosing the second input method from the screenshot in T228611#5470648 (see upper right corner of screenshot below) and following your steps (paste word; c... [13:01:44] 10WikiEditor: Selection of text misses starting character(s) for complex scripts like Telugu - https://phabricator.wikimedia.org/T228611 (10Aklapper) 05Open→03Stalled [13:57:03] 10VisualEditor, 10VisualEditor-ContentEditable, 10VisualEditor-CopyPaste: Pasting multiple list items in VisualEditor in Firefox results in incorrect display and saving - https://phabricator.wikimedia.org/T232528 (10Esanders) Indeed in this situation an exception is thrown on copy ` TypeError: startNode is n... [14:08:15] 10VisualEditor, 10Editing QA, 10I18n, 10Patch-For-Review, 10Readers-Web-Backlog (Readers-Web-Kanbanana-2019-20-Q1): In the Georgian Wikipedia, adding links in VisualEditor is broken because of incorrect capitalization - https://phabricator.wikimedia.org/T230559 (10Edtadros) a:05Edtadros→03ovasileva @... [14:13:43] 10VisualEditor: Potential API Errors Associated with VE on Meta-Wiki - https://phabricator.wikimedia.org/T232530 (10ifried) @Esanders Yes, it appeared to be a transient server issue. The issue was later not reproducible. Thanks. [14:35:02] 10Citoid: Citoid returns "HyperSwitch/errors/*" for all citations - https://phabricator.wikimedia.org/T233019 (10Urbanecm) [14:35:06] 10Citoid: Citoid returns "HyperSwitch/errors/*" for all citations - https://phabricator.wikimedia.org/T233019 (10Urbanecm) p:05Triage→03Unbreak! Since it breaks all citations AFAICS, triaging as UBN. [14:38:33] 10Citoid: Citoid returns "HyperSwitch/errors/*" for all citations - https://phabricator.wikimedia.org/T233019 (10mobrovac) 05Open→03Invalid p:05Unbreak!→03Normal The URL does not support query parameters. `curl https://en.wikipedia.org/api/rest_v1/data/citation/mediawiki/https%3A%2F%2Fwww.idnes.cz%2Fekon... [14:41:37] 10Citoid: Citoid returns "HyperSwitch/errors/*" for all citations - https://phabricator.wikimedia.org/T233019 (10Urbanecm) Well, I copied that URL from network tab, as it was requested by VisualEditor... Anyway, seems to work now. [15:35:21] (03CR) 10Esanders: [C: 04-1] "In which case add some inline comments, and we'll go with rem." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/536371 (https://phabricator.wikimedia.org/T222217) (owner: 10Bartosz Dziewoński) [15:48:39] (03PS3) 10DLynch: MWSaveDialog: add more instrumentation [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/535218 (https://phabricator.wikimedia.org/T229079) [16:23:16] 10VisualEditor-MediaWiki, 10Editing Design, 10Design, 10VisualEditor (Current work): Inconsistent "Edit" button label - https://phabricator.wikimedia.org/T206554 (10Volker_E) [16:28:17] (03PS4) 10Esanders: ve.ui.Surface: Fix check for active virtual keyboard in #adjustVisiblePadding [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/535253 (https://phabricator.wikimedia.org/T217769) (owner: 10Bartosz Dziewoński) [16:29:32] (03CR) 10Esanders: [C: 03+1] "Rebased, +2 if the rebase is clean." [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/535253 (https://phabricator.wikimedia.org/T217769) (owner: 10Bartosz Dziewoński) [16:32:06] (03PS2) 10Bartosz Dziewoński: Update VE core submodule to 509eeaa1 [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/536385 [16:32:44] (03CR) 10Bartosz Dziewoński: "Thanks for looking into them." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/536385 (owner: 10Bartosz Dziewoński) [16:33:42] (03CR) 10Bartosz Dziewoński: [C: 03+2] "I excluded the TreeModifier commits from this submodule update for now. I think we should only include them after the branch cut, since we" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/536385 (owner: 10Bartosz Dziewoński) [16:37:08] 10TemplateData, 10VisualEditor: Multiple parameters with same alias allowed - https://phabricator.wikimedia.org/T233034 (10Tacsipacsi) [16:55:19] (03CR) 10Bartosz Dziewoński: [C: 03+2] ve.ui.Surface: Fix check for active virtual keyboard in #adjustVisiblePadding [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/535253 (https://phabricator.wikimedia.org/T217769) (owner: 10Bartosz Dziewoński) [16:57:21] (03Merged) 10jenkins-bot: ve.ui.Surface: Fix check for active virtual keyboard in #adjustVisiblePadding [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/535253 (https://phabricator.wikimedia.org/T217769) (owner: 10Bartosz Dziewoński) [16:59:18] (03CR) 10jenkins-bot: ve.ui.Surface: Fix check for active virtual keyboard in #adjustVisiblePadding [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/535253 (https://phabricator.wikimedia.org/T217769) (owner: 10Bartosz Dziewoński) [17:01:42] (03CR) 10jenkins-bot: Localisation updates from https://translatewiki.net. [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/537153 (owner: 10L10n-bot) [17:20:47] 10VisualEditor, 10Editing QA, 10I18n, 10Patch-For-Review: In the Georgian Wikipedia, adding links in VisualEditor is broken because of incorrect capitalization - https://phabricator.wikimedia.org/T230559 (10Jdlrobson) Per T230559#5491229 untagging. Olga please bring it back to the sprint board if you need... [17:25:57] 10VisualEditor: 404 error when using VisualEditor: apierror-visualeditor-docserver-http - https://phabricator.wikimedia.org/T230272 (10kaldari) p:05Triage→03High Boldly marking this High priority since folks are losing edits, and, according to IndyJr, the bug happens "very frequently". [18:54:11] (03PS1) 10Bartosz Dziewoński: Update VE core submodule to master (a9d533280) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/537178 (https://phabricator.wikimedia.org/T215567) [18:55:05] (03CR) 10Bartosz Dziewoński: [C: 04-2] "To be merged after next branch cut. Also, we need to fix T232944 before the tests pass." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/537178 (https://phabricator.wikimedia.org/T215567) (owner: 10Bartosz Dziewoński) [18:58:42] (03CR) 10jenkins-bot: Update VE core submodule to 509eeaa1 [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/536385 (owner: 10Bartosz Dziewoński) [19:02:17] (03CR) 10jerkins-bot: [V: 04-1] Update VE core submodule to master (a9d533280) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/537178 (https://phabricator.wikimedia.org/T215567) (owner: 10Bartosz Dziewoński) [19:14:37] 10Citoid: List of urls unabled to cite - https://phabricator.wikimedia.org/T195080 (10Lofhi) https://www.lemonde.fr/economie/article/2019/09/16/michelin-confronte-a-des-problemes-de-productivite-lance-une-phase-de-dialogue-social_5511137_3234.html - too long [19:29:10] 10VisualEditor, 10Editing QA, 10I18n, 10Patch-For-Review: In the Georgian Wikipedia, adding links in VisualEditor is broken because of incorrect capitalization - https://phabricator.wikimedia.org/T230559 (10Amire80) I've just tested in production in the Georgian Wikipedia, and it appears to be fixed, but I... [19:35:24] 10VisualEditor, 10Editing QA, 10I18n, 10Patch-For-Review: In the Georgian Wikipedia, adding links in VisualEditor is broken because of incorrect capitalization - https://phabricator.wikimedia.org/T230559 (10Amire80) >>! In T230559#5496981, @Amire80 wrote: > I've just tested in production in the Georgian Wi... [19:35:55] 10VisualEditor, 10Editing QA, 10I18n, 10Patch-For-Review: In the Georgian Wikipedia, adding links in VisualEditor is broken because of incorrect capitalization - https://phabricator.wikimedia.org/T230559 (10Mehman97) Thanks to everyone, I tested and everything works :) [19:43:51] 10VisualEditor, 10VisualEditor-MediaWiki-Mobile, 10OOUI: Input field in link label dialog is not focused on iOS - https://phabricator.wikimedia.org/T230045 (10Volker_E) p:05Triage→03Lowest [20:47:53] (03PS2) 10Jforrester: build: Upgrade eslint-config-wikimedia from 0.13.1 to 0.14.0 [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/532790 [20:48:15] (03PS2) 10Jforrester: build: Upgrade grunt-banana-checker from 0.7.1 to 0.8.1 [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/532791 [21:40:02] 10VisualEditor, 10MobileFrontend: Mobile section editing no longer works (it always edits whole article) - https://phabricator.wikimedia.org/T233058 (10matmarex) [23:45:20] 10VisualEditor, 10MobileFrontend, 10Design (RW-Design-Debt), 10Patch-For-Review, 10Readers-Web-Backlog (Tracking): AbuseFilter warnings and errors are handled badly by the editor - https://phabricator.wikimedia.org/T202460 (10matmarex) a:03matmarex In the patch I'm proposing removing this workflow. The... [23:52:32] 10VisualEditor, 10MobileFrontend, 10Patch-For-Review, 10Readers-Web-Backlog (Tracking), 10Voice & Tone: Unhelpful error message: "Error, edit not saved." (mobile-frontend-editor-error) - https://phabricator.wikimedia.org/T228897 (10matmarex) I have not yet thoroughly tested the patch, there's a long list... [23:52:54] (03CR) 10VolkerE: [C: 03+2] build: Upgrade grunt-banana-checker from 0.7.1 to 0.8.1 [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/532791 (owner: 10Jforrester)