[00:00:28] <wikibugs>	 10Collaboration-Team-Triage, 10WikiLove, 10MassMessage, 10MediaWiki-extensions-MultimediaViewer, and 2 others: GSoC 2018 Proposal:Convert MediaWiki extensions WikiLove, MultimediaViewer and MassMessage to use OOUI - https://phabricator.wikimedia.org/T190764#4083067 (10ssaluja16)
[00:05:34] <Volker_E>	 mooeypoo: RoanKattouw: any specific reason for output of SortedListWidget with labelWidgets featuring <label> elements and PageFilterWidget with <div>s instead of, let's say <ol> & <li>?
[00:53:20] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10Maps, 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Patch-For-Review: Kartotherian is sending wrong etags and last-modified headers - https://phabricator.wikimedia.org/T187300#4083205 (10Etonkovidova) 05Open>03Resolved
[00:59:06] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Maps (Kartotherian), 10Patch-For-Review: Get kartotherian/kartotherian@master working with its dependencies (kartotherian/*) @master - https://phabricator.wikimedia.org/T187411#4083229 (10Etonkov...
[00:59:13] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10Maps, 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Patch-For-Review: Get kartotherian/tilerator@master working with its dependencies (kartotherian/*) @master - https://phabricator.wikimedia.org/T189049#4083230 (10Etonkovido...
[03:13:37] <wikibugs>	 (03PS3) 10VolkerE: Align Notifications popup header/footer with OOUI's default size [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421840 (https://phabricator.wikimedia.org/T190397)
[03:14:11] <wikibugs>	 (03CR) 10VolkerE: Align Notifications popup header/footer with OOUI's default size (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421840 (https://phabricator.wikimedia.org/T190397) (owner: 10VolkerE)
[03:15:58] <wikibugs>	 (03CR) 10VolkerE: "PS 3 unifies comment styles, addresses raised questions in PS2 and removes a distance glitch in “Mark all read” button." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421840 (https://phabricator.wikimedia.org/T190397) (owner: 10VolkerE)
[09:23:09] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10StructuredDiscussions, 10MW-1.31-release-notes (WMF-deploy-2018-03-27 (1.31.0-wmf.27)), 10Patch-For-Review, 10Wikimedia-log-errors (Jenkins Failure): Flow\Tests\Model\UUIDTest::invalidInputProvider... - https://phabricator.wikimedia.org/T170003#3416069
[09:46:37] <wikibugs>	 10Collaboration-Team-Triage, 10Wikimedia-Extension-setup, 10Wikimedia-Site-requests, 10Patch-For-Review, and 2 others: Enable Extension:StructuredDiscussions in Basque Wikipedia - https://phabricator.wikimedia.org/T190500#4083725 (10Trizek-WMF) >>! In T190500#4080987, @Theklan wrote: > @Trizek-WMF: here: h...
[09:47:25] <wikibugs>	 10Collaboration-Team-Triage, 10Collaboration-Community-Engagement, 10Community-Liaisons, 10StructuredDiscussions, and 2 others: Enable StructuredDiscussions on wikis that require it - https://phabricator.wikimedia.org/T117990#4083727 (10Trizek-WMF)
[09:47:27] <wikibugs>	 10Collaboration-Team-Triage, 10Wikimedia-Extension-setup, 10Wikimedia-Site-requests, 10Patch-For-Review, and 2 others: Enable Extension:StructuredDiscussions in Basque Wikipedia - https://phabricator.wikimedia.org/T190500#4083726 (10Trizek-WMF)
[10:26:20] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Maps (Kartotherian): Kartotherian should accept (and propagate) a language parameter for tiles - https://phabricator.wikimedia.org/T187596#4083906 (10SBisson) >>! In T187596#4082950, @Etonkovidova...
[10:28:53] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10StructuredDiscussions, 10MW-1.31-release-notes (WMF-deploy-2018-03-27 (1.31.0-wmf.27)), 10Patch-For-Review, 10Wikimedia-log-errors (Jenkins Failure): Flow\Tests\Model\UUIDTest::invalidInputProvider... - https://phabricator.wikimedia.org/T170003#4083910
[10:45:02] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10I18n, 10Maps (Map-Styles), 10Patch-For-Review: All map location names should be in the user's language - https://phabricator.wikimedia.org/T112948#4083967 (10Mooeypoo) >>! In T112948#4082448,...
[10:45:43] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10I18n, 10Maps (Map-Styles), 10Patch-For-Review: All map location names should be in the user's language - https://phabricator.wikimedia.org/T112948#4083969 (10SBisson) >>! In T112948#4082448, @...
[10:57:48] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Edit-Review-Improvements-Integrated-Filters, 10MediaWiki-User-preferences, 10MW-1.31-release-notes (WMF-deploy-2018-03-27 (1.31.0-wmf.27)), and 3 others: Hide preference for 'Reload the watchlist aut... - https://phabricator.wikimedia.org/T186277#4084011
[10:59:01] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10StructuredDiscussions, 10MW-1.31-release-notes (WMF-deploy-2018-03-27 (1.31.0-wmf.27)), 10Patch-For-Review, 10Wikimedia-log-errors (Jenkins Failure): Flow\Tests\Model\UUIDTest::invalidInputProvider... - https://phabricator.wikimedia.org/T170003#4084025
[11:06:23] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10I18n, 10Maps (Map-Styles), 10Patch-For-Review: All map location names should be in the user's language - https://phabricator.wikimedia.org/T112948#4084041 (10Trizek-WMF) >>! In T112948#4083969...
[11:28:36] <wikibugs>	 (03CR) 10Esanders: Fix DOM structure of mw.echo.ui.MenuItemWidget (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421722 (owner: 10Esanders)
[11:30:35] <wikibugs>	 (03CR) 10Esanders: Fix DOM structure of mw.echo.ui.MenuItemWidget (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421722 (owner: 10Esanders)
[11:32:56] <wikibugs>	 (03PS4) 10Esanders: Fix DOM structure of mw.echo.ui.MenuItemWidget [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421722
[11:33:46] <wikibugs>	 (03PS5) 10Esanders: Fix DOM structure of mw.echo.ui.MenuItemWidget [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421722
[11:34:16] <wikibugs>	 (03CR) 10Bartosz Dziewoński: Fix DOM structure of mw.echo.ui.MenuItemWidget (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421722 (owner: 10Esanders)
[11:34:52] <wikibugs>	 (03CR) 10Bartosz Dziewoński: Fix DOM structure of mw.echo.ui.MenuItemWidget (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421722 (owner: 10Esanders)
[12:14:15] <wikibugs>	 (03PS1) 10simetrical: API parameters with defaults cannot be required [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422129
[12:41:36] <stephanebisson>	 DanielK_WMDE_: re: failing Flow test, I can't get that test to fail locally on hhvm with your patch. As for what those tests are doing, they are checking, at the api level, that the cache is invalidated on POST and repopulated from replica on GET. The specific assertion failure is because flowCache.set() is called during a 'edit-header' POST action. 
[12:42:12] <stephanebisson>	 DanielK_WMDE_: I'll submit a diagnostic patch shortly. It's the only next step I can think of.
[12:45:16] <wikibugs>	 (03PS1) 10Sbisson: [DO NOT MERGE] Adding diagnostic code to find out why flowCache.set() is being called [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422139
[12:45:53] <wikibugs>	 (03CR) 10jerkins-bot: [V: 04-1] [DO NOT MERGE] Adding diagnostic code to find out why flowCache.set() is being called [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422139 (owner: 10Sbisson)
[12:46:09] <wikibugs>	 (03CR) 10jerkins-bot: [V: 04-1] [DO NOT MERGE] Adding diagnostic code to find out why flowCache.set() is being called [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422139 (owner: 10Sbisson)
[12:49:09] <stephanebisson>	 DanielK_WMDE_: I uploaded a Flow diagnostic patch (see ^) that Depends-On your Core patch with the hope that the Flow test will fail but will give us a stack trace. I think you need to rebase your patch so it can merge cleanly for Jenkins to run on mine. (I'll be back in ~1h)
[13:04:37] <wikibugs>	 10Collaboration-Team-Triage, 10MediaWiki-User-preferences, 10Easy, 10Patch-For-Review: Create a "Watching and notifications" tab in MediaWiki core - https://phabricator.wikimedia.org/T65577#4084304 (10Aklapper) No reply by @Zoranzoki21 hence reverting.
[13:07:18] <DanielK_WMDE_>	 stephanebisson: ...but there already is a stack trace?...
[13:09:10] <wikibugs>	 (03CR) 10Daniel Kinzler: "recheck" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422139 (owner: 10Sbisson)
[13:16:34] <wikibugs>	 (03CR) 10jerkins-bot: [V: 04-1] [DO NOT MERGE] Adding diagnostic code to find out why flowCache.set() is being called [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422139 (owner: 10Sbisson)
[13:24:52] <wikibugs>	 (03Abandoned) 10Sbisson: [DO NOT MERGE] Adding diagnostic code to find out why flowCache.set() is being called [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422139 (owner: 10Sbisson)
[13:27:14] <stephanebisson>	 DanielK_WMDE_: duh, yes, there is a stacktrace, I'll look into it now. Any change in your patch that would affect the timing of DeferredUpdates?
[13:59:00] <DanielK_WMDE_>	 stephanebisson: it should not. 
[13:59:36] <DanielK_WMDE_>	 but it's possible.
[13:59:59] <DanielK_WMDE_>	 i did change the error handling in DeferredUpdates, to no longer silently ignore exceptions. 
[14:01:08] <DanielK_WMDE_>	 this could cause some code to not be executed. i'd expect to see an error, but if the exception is again caught and logged later, that may cause inconsistencies.
[14:01:57] <wikibugs>	 (03CR) 10Catrope: [C: 032] API parameters with defaults cannot be required [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422129 (owner: 10simetrical)
[14:03:04] <wikibugs>	 (03CR) 10Catrope: [C: 032] Revert my fix for summary pings in reverts [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422078 (https://phabricator.wikimedia.org/T190374) (owner: 10MaxSem)
[14:04:39] <wikibugs>	 (03CR) 10Catrope: [C: 032] Detect reverts differently [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422079 (https://phabricator.wikimedia.org/T189819) (owner: 10MaxSem)
[14:04:56] <DanielK_WMDE_>	 stephanebisson: huh... commenting out the relevant line that re-throws the exception in DeferredUpdates causes the failure message to change (and the stack trace to become nonsensical). This may indicate that the failure was there all along, but was betting swallowed by the exception handling in DeferredUpdates.
[14:05:35] <stephanebisson>	 DanielK_WMDE_: does it fail locally for you?
[14:05:39] <DanielK_WMDE_>	 stephanebisson: find if ( defined( 'MW_PHPUNIT_TEST' ) ) in DeferredUpdates.php in my patch. comment out the block.
[14:05:56] <DanielK_WMDE_>	 yes, it fails, with and without the hack in DeferredUpdates, btu in different ways
[14:06:48] <DanielK_WMDE_>	 without the re-throw hack, I get: https://phabricator.wikimedia.org/P6903
[14:07:18] <DanielK_WMDE_>	 that failrue message is awkward. i'm not sure what it actualyl indicates.
[14:08:18] <stephanebisson>	 That's really weird but it means the same thing: set() was called ut it was not supposed to
[14:08:50] <DanielK_WMDE_>	 stephanebisson: perhaps this is a problem with how the mock is created? https://github.com/sebastianbergmann/phpunit/issues/269
[14:10:27] <stephanebisson>	 One way or another, it doesn't fail for me locally. Actually none of the deferred stuff even run. Any idea what would cause that?
[14:11:25] <DanielK_WMDE_>	 no... the deferred stuff must run, otherwise none of the links tables would get updated. that would also cause some tests to fails
[14:11:59] <DanielK_WMDE_>	 so, wehn i revert core to master, put with the patched DeferredUpdates code, the test still passes.
[14:12:15] <DanielK_WMDE_>	 so my patch causes the failure, it doesn't just make it visible
[14:12:47] <DanielK_WMDE_>	 stephanebisson: check whether DeferredUpdates::addUpdate gets called
[14:13:13] <DanielK_WMDE_>	 stephanebisson: how are you running the unit test? I'm using 
[14:13:21] <DanielK_WMDE_>	 /usr/bin/php tests/phpunit/phpunit.php extensions/Flow/tests/phpunit/api/ApiFlowEditHeaderTest.php
[14:14:14] <stephanebisson>	 AddUpdates gets called a bunch of times but I didn't check the arguments
[14:14:25] <stephanebisson>	 I'm running using: sudo -u www-data hhvm tests/phpunit/phpunit.php --wiki wiki extensions/Flow/tests/phpunit/api/ApiFlowEditHeaderTest.php --filter=testCache
[14:14:33] <stephanebisson>	 from inside mw-vagrant
[14:15:46] <wikibugs>	 (03Merged) 10jenkins-bot: API parameters with defaults cannot be required [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422129 (owner: 10simetrical)
[14:15:51] <wikibugs>	 (03Merged) 10jenkins-bot: Revert my fix for summary pings in reverts [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422078 (https://phabricator.wikimedia.org/T190374) (owner: 10MaxSem)
[14:16:04] <stephanebisson>	 This is the output when I use the same command as you do:
[14:16:05] <stephanebisson>	 PHP Fatal error:  Uncaught UnexpectedValueException: There is no existing directory at "/vagrant/logs/mediawiki-extensions/Flow/tests/phpunit/api" and its not buildable: Permission denied in /vagrant/mediawiki/vendor/monolog/monolog/src/Monolog/Handler/StreamHandler.php:171
[14:16:39] <DanielK_WMDE_>	 stephanebisson: in the context of unit tests, addUpdate should be immidiately executing the job
[14:16:45] <DanielK_WMDE_>	 if it isn't, something is wrong
[14:17:25] <stephanebisson>	 I've added $this->assertEquals( 0, \DeferredUpdates::pendingUpdatesCount() ); at the end of the test and it still passes
[14:17:50] <DanielK_WMDE_>	 sounds liek the deferred stuff *is* running, then
[14:18:11] <stephanebisson>	 must be, time to debug into it
[14:19:23] <DanielK_WMDE_>	 so one of the things that may have changed is how null-edits are detected. maybe that's the issue, for some reason? Flow does some strange things with the Content object
[14:20:41] <stephanebisson>	 You can easily change the test to avoid null-edits. Add a random string to 'ehcontent'.
[14:22:46] <DanielK_WMDE_>	 doesn't make a difference
[14:23:24] <DanielK_WMDE_>	 stephanebisson: can you explain to me what exactly is being tested here, and what expectations are asserted, and how that assertion fails?
[14:23:34] <wikibugs>	 (03Merged) 10jenkins-bot: Detect reverts differently [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422079 (https://phabricator.wikimedia.org/T189819) (owner: 10MaxSem)
[14:24:02] <stephanebisson>	 DanielK_WMDE_: It tests that a POST doesn't trigger a flowCache.set(). 
[14:24:22] <DanielK_WMDE_>	 but why shouldn't it?
[14:26:58] <wikibugs>	 (03CR) 10jenkins-bot: API parameters with defaults cannot be required [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422129 (owner: 10simetrical)
[14:27:47] <DanielK_WMDE_>	 stephanebisson: i can't even tell what FeatureIndex is used for, or what the things in the cache are, or what $queries are in this context... it's all a bit opaque to me :)
[14:28:50] <stephanebisson>	 it is for all of us, be back in a few minutes
[14:30:01] <wikibugs>	 (03CR) 10jenkins-bot: Detect reverts differently [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422079 (https://phabricator.wikimedia.org/T189819) (owner: 10MaxSem)
[14:32:32] <wikibugs>	 (03CR) 10jenkins-bot: Revert my fix for summary pings in reverts [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422078 (https://phabricator.wikimedia.org/T190374) (owner: 10MaxSem)
[15:02:32] <DanielK_WMDE_>	 stephanebisson: ok, found the cuplrit. 
[15:03:36] <DanielK_WMDE_>	 stephanebisson: the new code tries harder to avoid generating HTML until it's actually needed, and uses more aggressive caching. This causes BoardContent::getParserOutput to be called later, and only once instead of twice. That causes the problem.
[15:04:08] <wikibugs>	 (03PS1) 10Esanders: Prevent FOUC when loading OOUI notification badges [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422166
[15:04:33] <DanielK_WMDE_>	 I heave not figured out the details, but conceptually, the issue is that getParserOutput should not have side-effects, but it does.
[15:05:53] <DanielK_WMDE_>	 I suppose because it loads the actualy content from the database, via some caching layer
[15:06:19] <DanielK_WMDE_>	 I suppose that's an excellent reason to make use of more aggressive caching :)
[15:06:49] <DanielK_WMDE_>	 stephanebisson: how do you want to track this for a follow-up? 
[15:09:25] <stephanebisson>	 DanielK_WMDE_: I don't quite understand why calling it later causes this problem.
[15:11:31] <DanielK_WMDE_>	 stephanebisson: me neither. well, it'S not really called later. it's called once instead of twice. callign it twice may be correct or it may be duplicate effort. and the problem may be real, or an artifact of the test setup. I did not investigate further
[15:11:45] <DanielK_WMDE_>	 for now, i have a workaround.
[15:11:53] <stephanebisson>	 I also don't understand why it doesn't fail for me locally, and I'm questioning whether the test makes sense in the first place given that all deferred updates and executed synchronously... so many questions
[15:12:34] <DanielK_WMDE_>	 deferred updates are executed inline during unit testing (or rather, always in command line mode)
[15:12:49] <DanielK_WMDE_>	 but whether the texst makes sense is a question for you to answer - after all, you wrote it :)
[15:13:45] <stephanebisson>	 In Flow, git blame gives you only a fraction of the picture ;)
[15:14:38] <stephanebisson>	 DanielK_WMDE_: Honestly, there's very low probability that we'll get back to it. If you think calling it once is better in MW, and we can see it with some other tests, we should compromise on the Flow side.
[15:17:34] <DanielK_WMDE_>	 stephanebisson: i'll see if i can understand a bit better why it was called twice 
[15:31:02] <DanielK_WMDE_>	 it's all rather confusing. and related to another patch I'm working on. will need to revisit this later
[15:39:06] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Edit-Review-Improvements-Integrated-Filters, 10MediaWiki-User-preferences, 10MW-1.31-release-notes (WMF-deploy-2018-03-27 (1.31.0-wmf.27)), and 3 others: Hide preference for 'Reload the watchlist aut... - https://phabricator.wikimedia.org/T186277#4084974
[16:40:23] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Maps (Kartotherian): Kartotherian should accept (and propagate) a language parameter for tiles - https://phabricator.wikimedia.org/T187596#4085277 (10Etonkovidova) 05Open>03Resolved
[16:40:32] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10I18n, 10Maps (Map-Styles), 10Patch-For-Review: All map location names should be in the user's language - https://phabricator.wikimedia.org/T112948#4085278 (10Etonkovidova)
[17:46:31] <wikibugs>	 10Collaboration-Team-Triage, 10Thanks, 10MediaWiki-History-or-Diffs, 10I18n: Thanking messages don’t check gender on Special:History - https://phabricator.wikimedia.org/T190563#4085600 (10Catrope) a:03Catrope I can reproduce this, I'll work on fixing this shortly.
[17:47:18] <wikibugs>	 10Collaboration-Team-Triage, 10Wikimedia-Extension-setup, 10Wikimedia-Site-requests, 10Patch-For-Review, and 2 others: Enable Extension:StructuredDiscussions in Basque Wikipedia - https://phabricator.wikimedia.org/T190500#4085608 (10Catrope)  @Urbanecm The patch you wrote is OK to deploy. I've [[https://wi...
[17:52:19] <wikibugs>	 10Collaboration-Team-Triage, 10StructuredDiscussions: Numbered external link not showing properly in Structured Discussion diff - https://phabricator.wikimedia.org/T190466#4074376 (10Catrope) It's because the `mw-parser-output` class is missing in the diff output.
[18:10:01] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Maps-Sprint, 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Maps (Kartotherian), 10Security: Kartotherian ReDoS vulnerability - https://phabricator.wikimedia.org/T176168#4085751 (10Catrope)
[18:18:06] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10Maps, 10Maps-Sprint, and 3 others: Decide on Cache-Control headers for map tiles - https://phabricator.wikimedia.org/T186732#4085808 (10Pnorman) a:03Pnorman Going to take another look at a couple of things here
[18:25:11] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Maps (Kartographer): Investigate T141304 and, at minimum, document issue - https://phabricator.wikimedia.org/T187963#4085828 (10jmatazzoni)
[18:26:05] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Maps (Kartographer): Investigate T141304 and, at minimum, document issue - https://phabricator.wikimedia.org/T187963#3991430 (10jmatazzoni) We added a link to the actual maki symbol...
[18:28:16] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10TCB-Team, 10Collaboration-Feature-Rollouts (Collaboration-Maps), and 2 others: Investigate how <mapframe/link> work with the Flagged Revisions extension - https://phabricator.wikimedia.org/T151665#4085839 (10Catrope) >>! In T151665#4045...
[19:06:19] <wikibugs>	 10Collaboration-Team-Triage, 10Wikimedia-Extension-setup, 10Wikimedia-Site-requests, 10Patch-For-Review, and 2 others: Enable Extension:StructuredDiscussions in Basque Wikipedia - https://phabricator.wikimedia.org/T190500#4085982 (10Trizek-WMF) Before deploying it effectively, please wait to have more inpu...
[19:07:26] <wikibugs>	 10Collaboration-Team-Triage, 10StructuredDiscussions, 10Performance: Saving a new topic can take >5s on beta labs - https://phabricator.wikimedia.org/T73476#4085990 (10Krinkle) 05Open>03declined
[19:20:13] <wikibugs>	 (03CR) 10Mooeypoo: [C: 032] "This is great. Thanks!" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421722 (owner: 10Esanders)
[19:25:49] <wikibugs>	 (03CR) 10Catrope: [C: 032] Prevent FOUC when loading OOUI notification badges [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422166 (owner: 10Esanders)
[19:32:16] <wikibugs>	 (03Merged) 10jenkins-bot: Fix DOM structure of mw.echo.ui.MenuItemWidget [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421722 (owner: 10Esanders)
[19:35:40] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Maps-Sprint, 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10I18n, 10Maps (Kartographer): Maps must link multilingual terms of use - https://phabricator.wikimedia.org/T169071#4086069 (10jmatazzoni)
[19:38:36] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Maps (Kartographer): Investigate T141304 and, at minimum, document issue - https://phabricator.wikimedia.org/T187963#4086075 (10CKoerner_WMF) I'm working on updating Map documentati...
[19:58:38] <wikibugs>	 (03Merged) 10jenkins-bot: Prevent FOUC when loading OOUI notification badges [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422166 (owner: 10Esanders)
[20:06:38] <wikibugs>	 (03CR) 10jenkins-bot: Fix DOM structure of mw.echo.ui.MenuItemWidget [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421722 (owner: 10Esanders)
[20:22:05] <wikibugs>	 (03CR) 10jenkins-bot: Prevent FOUC when loading OOUI notification badges [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422166 (owner: 10Esanders)
[20:44:29] <wikibugs>	 (03CR) 10jenkins-bot: Localisation updates from https://translatewiki.net. [extensions/Echo] - 10https://gerrit.wikimedia.org/r/422289 (owner: 10L10n-bot)
[20:48:39] <wikibugs>	 (03CR) 10Catrope: [C: 032] NotificationsOptionWidget: Fix icon alignment [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421841 (owner: 10VolkerE)
[20:49:54] <wikibugs>	 (03CR) 10jenkins-bot: Localisation updates from https://translatewiki.net. [extensions/Flow] - 10https://gerrit.wikimedia.org/r/422292 (owner: 10L10n-bot)
[20:51:53] <wikibugs>	 (03CR) 10Catrope: [C: 032] Align Notifications popup header/footer with OOUI's default size [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421840 (https://phabricator.wikimedia.org/T190397) (owner: 10VolkerE)
[21:00:26] <wikibugs>	 (03Merged) 10jenkins-bot: NotificationsOptionWidget: Fix icon alignment [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421841 (owner: 10VolkerE)
[21:03:38] <wikibugs>	 (03Merged) 10jenkins-bot: Align Notifications popup header/footer with OOUI's default size [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421840 (https://phabricator.wikimedia.org/T190397) (owner: 10VolkerE)
[21:04:33] <wikibugs>	 10Collaboration-Team-Triage, 10Notifications, 10Community-Tech, 10MediaWiki-Page-editing, and 4 others: [Epic] Ping/notify user when username used in an edit summary - https://phabricator.wikimedia.org/T32750#4086438 (10TBolliger)
[21:05:35] <wikibugs>	 10Collaboration-Team-Triage, 10Notifications, 10Community-Tech, 10MediaWiki-Page-editing, and 4 others: [Epic] Ping/notify user when username used in an edit summary - https://phabricator.wikimedia.org/T32750#3998400 (10TBolliger)
[21:05:42] <wikibugs>	 10Collaboration-Team-Triage, 10Notifications, 10Community-Tech-Sprint, 10MW-1.31-release-notes (WMF-deploy-2018-03-20 (1.31.0-wmf.26)), 10Patch-For-Review: Revert to last revision by user should not mention - https://phabricator.wikimedia.org/T189819#4086440 (10TBolliger) 05Open>03Resolved
[21:06:43] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10MW-1.31-release-notes (WMF-deploy-2018-03-20 (1.31.0-wmf.26)), and 2 others: <maplink> and/or <mapframe>: clickable marker area too... - https://phabricator.wikimedia.org/T141750#4086447
[21:17:44] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10I18n, 10Maps (Map-Styles), 10Patch-For-Review: All map location names should be in the user's language - https://phabricator.wikimedia.org/T112948#4086457 (10Etonkovidova)
[21:17:46] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10I18n, 10MW-1.31-release-notes (WMF-deploy-2018-03-20 (1.31.0-wmf.26)), and 2 others: Pass MW user language to Kartotherian from Kartographer - https://phabricator.wikimedia.org/T187598#4086456 (1...
[21:19:41] <wikibugs>	 (03CR) 10jenkins-bot: Align Notifications popup header/footer with OOUI's default size [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421840 (https://phabricator.wikimedia.org/T190397) (owner: 10VolkerE)
[21:22:52] <wikibugs>	 (03CR) 10jenkins-bot: NotificationsOptionWidget: Fix icon alignment [extensions/Echo] - 10https://gerrit.wikimedia.org/r/421841 (owner: 10VolkerE)
[21:42:51] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10Collaboration-Feature-Rollouts (Collaboration-Maps), 10Maps (Kartographer): Investigate T141304 and, at minimum, document issue - https://phabricator.wikimedia.org/T187963#4086507 (10jmatazzoni)  In T187963#4086075, @CKoerner_WMF wrote:...
[21:43:13] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10StructuredDiscussions, 10MW-1.31-release-notes (WMF-deploy-2018-03-27 (1.31.0-wmf.27)), 10Patch-For-Review, 10Wikimedia-log-errors (Jenkins Failure): Flow\Tests\Model\UUIDTest::invalidInputProvider... - https://phabricator.wikimedia.org/T170003#4086508
[22:30:39] <wikibugs>	 10Collaboration-Team-Triage (Collab-Team-This-Quarter), 10Discovery, 10Maps, 10Collaboration-Feature-Rollouts (Collaboration-Maps): Consult with English Wikipedia and re. whether they still want mapframe (given limitations) - https://phabricator.wikimedia.org/T187931#4086612 (10jmatazzoni) @CKoerner_WMF, i...
[23:24:10] <wikibugs>	 10Collaboration-Team-Triage, 10Notifications, 10Community-Tech, 10MediaWiki-Page-editing, and 4 others: [Epic] Ping/notify user when username used in an edit summary - https://phabricator.wikimedia.org/T32750#4086752 (10TBolliger)
[23:24:16] <wikibugs>	 10Collaboration-Team-Triage, 10Notifications, 10MediaWiki-Page-editing: Ping user in edit summary doesn't work when the username includes a & - https://phabricator.wikimedia.org/T190491#4086750 (10TBolliger) 05Open>03Invalid Cannot reproduce here: https://test.wikipedia.org/w/index.php?title=Page772&oldi...