[00:02:43] 06Wikipedia-Android-App-Backlog, 13Patch-For-Review, 07Unplanned-Sprint-Work: [Dev] Upgrade to Support Library v24.0.0+ - https://phabricator.wikimedia.org/T138105#2578001 (10Dzahn) [00:07:19] (03CR) 10Catrope: [C: 032] EditPage: Show a different label for the button on create vs. modify [core] - 10https://gerrit.wikimedia.org/r/296718 (https://phabricator.wikimedia.org/T139033) (owner: 10Jforrester) [00:09:35] (03CR) 10Catrope: [C: 032] EditPage: Allow the 'save' button's label to be 'publish' for public wikis [core] - 10https://gerrit.wikimedia.org/r/296719 (https://phabricator.wikimedia.org/T131132) (owner: 10Jforrester) [00:13:40] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 07Wikimedia-log-errors: UnresolvedRedirectException in EntityAccessor::getEntity (scribunto) - https://phabricator.wikimedia.org/T143756#2578010 (10Dereckson) To fix the double redirect, an API call to wbcreateredirect, with from at Q24668012 and to at Q1972... [00:14:39] (03Merged) 10jenkins-bot: EditPage: Show a different label for the button on create vs. modify [core] - 10https://gerrit.wikimedia.org/r/296718 (https://phabricator.wikimedia.org/T139033) (owner: 10Jforrester) [00:16:21] (03PS1) 10Aaron Schulz: Make RedisBagOStuff respect SYNC_WRITE [core] - 10https://gerrit.wikimedia.org/r/306312 [00:17:39] (03Merged) 10jenkins-bot: EditPage: Allow the 'save' button's label to be 'publish' for public wikis [core] - 10https://gerrit.wikimedia.org/r/296719 (https://phabricator.wikimedia.org/T131132) (owner: 10Jforrester) [00:20:45] (03PS1) 10Jforrester: RELEASE-NOTES: Add release note for 74fa607 [core] - 10https://gerrit.wikimedia.org/r/306313 [00:25:22] (03PS1) 10Kaldari: [WIP] Refactoring job queue logic for PageAssessments [extensions/PageAssessments] - 10https://gerrit.wikimedia.org/r/306314 [00:26:29] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Refactoring job queue logic for PageAssessments [extensions/PageAssessments] - 10https://gerrit.wikimedia.org/r/306314 (owner: 10Kaldari) [00:27:38] 10MediaWiki-extensions-DonationInterface, 10FR-ActiveMQ, 10FR-Smashpig, 10Fundraising-Backlog, and 5 others: Run the orphan rectifier job from CRM Jenkins - https://phabricator.wikimedia.org/T141487#2578014 (10awight) The new production config patch is on branch `awight-orphanage3` [00:40:06] (03PS10) 10Aaron Schulz: Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds [core] - 10https://gerrit.wikimedia.org/r/305952 [00:40:08] Krinkle: ^ [00:43:08] 10MediaWiki-ContentHandler, 10MediaWiki-Page-editing: Fix Content validation - something to replace Content::isValid() - https://phabricator.wikimedia.org/T143761#2578028 (10Legoktm) [00:43:59] Quick question, anyone: when does Mediawiki:Common.js execute? Is it guaranteed after all modules are loaded? [00:44:04] (03PS2) 10Aaron Schulz: Make RedisBagOStuff respect SYNC_WRITE [core] - 10https://gerrit.wikimedia.org/r/306312 [00:45:05] (03CR) 10jenkins-bot: [V: 04-1] Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds [core] - 10https://gerrit.wikimedia.org/r/305952 (owner: 10Aaron Schulz) [00:45:26] Krinkle: ^ ? [00:46:16] AndyRussG: It executes in the bottom queue. It doesn't have any dependencies, so there's no reason for its execution to be delayed. [00:46:43] (03PS3) 10Aaron Schulz: Make RedisBagOStuff respect SYNC_WRITE [core] - 10https://gerrit.wikimedia.org/r/306312 [00:48:33] AndyRussG: Stuff in Common.js that needs other modules will use mw.loader.using(). Stuff in your code that wants to allow static modification by Common.js (E.g. VisualEditor allows Common.js to register plugins), then you can wait for mw.loader.using('site').always(callback) [00:48:53] Krinkle: ah yes of course [00:49:04] K that'll work [00:49:17] (03PS11) 10Aaron Schulz: Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds [core] - 10https://gerrit.wikimedia.org/r/305952 [00:49:26] use .always() instead of .done() or .then() since you don't want to abort your code in case of local wiki errors (e.g. syntax error in Common.js) [00:49:28] AndyRussG: [00:49:40] (FYI this is just to provide a hackey option to 3rd party users following the removal of JSONP GeoIP lookup fallback.) [00:52:09] Krinkle: K thanks! [00:53:10] (03CR) 10Krinkle: [C: 031] "Makes sense :) This still adds the behaviour of automatically undoing flag changes after commit(), though. But now limited to flag changes" [core] - 10https://gerrit.wikimedia.org/r/305952 (owner: 10Aaron Schulz) [00:55:49] 10SyntaxHighlight: SyntaxHighlight fails to highlight code and adds 'Pages with syntax highlighting errors' category to page - https://phabricator.wikimedia.org/T133371#2578043 (10Donald.nicolson89) >>! In T133371#2234261, @Aklapper wrote: > Thanbks for investigating! **Any** external version, or a more recent o... [00:57:09] Krinkle: getting closer... [00:57:26] it should only restore if did the set [00:57:34] * AaronSchulz tweaks [00:57:40] (03PS8) 10Krinkle: [WIP] resourceloader: Make cache-eval in mw.loader.work asynchronous [core] - 10https://gerrit.wikimedia.org/r/304158 (https://phabricator.wikimedia.org/T142129) [00:58:30] (03PS1) 10Catrope: Factor out Cache::$classMap into $wgOresModelClasses [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306316 [01:00:05] 06Discovery, 10Wikimedia-Portals, 06Discovery-Analysis (Current work): Wikipedia.org Portal Visitors' Session Lengths (Redux) - https://phabricator.wikimedia.org/T143137#2578044 (10chelsyx) {F4395103} I put the investigation on os and browsers in a second report. [01:03:08] (03PS9) 10Krinkle: [WIP] resourceloader: Make cache-eval in mw.loader.work asynchronous [core] - 10https://gerrit.wikimedia.org/r/304158 (https://phabricator.wikimedia.org/T142129) [01:03:59] duh, onTransactionResolution() [01:04:03] much easier [01:05:46] Krinkle: mmm another quesiton... Would you say Common.js is the correct config-y place in which 3rd party users should add bits of one-off JS they may require? The use case is, we're removing our geoiplookup endpoint, but some 3rd-party users might want to be able to set window.Geo using another service, and have other stuff that worked with GeoIP still work. There are several JSONP services they could [01:05:48] use, and they return the data in different formats. For such cases, I'm thinking of providing a simple way for them to put stuff in Mediawiki:Common.js [01:06:29] (and have the GeoIP promise work as before) [01:07:11] AndyRussG: I'd avoid Common.js in that case since this isn't content-admin customisation but site-admin customisation. Imagine ourselves or Wikia, not in Common.js on every site. [01:08:30] AndyRussG: Maybe something like $wgCentralNoticeGeoIpModule=string|null, which is set to a module name that you'll wait for to call a particular hook in CN js code. [01:08:50] Then the 3rd party would register their script as module and assign its name there. [01:09:03] If they want to do it via common.js, they can assign it 'site' [01:09:57] Or, more likely, they can create a litlte JS file on disk, register it via $wgResourceModules[..] and set $wgCentralNoticeGeoIpModule [01:10:20] (03PS12) 10Aaron Schulz: Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds [core] - 10https://gerrit.wikimedia.org/r/305952 [01:10:53] (03CR) 10Krinkle: Factor out Cache::$classMap into $wgOresModelClasses (031 comment) [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306316 (owner: 10Catrope) [01:10:57] Krinkle: does that make sense yet? :) [01:11:17] AaronSchulz: Yes [01:12:10] AaronSchulz: doc comment [01:12:15] of beginMasterChanges [01:12:19] also I forgot the REMEMBER_PRIOR in that other patch, aside from always doing restore on commit [01:14:23] Krinkle: K makes sense, thx again!! [01:16:32] 06Discovery-Analysis (Current work), 07Epic: [EPIC] Learn about our databases and how to use them - https://phabricator.wikimedia.org/T143128#2578068 (10mpopov) [01:17:29] Krinkle: good catch on that, I wasn't thinking about other callers using those flags at the same time [01:30:14] (03CR) 10Aaron Schulz: "OK, now it should restore to the flags before beginMasterChanges(), and not affect commitMasterChanges() cases where that method isn't cal" [core] - 10https://gerrit.wikimedia.org/r/305952 (owner: 10Aaron Schulz) [01:34:49] Krinkle: what about the doc comment? [01:35:04] AaronSchulz: it mentions where it gets removed still [01:35:38] well it gets reverted on commit/rollback, which will happen with those methods [01:38:04] Krinkle: the comment seems fine then, right? [01:56:33] (03CR) 10小璋丸: "Thanks" [extensions/QuickGV] - 10https://gerrit.wikimedia.org/r/303991 (owner: 10小璋丸) [01:57:47] (03PS1) 10Legoktm: EditPage: Add getContext() function [core] - 10https://gerrit.wikimedia.org/r/306317 [01:57:51] (03PS1) 10Legoktm: EditPage: Fix some doc blocks [core] - 10https://gerrit.wikimedia.org/r/306318 [01:57:53] (03PS1) 10Legoktm: EditPage: Use context instead of globals (1/4) [core] - 10https://gerrit.wikimedia.org/r/306319 [01:57:56] (03PS1) 10Legoktm: EditPage: Use context instead of globals (2/4) [core] - 10https://gerrit.wikimedia.org/r/306320 [01:58:00] (03PS1) 10Legoktm: EditPage: Use context instead of globals (3/4) [core] - 10https://gerrit.wikimedia.org/r/306321 [01:58:02] (03PS1) 10Legoktm: EditPage: Use context instead of globals (4/4) [core] - 10https://gerrit.wikimedia.org/r/306322 [01:58:10] boom [01:59:28] TimStarling: I re-did it in 1000 line chunks this time, maybe it'll be easier to review? https://gerrit.wikimedia.org/r/#/q/topic:editpage-context [01:59:59] (03Abandoned) 10Legoktm: EditPage: Reduce dependence upon global objects [core] - 10https://gerrit.wikimedia.org/r/155692 (owner: 10Legoktm) [02:00:02] (03CR) 10Aaron Schulz: [C: 032] EditPage: Add getContext() function [core] - 10https://gerrit.wikimedia.org/r/306317 (owner: 10Legoktm) [02:00:34] (03CR) 10Aaron Schulz: [C: 032] EditPage: Fix some doc blocks [core] - 10https://gerrit.wikimedia.org/r/306318 (owner: 10Legoktm) [02:02:09] 10MediaWiki-General-or-Unknown, 07Regression: Back out the change that changed action=purge to require a confirmation page - https://phabricator.wikimedia.org/T143531#2578073 (10Pppery) >>! In T143531#2574043, @MZMcBride wrote: >>>! In T143531#2574033, @Pppery wrote: >>>>! In T143531#2574009, @MZMcBride wrote:... [02:05:50] AaronSchulz: are you reviewing all of them? or just those two? [02:06:06] 10Wikimedia-Site-requests, 06Wiktionary, 07Community-consensus-needed: request to localise namespace in km.wiktionary - https://phabricator.wikimedia.org/T140219#2578074 (10Octahedron80) 05Open>03Invalid Seems no one further discuss about this. I withdraw this request. [02:07:58] * legoktm is now having dinner with the pasta he bought in Italy, bbl [02:09:24] (03Merged) 10jenkins-bot: EditPage: Add getContext() function [core] - 10https://gerrit.wikimedia.org/r/306317 (owner: 10Legoktm) [02:09:46] 06ArchCom, 10ArchCom-RfC, 07RfC: [RFC] STUB: Creating a spec for legacy wikitext / future wiki content processing model - https://phabricator.wikimedia.org/T142803#2578076 (10ssastry) Sounds good. Where we stand today, Parsoid does a fair bit of **[[https://github.com/wikimedia/parsoid/commit/75f4841019182fb... [02:12:15] (03Merged) 10jenkins-bot: EditPage: Fix some doc blocks [core] - 10https://gerrit.wikimedia.org/r/306318 (owner: 10Legoktm) [02:19:41] (03CR) 10jenkins-bot: [V: 04-1] EditPage: Use context instead of globals (4/4) [core] - 10https://gerrit.wikimedia.org/r/306322 (owner: 10Legoktm) [02:22:44] TimStarling: all of them probably [02:24:35] (03PS2) 10Legoktm: EditPage: Use context instead of globals (4/4) [core] - 10https://gerrit.wikimedia.org/r/306322 [02:28:23] ok, thanks [02:29:10] (03PS3) 10Legoktm: EditPage: Use context instead of globals (4/4) [core] - 10https://gerrit.wikimedia.org/r/306322 [02:30:55] (03CR) 10Aaron Schulz: [C: 032] EditPage: Use context instead of globals (1/4) [core] - 10https://gerrit.wikimedia.org/r/306319 (owner: 10Legoktm) [02:34:09] (03CR) 10Aaron Schulz: [C: 032] EditPage: Use context instead of globals (2/4) [core] - 10https://gerrit.wikimedia.org/r/306320 (owner: 10Legoktm) [02:36:41] (03PS1) 10Gergő Tisza: Remove some unused hooks from hooks.txt [core] - 10https://gerrit.wikimedia.org/r/306323 [02:36:59] (03CR) 10Aaron Schulz: [C: 032] EditPage: Use context instead of globals (3/4) [core] - 10https://gerrit.wikimedia.org/r/306321 (owner: 10Legoktm) [02:39:10] (03Merged) 10jenkins-bot: EditPage: Use context instead of globals (1/4) [core] - 10https://gerrit.wikimedia.org/r/306319 (owner: 10Legoktm) [02:40:32] (03CR) 10Aaron Schulz: [C: 032] EditPage: Use context instead of globals (4/4) [core] - 10https://gerrit.wikimedia.org/r/306322 (owner: 10Legoktm) [02:41:00] (03Merged) 10jenkins-bot: EditPage: Use context instead of globals (2/4) [core] - 10https://gerrit.wikimedia.org/r/306320 (owner: 10Legoktm) [02:45:49] * legoktm hugs AaronSchulz [02:45:57] thanks :D [02:46:09] legoktm: well, there is https://gerrit.wikimedia.org/r/#/c/305952/12 :) [02:46:20] * AaronSchulz should probably go out at some point today [02:47:12] (03Merged) 10jenkins-bot: EditPage: Use context instead of globals (3/4) [core] - 10https://gerrit.wikimedia.org/r/306321 (owner: 10Legoktm) [02:47:28] (03CR) 10Aaron Schulz: [C: 032] Remove some unused hooks from hooks.txt [core] - 10https://gerrit.wikimedia.org/r/306323 (owner: 10Gergő Tisza) [02:49:31] (03Merged) 10jenkins-bot: EditPage: Use context instead of globals (4/4) [core] - 10https://gerrit.wikimedia.org/r/306322 (owner: 10Legoktm) [02:50:26] haha alright, I'll look [02:51:09] I'm trying to kill DataUpdate trying to do what DeferredUpdates should do [02:54:00] (03Merged) 10jenkins-bot: Remove some unused hooks from hooks.txt [core] - 10https://gerrit.wikimedia.org/r/306323 (owner: 10Gergő Tisza) [02:54:03] (03PS4) 10Niedzielski: Hygiene: functionalize code in pre-commit hook [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/305759 [02:54:05] (03PS4) 10Niedzielski: Hygiene: allow for many files in pre-commit hook [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/305758 [02:54:07] (03PS4) 10Niedzielski: Hygiene: dry up code in pre-commit hook [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/305757 [02:54:09] (03PS4) 10Niedzielski: add pre-commit hook test [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/305760 [02:55:21] (03CR) 10Niedzielski: "since this is just an interim commit, i've reverted to new lines in this patchset and restore null delimiters in subsequent patchsets" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/305757 (owner: 10Niedzielski) [02:56:55] (03CR) 10Legoktm: [C: 031] "LGTM, but I'll let Krinkle take another look and let him merge" [core] - 10https://gerrit.wikimedia.org/r/305952 (owner: 10Aaron Schulz) [02:57:37] (03PS8) 10Aaron Schulz: Various fixes to DeferrableUpdates related classes [core] - 10https://gerrit.wikimedia.org/r/305953 [03:01:07] AaronSchulz: tbh, I don't understand what the distinction between DataUpdates and DeferredUpdates are, and whether that distinction is useful [03:02:01] I like how rollbackTransaction does nothing because the sql subclass misnamed the method [03:03:32] -.- [03:08:06] (03CR) 10Legoktm: [C: 032] A few more DBLockManager fixes and cleanups [core] - 10https://gerrit.wikimedia.org/r/306147 (owner: 10Aaron Schulz) [03:08:07] legoktm: the only thing different it lets you do is run certain updates all in a transaction round rather than per update [03:08:45] I'm not sure how useful that it, and ironically, the updates don't cancel via rollbackMasterChanges() of the main transaction round before deferred updates [03:08:57] ...I need to add that logic to sqldataupdate at least [03:11:09] would it be too breaking to run the deferredupdates in one transaction round? [03:12:13] generally when the first round commits, you are obliged to doing as many secondary updates as you can. It's usually odd to fail the remaining ones because one failed. [03:12:41] it's almost tempting to do the opposite and have dataupdates be just like other deferredupdates [03:15:32] (03Merged) 10jenkins-bot: A few more DBLockManager fixes and cleanups [core] - 10https://gerrit.wikimedia.org/r/306147 (owner: 10Aaron Schulz) [03:17:16] (03CR) 10Legoktm: [C: 04-1] VirtualRESTServiceClient management cleanups (032 comments) [core] - 10https://gerrit.wikimedia.org/r/305943 (owner: 10Aaron Schulz) [03:17:20] I think https://gerrit.wikimedia.org/r/#/c/306249/ fixes the last core insert..select that can cause drift (or fsck labs slaves) [03:18:55] 10Wikimedia-Site-requests: Add templateeditor users group for fa.wikivoyage - https://phabricator.wikimedia.org/T143288#2578095 (10Dereckson) 05Open>03stalled [03:19:25] (03CR) 10Legoktm: [C: 032] Avoid INSERT..SELECT in LocalFileDeleteBatch [core] - 10https://gerrit.wikimedia.org/r/306249 (owner: 10Aaron Schulz) [03:21:37] (03PS5) 10Aaron Schulz: VirtualRESTServiceClient management cleanups [core] - 10https://gerrit.wikimedia.org/r/305943 [03:21:53] the others only move one row around [03:22:50] (03CR) 10Legoktm: [C: 032] VirtualRESTServiceClient management cleanups [core] - 10https://gerrit.wikimedia.org/r/305943 (owner: 10Aaron Schulz) [03:23:49] 10ProofreadPage: Makes Special:IndexPages show document covers - https://phabricator.wikimedia.org/T143764#2578096 (10Dereckson) [03:24:28] 10ProofreadPage, 13Patch-Needs-Improvement: Makes Special:IndexPages show document covers - https://phabricator.wikimedia.org/T143764#2578109 (10Dereckson) [03:25:27] (03Merged) 10jenkins-bot: Avoid INSERT..SELECT in LocalFileDeleteBatch [core] - 10https://gerrit.wikimedia.org/r/306249 (owner: 10Aaron Schulz) [03:26:41] (03CR) 10Legoktm: [C: 04-1] "The i18n message MZ pointed out needs to be updated. I also wonder if we somehow switched the paraminfo value to say this method needs pos" [core] - 10https://gerrit.wikimedia.org/r/288462 (https://phabricator.wikimedia.org/T135170) (owner: 10Aaron Schulz) [03:30:55] (03Merged) 10jenkins-bot: VirtualRESTServiceClient management cleanups [core] - 10https://gerrit.wikimedia.org/r/305943 (owner: 10Aaron Schulz) [03:38:29] 10MediaWiki-Codesniffer (MediaWiki-Codesniffer-GSoC-2016), 05Google-Summer-of-Code-2016: Endterm evaluation for "Improving static analysis tools for MediaWiki" - https://phabricator.wikimedia.org/T143495#2578111 (10Lethexie) Wiki page is a great idea. But Google says once I submit the link cannot be changed...... [03:43:04] (03PS1) 10Aaron Schulz: [WIP] Add shouldAbort() method to DataUpdate updates [core] - 10https://gerrit.wikimedia.org/r/306324 [03:47:10] (03PS1) 10Legoktm: Automatically submit the form on action=purge with JavaScript [core] - 10https://gerrit.wikimedia.org/r/306325 (https://phabricator.wikimedia.org/T143531) [03:48:30] (03CR) 10Legoktm: "The UI here is a little weird because you see the form for half a second and then it goes away. But I think this will assuage most people'" [core] - 10https://gerrit.wikimedia.org/r/306325 (https://phabricator.wikimedia.org/T143531) (owner: 10Legoktm) [03:51:59] 10MediaWiki-API, 07Performance: Option on API lists to only have count of links/categories/whatever returned, rather than a full resultset - https://phabricator.wikimedia.org/T19993#2578117 (10Od1n) [03:52:56] (03CR) 10jenkins-bot: [V: 04-1] Automatically submit the form on action=purge with JavaScript [core] - 10https://gerrit.wikimedia.org/r/306325 (https://phabricator.wikimedia.org/T143531) (owner: 10Legoktm) [03:57:09] (03PS1) 10Jack Phoenix: Use the correct variable to display the logo instead of just its URL [skins/Refreshed] - 10https://gerrit.wikimedia.org/r/306326 (https://phabricator.wikimedia.org/T142956) [04:00:58] (03PS2) 10Legoktm: Automatically submit the form on action=purge with JavaScript [core] - 10https://gerrit.wikimedia.org/r/306325 (https://phabricator.wikimedia.org/T143531) [04:01:03] (03PS1) 10Majr: WebRequest: Use TextContent::normalizeLineEndings [core] - 10https://gerrit.wikimedia.org/r/306327 [04:05:45] (03PS1) 10Dereckson: Fix whitespace issues in ProofreadPages [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306328 [04:05:47] (03PS1) 10Dereckson: Show document covers in Special:IndexPages [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306329 (https://phabricator.wikimedia.org/T143764) [04:06:41] 10ProofreadPage, 13Patch-For-Review: Makes Special:IndexPages show document covers - https://phabricator.wikimedia.org/T143764#2578130 (10Dereckson) [04:07:05] (03CR) 10jenkins-bot: [V: 04-1] Show document covers in Special:IndexPages [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306329 (https://phabricator.wikimedia.org/T143764) (owner: 10Dereckson) [04:09:16] (03PS2) 10Dereckson: Show document covers in Special:IndexPages [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306329 (https://phabricator.wikimedia.org/T143764) [04:10:37] 10Wiki-Loves-Monuments (2016): On-board Turkey - https://phabricator.wikimedia.org/T143765#2578134 (10LilyOfTheWest) [04:10:57] 10Wiki-Loves-Monuments (2016): On-board Turkey - https://phabricator.wikimedia.org/T143765#2578134 (10LilyOfTheWest) a:05leila>03LilyOfTheWest [04:24:55] (03CR) 10Legoktm: "Huh. Maybe we need to re-think this again...WebRequest shouldn't be calling TextContent. I guess this should just be a general function? w" [core] - 10https://gerrit.wikimedia.org/r/306327 (owner: 10Majr) [04:40:06] 10Wikimedia-General-or-Unknown: Investigate replacements for Google Hangouts - https://phabricator.wikimedia.org/T143247#2578159 (10Pine) 05Open>03Resolved a:03Pine I'm thinking about closing this as resolved, but I'm a bit reluctant because I still suspect that Hangouts is at fault for some problems. @bca... [04:44:31] (03PS1) 10Majr: Fix file summary message [extensions/MsUpload] (REL1_27) - 10https://gerrit.wikimedia.org/r/306330 [04:45:17] 07Design, 10VisualEditor, 10VisualEditor-MediaWiki, 07Epic: Provide a modern wikitext editor - https://phabricator.wikimedia.org/T104479#2578164 (10Pine) I'm wondering for the timeline of the rollout for this project. Is this 2 to 3 years away from going live, or are we likely to see changes to the wikitex... [04:47:26] 07Design, 10UI-Standardization, 10VisualEditor, 10WikiEditor: Use consistent icons for same/similar functions between editors - https://phabricator.wikimedia.org/T91467#2578166 (10Pine) Thanks. I've asked about the timeline in that other task. [05:21:55] 10Wikimedia-Site-requests: Namespace problem on Azerbaijani Wiktionary - https://phabricator.wikimedia.org/T143580#2578169 (10Aabdullayev851) 05Resolved>03Open Thank you very much. I think problem solved. [05:34:48] 06Wikipedia-Android-App-Backlog, 10Android-app-feature-Navigation (Android-app-feature-Navigation-MVP), 03Mobile-App-Android-Sprint-89-Actinium, 13Patch-For-Review: Move article actions into article action toolbar - https://phabricator.wikimedia.org/T135295#2578174 (10Mholloway) For QA: Please test out the... [05:46:06] 10Phabricator-Bot-Requests, 10Striker: Create Phabricator bot account for Striker - https://phabricator.wikimedia.org/T142539#2578175 (10mmodell) @bd808 done. [05:46:22] 10Phabricator-Bot-Requests, 10Striker: Create Phabricator bot account for Striker - https://phabricator.wikimedia.org/T142539#2578176 (10mmodell) 05Open>03Resolved a:03mmodell [05:58:33] 10MediaWiki-Revision-deletion, 10MediaWiki-extensions-CentralAuth, 06Stewards-and-global-tools: Global suppression does not work properly when the target has already been locally blocked - https://phabricator.wikimedia.org/T25310#2578180 (10Glaisher) I have written https://meta.wikimedia.org/wiki/User:Glaish... [05:59:15] 10MediaWiki-Revision-deletion, 10MediaWiki-extensions-CentralAuth, 06Stewards-and-global-tools: Global suppression does not work properly when the target has already been locally blocked - https://phabricator.wikimedia.org/T25310#2578181 (10Glaisher) a:05Glaisher>03None [06:11:48] (03PS1) 10Victor Vasiliev: Implement a tree-caching abuse filter parser [extensions/AbuseFilter] - 10https://gerrit.wikimedia.org/r/306331 [06:17:08] (03PS1) 10Robert Vogel: BSMassEditBase: Fix for NULL content [extensions/BlueSpiceFoundation] (REL1_23) - 10https://gerrit.wikimedia.org/r/306332 [06:17:41] (03CR) 10jenkins-bot: [V: 04-1] BSMassEditBase: Fix for NULL content [extensions/BlueSpiceFoundation] (REL1_23) - 10https://gerrit.wikimedia.org/r/306332 (owner: 10Robert Vogel) [06:18:01] (03CR) 10Robert Vogel: [C: 032 V: 032] BSMassEditBase: Fix for NULL content [extensions/BlueSpiceFoundation] (REL1_23) - 10https://gerrit.wikimedia.org/r/306332 (owner: 10Robert Vogel) [06:18:13] (03PS1) 10Robert Vogel: BSMassEditBase: Fix for NULL content [extensions/BlueSpiceFoundation] - 10https://gerrit.wikimedia.org/r/306333 [06:18:29] (03CR) 10jenkins-bot: [V: 04-1] BSMassEditBase: Fix for NULL content [extensions/BlueSpiceFoundation] (REL1_23) - 10https://gerrit.wikimedia.org/r/306332 (owner: 10Robert Vogel) [06:19:54] (03CR) 10Robert Vogel: [C: 031] "Okay. Waiting for L10N review." [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/305585 (owner: 10Mglaser) [06:42:48] (03PS3) 10Ljonka: bstaskapi add posibility to share task crud permissions with visual elements [extensions/BlueSpiceFoundation] - 10https://gerrit.wikimedia.org/r/305027 [06:51:20] (03PS4) 10Aaron Schulz: [WIP] Use adaptive CDN TTLs for page views [core] - 10https://gerrit.wikimedia.org/r/306140 [06:53:32] 07Design, 10Revision-Slider, 06TCB-Team, 06WMDE-Design, 03TCB-Team-Sprint-2016-08-11: Mobile behavior: Revision slider may always only have one tooltip - https://phabricator.wikimedia.org/T143090#2578193 (10Jan_Dittrich) > @Jan_Dittrich anything speaking against this from UX perspective? @Lea_WMDE: It... [06:54:06] (03CR) 10Robert Vogel: [C: 04-1] bstaskapi add posibility to share task crud permissions with visual elements (034 comments) [extensions/BlueSpiceFoundation] - 10https://gerrit.wikimedia.org/r/305027 (owner: 10Ljonka) [06:54:16] (03PS1) 10Aaron Schulz: Move Article checkLastModified() up to MediaWiki::performRequest [core] - 10https://gerrit.wikimedia.org/r/306334 [06:54:29] (03CR) 10Robert Vogel: [C: 032] BSMassEditBase: Fix for NULL content [extensions/BlueSpiceFoundation] - 10https://gerrit.wikimedia.org/r/306333 (owner: 10Robert Vogel) [06:55:47] (03Merged) 10jenkins-bot: BSMassEditBase: Fix for NULL content [extensions/BlueSpiceFoundation] - 10https://gerrit.wikimedia.org/r/306333 (owner: 10Robert Vogel) [06:55:51] 06Wikipedia-Android-App-Backlog, 10Android-app-feature-Navigation (Android-app-feature-Navigation-MVP), 03Mobile-App-Android-Sprint-89-Actinium: Hide/show the article action toolbar based on scroll actions - https://phabricator.wikimedia.org/T143766#2578194 (10Mholloway) [06:56:23] (03PS1) 10Mholloway: Hide/show article action toolbar depending on vertical scrolling actions [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306335 (https://phabricator.wikimedia.org/T143766) [06:56:31] 04(no projects): Using Quarry as the source in Massviews doesn't work in Safari - https://phabricator.wikimedia.org/T143767#2578210 (10MusikAnimal) [06:57:59] 06Wikipedia-Android-App-Backlog, 10Android-app-feature-Navigation (Android-app-feature-Navigation-MVP), 03Mobile-App-Android-Sprint-89-Actinium, 13Patch-For-Review: Hide/show the article action toolbar based on scroll actions - https://phabricator.wikimedia.org/T143766#2578194 (10Mholloway) a:03Mholloway [07:05:52] 10Hovercards, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, 07Spike: [ 8 hours ] Spike: Analyze Hovercards and general instrumentation code for potential sources of unexpected data outcomes - https://phabricator.wikimedia.org/T141922#2578243 (10phuedx) a:05phuedx>03Jhernandez Passing the baton... [07:07:54] 06Reading-Web-Backlog, 10TextExtracts, 13Patch-For-Review, 03Reading-Web-Sprint-79-Uh-oh, 05WMF-deploy-2016-08-23_(1.28.0-wmf.16): ExtractFormatterTest are broken - https://phabricator.wikimedia.org/T142230#2578248 (10phuedx) 05Open>03Resolved a:03phuedx ad435fb4 was rebased and merged. [07:23:10] 07Design, 10MediaWiki-History-or-Diffs, 06TCB-Team, 06WMDE-Design, 03TCB-Team-Sprint-2016-08-11: Indicate when text chunks have been moved, but not changed otherwise - https://phabricator.wikimedia.org/T143422#2578265 (10Jan_Dittrich) > I was still completely blind to the nl: line move. The nl: is grey... [07:27:07] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: [feature request] Watchlist display of wikidata edits (of items connected to a page via arbitrary access) is a bit confusing [add label of linked item] - https://phabricator.wikimedia.org/T125768#2578266 (10Esc3300) [07:30:40] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: [feature request] Watchlist display of wikidata edits (of items connected to a page via arbitrary access) is a bit confusing [add label of linked item] - https://phabricator.wikimedia.org/T125768#2578269 (10Esc3300) It would really help if the label of the lin... [07:31:30] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: [feature request] Watchlist display of wikidata edits (of items connected to a page via arbitrary access) is a bit confusing [add label of linked item] - https://phabricator.wikimedia.org/T125768#1996868 (10Esc3300) Forum discussion: https://www.wikidata.org/w... [07:34:40] (03PS1) 10Matthias Mullie: Add API warnings when upload is same as older versions [core] - 10https://gerrit.wikimedia.org/r/306336 (https://phabricator.wikimedia.org/T141822) [07:38:26] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 07Schema-change, 03Wikidata-Sprint-2016-08-16: Drop eu_touched - https://phabricator.wikimedia.org/T132628#2578283 (10Lydia_Pintscher) [07:45:40] 10UI-Standardization, 10OOjs-UI (OOjs-UI-0.17.9), 13Patch-For-Review, 03UI-Standardization-Kanban: MediaWiki theme: Unify selected toolbar menu, dropdown and outlineOptionWidget background-color - https://phabricator.wikimedia.org/T143634#2578287 (10Pginer-WMF) In our exploration around menus we were consi... [08:00:02] 10MobileFrontend, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, and 2 others: [SPIKE: 1 hour] Avoid unintended wrapping on en.m.wikipedia.org "2016 Formula One season" article - https://phabricator.wikimedia.org/T143558#2571698 (10TheDJ) People report that the flag i... [08:09:29] 07Design, 10MediaWiki-History-or-Diffs, 06TCB-Team, 06WMDE-Design, 03TCB-Team-Sprint-2016-08-11: Indicate when text chunks have been moved, but not changed otherwise - https://phabricator.wikimedia.org/T143422#2578304 (10Jan_Dittrich) @Alsee: You suggested to highlight the moved text, possibly with green... [08:12:24] 10Connected-Open-Heritage-Batch-uploads (GAR_2016-06): Add missing photographers for GAR images - https://phabricator.wikimedia.org/T141990#2519032 (10Lokal_Profil) a:05Lokal_Profil>03Mattias_Ostmar-WMSE [08:13:43] 10Connected-Open-Heritage-Batch-uploads (GAR_2016-06): Create infrastructure on Wikimedia Commons for GAR-Syria - https://phabricator.wikimedia.org/T140715#2578315 (10Lokal_Profil) 05stalled>03Resolved Closing this as the need to update the COH template does not block the usage of the GAR template. [08:17:31] (03PS1) 10Mholloway: Restore lead image gradient [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306344 (https://phabricator.wikimedia.org/T135294) [08:18:18] (03CR) 10Mxn: [C: 04-1] "There’s been some confusion over at Translatewiki.net regarding the intended meaning of the language name strings. I’ve reverted the probl" (033 comments) [wikimedia/portals] - 10https://gerrit.wikimedia.org/r/306241 (owner: 10Jdrewniak) [08:19:35] (03CR) 10jenkins-bot: [V: 04-1] Restore lead image gradient [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306344 (https://phabricator.wikimedia.org/T135294) (owner: 10Mholloway) [08:21:19] 10MobileFrontend, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, and 2 others: [SPIKE: 1 hour] Avoid unintended wrapping on en.m.wikipedia.org "2016 Formula One season" article - https://phabricator.wikimedia.org/T143558#2578321 (10Jhernandez) @TheDJ What happens is t... [08:23:47] 10MediaWiki-File-management, 10MediaWiki-Uploading, 06Commons, 06Multimedia, 13Patch-For-Review: Add 'no-change' and 'duplicate-of-old-version' warnings for uploading files - https://phabricator.wikimedia.org/T141822#2578322 (10matthiasmullie) a:03matthiasmullie [08:27:41] 10MediaWiki-extensions-WikibaseRepository, 10Wikibase-DataModel, 10Wikidata, 13Patch-For-Review, and 2 others: "The serialization "Q0" is not recognized by the configured id builders" - https://phabricator.wikimedia.org/T143744#2578327 (10thiemowmde) a:03thiemowmde [08:33:37] 10MobileFrontend, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, and 2 others: [SPIKE: 1 hour] Avoid unintended wrapping on en.m.wikipedia.org "2016 Formula One season" article - https://phabricator.wikimedia.org/T143558#2578331 (10Jhernandez) @Jdlrobson the current s... [08:38:04] 06Community-Liaisons (Jul-Sep-2016): Announce changes to login length to the communities - https://phabricator.wikimedia.org/T143112#2578334 (10Whatamidoing-WMF) 05Open>03Resolved This change seems to be stable. Please feel free to re-open if you want to discuss anything. [08:38:08] 10MediaWiki-extensions-CentralAuth, 10Wikimedia-General-or-Unknown, 03Collab-Team-Q1-July-Sep-2016, 06Community-Liaisons (Jul-Sep-2016), and 4 others: Increase "remember me" login cookie expiry from 30 days to 1 year on Wikimedia wikis - https://phabricator.wikimedia.org/T68699#2578336 (10Whatamidoing-WMF) [08:38:18] (03PS2) 10Mholloway: Restore lead image gradient [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306344 (https://phabricator.wikimedia.org/T135294) [08:44:59] (03PS1) 10Muehlenhoff: Mention security release and CVE IDs in changelog [debian] - 10https://gerrit.wikimedia.org/r/306397 [08:58:39] 10MediaWiki-General-or-Unknown, 06Operations, 06Services, 10Traffic: Investigate query parameter normalization for MW/services - https://phabricator.wikimedia.org/T138093#2578360 (10Jhernandez) It would be interesting to know if query parameters end up being serialized in alphabetical order, which would ma... [08:58:45] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Improve user experience in search - https://phabricator.wikimedia.org/T137068#2578362 (10Peter) Think this will be cool, really exciting! @nirzar I'm not sure about getting suggestions after one/two characters do we act... [09:00:37] (03CR) 10Thiemo Mättig (WMDE): "It looks like this code could have at least some tests when you start extracting functionality in public functions (or even other classes)" (032 comments) [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306316 (owner: 10Catrope) [09:06:53] 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-General-or-Unknown, 06Front-end-Standards-Group, 10Librarization: Split mediawiki module into multiple components - https://phabricator.wikimedia.org/T143463#2578365 (10Jhernandez) I'm not sure how the mediawiki implementation differs from the standard, bu... [09:09:59] (03CR) 10Daniel Kinzler: WebRequest: Use TextContent::normalizeLineEndings (031 comment) [core] - 10https://gerrit.wikimedia.org/r/306327 (owner: 10Majr) [09:10:25] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Technical-Debt: Fix Browser#supportsPositionFixed - https://phabricator.wikimedia.org/T143337#2578367 (10Jhernandez) {icon thumbs-up} [09:12:23] 10TimedMediaHandler, 10Wikispeech-Player, 06Wikispeech-WMSE, 10Wikispeech (Sprint 2016-08-24): make use of TMH in Wikispeech - https://phabricator.wikimedia.org/T142562#2578368 (10Lokal_Profil) [09:13:58] 10Wikispeech-Player, 06Wikispeech-WMSE, 10Wikispeech (Sprint 2016-08-24): Skip ahead (word) - https://phabricator.wikimedia.org/T140089#2578369 (10Lokal_Profil) [09:14:08] 10Wikispeech-Player, 06Wikispeech-WMSE, 10Wikispeech (Sprint 2016-08-24): [Task] Skip ahead (Wikispeech) - https://phabricator.wikimedia.org/T133688#2578370 (10Lokal_Profil) [09:15:08] (03PS1) 10Robert Vogel: BSApiTitleQueryStore: Fine tuning of database query [extensions/BlueSpiceFoundation] (REL1_23) - 10https://gerrit.wikimedia.org/r/306401 [09:15:43] (03CR) 10jenkins-bot: [V: 04-1] BSApiTitleQueryStore: Fine tuning of database query [extensions/BlueSpiceFoundation] (REL1_23) - 10https://gerrit.wikimedia.org/r/306401 (owner: 10Robert Vogel) [09:17:29] 10MediaWiki-API, 10MediaWiki-Uploading, 10Beta-Cluster-Infrastructure, 06Commons, and 3 others: internal_api_error_LocalFileLockError while uploading file via API to commons.wikimedia.beta.wmflabs.org - https://phabricator.wikimedia.org/T143655#2578374 (10zeljkofilipin) Tried again. ``` $ ls screenshots/... [09:17:37] 10Gerrit: Update gerrit to 2.12.4 - https://phabricator.wikimedia.org/T143089#2578375 (10Paladox) @demon gerrit 2.12.4 has been released now. https://www.gerritcodereview.com/download/gerrit-2.12.4.war Can we upgrade to it please? [09:18:35] 10Wikispeech-Player, 06Wikispeech-WMSE, 10Wikispeech (Sprint 2016-08-24): [Task] Re-listening to recitation (Wikispeech) - https://phabricator.wikimedia.org/T133687#2578376 (10Lokal_Profil) [09:19:39] 10MediaWiki-API, 10MediaWiki-Uploading, 10Beta-Cluster-Infrastructure, 06Commons, and 3 others: internal_api_error_LocalFileLockError while uploading file via API to commons.wikimedia.beta.wmflabs.org - https://phabricator.wikimedia.org/T143655#2578377 (10zeljkofilipin) I am doing something wrong, but not... [09:20:04] 10Gerrit: Update gerrit to 2.12.4 - https://phabricator.wikimedia.org/T143089#2578378 (10Paladox) This update may have performance improvements when we upgrade. But only for the dashboard. [09:21:46] (03PS3) 10DCausse: Use the UserTesting framework in maint scripts [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306169 [09:22:55] 10Increasing-content-coverage: Log the title the user wants to give the new article in the target language in "Create from scratch" - https://phabricator.wikimedia.org/T139790#2578379 (10schana) a:03schana [09:23:05] 10Increasing-content-coverage: Make "Create from scratch" more discoverable - https://phabricator.wikimedia.org/T134905#2578381 (10schana) a:03schana [09:23:18] 10MediaWiki-API, 10MediaWiki-Uploading, 10Beta-Cluster-Infrastructure, 06Commons, and 3 others: internal_api_error_LocalFileLockError while uploading file via API to commons.wikimedia.beta.wmflabs.org - https://phabricator.wikimedia.org/T143655#2575006 (10hashar) Date filter prevent that reqId from being s... [09:23:48] 10Gerrit: Full Phabricator links are being expanded weirdly in gerrit - https://phabricator.wikimedia.org/T75997#2578386 (10Paladox) @thiemowmde yes that's what I mean some plain T 1 for example links will break, I found they broke in the comment box but not in commit msg. [09:24:12] 10MediaWiki-API, 10MediaWiki-Uploading, 10Beta-Cluster-Infrastructure, 06Commons, and 3 others: internal_api_error_LocalFileLockError while uploading file via API to commons.wikimedia.beta.wmflabs.org - https://phabricator.wikimedia.org/T143655#2578387 (10zeljkofilipin) Thanks @hashar, did not notice time... [09:26:08] 10Gerrit: Full Phabricator links are being expanded weirdly in gerrit - https://phabricator.wikimedia.org/T75997#2578392 (10Paladox) I can test any proposed solutions, since I have a test instance. But we doint want to be breaking things when it only breaks if you link to the full link with a comment Id. [09:26:15] 10Wikispeech-Player, 06Wikispeech-WMSE, 10Wikispeech (Sprint 2016-08-24): [Task] Functionality: Play selected text (Wikispeech) - https://phabricator.wikimedia.org/T133680#2578393 (10Lokal_Profil) [09:27:59] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 07Story: [Story] Versioning in JSON output - https://phabricator.wikimedia.org/T92961#2578399 (10daniel) >>! In T92961#2577993, @Pfps wrote: > There is nothing preventing the dump format from having the first JSON object be information about the dump, in... [09:28:43] (03PS1) 10Robert Vogel: CSS class "wikitable": reset to MW default [skins/BlueSpiceSkin] (REL1_23) - 10https://gerrit.wikimedia.org/r/306402 [09:29:30] (03CR) 10DCausse: Use the UserTesting framework in maint scripts (031 comment) [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306169 (owner: 10DCausse) [09:31:16] 10MediaWiki-Codesniffer (MediaWiki-Codesniffer-GSoC-2016), 05Google-Summer-of-Code-2016: Endterm evaluation for "Improving static analysis tools for MediaWiki" - https://phabricator.wikimedia.org/T143495#2578401 (10Aklapper) Don't care about Google and copy stuff from a Google Doc to a wiki page which is a sub... [09:31:52] 10MediaWiki-extensions-WikibaseMediaInfo, 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 13Patch-For-Review, 03Wikidata-Sprint-2016-08-16: [Task] Create mediainfo entity on demand when adding a statement the first time - https://phabricator.wikimedia.org/T140760#2578402 (10daniel) 05Open>03Res... [09:32:01] 10Wikispeech-Player, 06Wikispeech-WMSE, 10Wikispeech (Sprint 2016-08-10): Map TTS response to page HTML - https://phabricator.wikimedia.org/T140105#2578405 (10Lokal_Profil) Worked on in: 2016-08-10: * Initial investigation To do in: Sprint 2016-08-24: * Remainder [09:33:50] 10MediaWiki-API, 10MediaWiki-Uploading, 10Beta-Cluster-Infrastructure, 06Commons, and 3 others: internal_api_error_LocalFileLockError while uploading file via API to commons.wikimedia.beta.wmflabs.org - https://phabricator.wikimedia.org/T143655#2578414 (10hashar) On beta in `wmf-config/fileback-labs.php` t... [09:33:59] 10Wikispeech-Player, 06Wikispeech-WMSE, 13Patch-For-Review, 10Wikispeech (Sprint 2016-08-10): Skip ahead (sentence) - https://phabricator.wikimedia.org/T140090#2452686 (10Lokal_Profil) a:05Sebastian_Berlin-WMSE>03Lokal_Profil Worked on in: 2016-08-10: * Code written and committed To do in: Sprint 2016... [09:34:40] 10Design Research tools and infrastructure, 06Design-Research-Backlog: document YouTube Live user study session setup - https://phabricator.wikimedia.org/T143752#2578421 (10Aklapper) Where are "the existing instructions" exactly located (URL)? [09:36:57] (03PS1) 10Phedenskog: Set right MessagesDirs [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306403 (https://phabricator.wikimedia.org/T143770) [09:39:04] 10Wikimedia-General-or-Unknown: Investigate replacements for Google Hangouts - https://phabricator.wikimedia.org/T143247#2578450 (10Aklapper) Audio issues/problems were already explained in T143247#2572009? [09:39:28] (03PS4) 10Ljonka: bstaskapi add posibility to share task crud permissions with visual elements [extensions/BlueSpiceFoundation] - 10https://gerrit.wikimedia.org/r/305027 [09:39:50] (03CR) 10Lokal Profil: "Note to self: The dependency tree is broken. rebase once a few of the other commits have been merged to ensure that tests pass" [extensions/Wikispeech] - 10https://gerrit.wikimedia.org/r/305640 (https://phabricator.wikimedia.org/T140221) (owner: 10Sebastian Berlin (WMSE)) [09:40:27] 10Gerrit: Full Phabricator links are being expanded weirdly in gerrit - https://phabricator.wikimedia.org/T75997#2578452 (10thiemowmde) Again: What does "break" mean? What **exactly** do you input? What **exactly** do you get? [09:40:58] 06Wikispeech-WMSE, 13Patch-For-Review, 10Wikispeech (Sprint 2016-08-10): Set up framework for extension specific variables - https://phabricator.wikimedia.org/T140221#2578455 (10Lokal_Profil) a:05Sebastian_Berlin-WMSE>03Lokal_Profil Worked on in: 2016-08-10: * Code written and committed To do in: Sprint... [09:41:38] 10MediaWiki-General-or-Unknown, 13Patch-For-Review, 07Regression: Back out the change that changed action=purge to require a confirmation page - https://phabricator.wikimedia.org/T143531#2578458 (10Legoktm) >>! In T143531#2578073, @Pppery wrote: > Ignoring pages than transclude magic words like {{CURRENTTIME... [09:43:02] 10MediaWiki-API, 10MediaWiki-Uploading, 10Beta-Cluster-Infrastructure, 06Commons, and 3 others: internal_api_error_LocalFileLockError while uploading file via API to commons.wikimedia.beta.wmflabs.org - https://phabricator.wikimedia.org/T143655#2578461 (10hashar) On deployment-redis02: ``` name=/var/log/r... [09:44:52] 10Wikispeech-Player, 06Wikispeech-WMSE, 13Patch-For-Review, 10Wikispeech (Sprint 2016-08-10): [Task] Enable start/stop using keyboard shortcuts (Wikispeech) - https://phabricator.wikimedia.org/T133676#2578468 (10Lokal_Profil) Worked on in: 2016-08-10: * //nothing// (depends on changes which were done in T1... [09:46:29] (03PS1) 10Phedenskog: Remove unused message keys in mustache [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306406 (https://phabricator.wikimedia.org/T143770) [09:47:20] 10Wikispeech-Player, 06Wikispeech-WMSE, 13Patch-For-Review, 10Wikispeech (Sprint 2016-08-10): [Task] Enable start/stop using mouse (Wikispeech) - https://phabricator.wikimedia.org/T133672#2239064 (10Lokal_Profil) a:05Sebastian_Berlin-WMSE>03Lokal_Profil Worked on in: 2016-08-10: * Unit tests implemente... [09:48:50] 10MediaWiki-API, 10MediaWiki-Uploading, 10Beta-Cluster-Infrastructure, 06Commons, and 3 others: internal_api_error_LocalFileLockError while uploading file via API to commons.wikimedia.beta.wmflabs.org - https://phabricator.wikimedia.org/T143655#2578477 (10hashar) Found the config in /etc/redis/tcp_6379.con... [09:49:21] 10Wikispeech, 06Wikispeech-WMSE: Set up Wikispeech stories on Phabricator - https://phabricator.wikimedia.org/T121206#2578478 (10Lokal_Profil) Note that there is now a #goal tag being used. [09:49:43] 10MediaWiki-General-or-Unknown, 06Operations, 06Services, 10Traffic: Investigate query parameter normalization for MW/services - https://phabricator.wikimedia.org/T138093#2578479 (10BBlack) My largest concern is still whether we can guarantee our application layer is insensitive to query parameter order, a... [09:58:36] 10Wikispeech, 06Wikispeech-STTS, 07Story: Add mapper functionality to map with two symbol set files - https://phabricator.wikimedia.org/T143771#2578493 (10HannaLindgren) [09:59:05] 10MediaWiki-General-or-Unknown, 13Patch-For-Review, 07Regression: Back out the change that changed action=purge to require a confirmation page - https://phabricator.wikimedia.org/T143531#2578508 (10Legoktm) To summarize: * This change was motivated by the multi-dc project, of which one of the requirements i... [10:01:26] 10Wikispeech, 10Wikispeech-NLP, 06Wikispeech-STTS: [Task] Conversion to phonetic markup (Wikispeech) - https://phabricator.wikimedia.org/T133838#2578513 (10Lokal_Profil) [10:01:28] 10Wikispeech, 10Wikispeech-NLP, 06Wikispeech-STTS: [Task] Define what subset of ssml we should initially support in the input text (Wikispeech) - https://phabricator.wikimedia.org/T134739#2578510 (10Lokal_Profil) 05Resolved>03Open Document (a list?) of which SSML can be sent on to the TTS. Create a foll... [10:02:49] 10MediaWiki-API, 10MediaWiki-Uploading, 10Beta-Cluster-Infrastructure, 06Commons, and 3 others: internal_api_error_LocalFileLockError while uploading file via API to commons.wikimedia.beta.wmflabs.org - https://phabricator.wikimedia.org/T143655#2578515 (10zeljkofilipin) 05Open>03Resolved a:03zeljkofil... [10:03:10] 10MediaWiki-Codesniffer (MediaWiki-Codesniffer-GSoC-2016), 05Google-Summer-of-Code-2016: Endterm evaluation for "Improving static analysis tools for MediaWiki" - https://phabricator.wikimedia.org/T143495#2578519 (10Legoktm) I think they mean you can't change the link, not the contents at the link? I'll email G... [10:06:01] 10MediaWiki-Codesniffer (MediaWiki-Codesniffer-GSoC-2016), 05Google-Summer-of-Code-2016: Endterm evaluation for "Improving static analysis tools for MediaWiki" - https://phabricator.wikimedia.org/T143495#2578520 (10Legoktm) Heh, I reloaded and... [10:06:39] 10Wikispeech: Determine which types of annotation are needed to handle an article in Wikispeech - https://phabricator.wikimedia.org/T143772#2578521 (10Lokal_Profil) [10:11:41] 10Gerrit: Full Phabricator links are being expanded weirdly in gerrit - https://phabricator.wikimedia.org/T75997#2578536 (10Paladox) @thiemowmde You said this This may lead to a situation where some specific T… numbers in specific contexts are not replaced any more. Which is, in my humble opinion, acceptable.... [10:14:14] 07Design, 10VisualEditor, 10VisualEditor-MediaWiki, 07Epic: Provide a modern wikitext editor - https://phabricator.wikimedia.org/T104479#2578539 (10Elitre) [10:23:00] (03CR) 10Robert Vogel: [C: 032] bstaskapi add posibility to share task crud permissions with visual elements [extensions/BlueSpiceFoundation] - 10https://gerrit.wikimedia.org/r/305027 (owner: 10Ljonka) [10:23:33] 10Wikispeech, 06Wikispeech-STTS, 07Story: Create new version of CMU dic - https://phabricator.wikimedia.org/T143773#2578540 (10HannaLindgren) [10:24:33] (03Merged) 10jenkins-bot: bstaskapi add posibility to share task crud permissions with visual elements [extensions/BlueSpiceFoundation] - 10https://gerrit.wikimedia.org/r/305027 (owner: 10Ljonka) [10:25:29] (03PS1) 10Ljonka: merge changes from bssmwconnector [extensions/BlueSpiceSMWConnector] - 10https://gerrit.wikimedia.org/r/306410 [10:30:03] (03PS1) 10Phedenskog: Use i18n messages for size [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306411 (https://phabricator.wikimedia.org/T143770) [10:31:01] 10Gerrit: Full Phabricator links are being expanded weirdly in gerrit - https://phabricator.wikimedia.org/T75997#2578563 (10thiemowmde) Is there something wrong with the question I'm now asking for the third time? Can you please, please, please explain your test scenario, what exactly you entered (please copy an... [10:34:37] 06WMF-NDA-Requests: NDA request for TheDJ - https://phabricator.wikimedia.org/T127430#2578564 (10Aklapper) 05Open>03Resolved Thanks everybody! @TheDJ is now member of #WMF-NDA. Closing as resolved. [10:35:22] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service: Edits being flagged by review tool on enwiki aren't likely to be damaging - https://phabricator.wikimedia.org/T143738#2577355 (10Ladsgroup) Per [[ https://ores.wikimedia.org/v2/scores/enwiki/damaging/?model_info | model info]] I set the high (defaul... [10:37:45] 10MediaWiki-extensions-ContentTranslation, 10ContentTranslation-non-beta-2016, 05ContentTranslation-Release10, 05Language-Engineering July-September 2016, and 2 others: Citations templates are sometimes translated to useless markup without content - https://phabricator.wikimedia.org/T139268#2578569 (10santh... [10:37:53] 10Wikispeech, 06Wikispeech-STTS, 07Story: Create error when symbol set file doesn't contain the right from/to name - https://phabricator.wikimedia.org/T143774#2578574 (10HannaLindgren) [10:38:52] 10Wikimedia-Site-requests: Namespace problem on Azerbaijani Wiktionary - https://phabricator.wikimedia.org/T143580#2578588 (10Aklapper) 05Open>03Resolved If it's solved there is no reason to reopen this task. :) Glad it works for you! [10:44:49] 10Gerrit: Full Phabricator links are being expanded weirdly in gerrit - https://phabricator.wikimedia.org/T75997#2578591 (10Paladox) @thiemowmde I have told you. When you use your change, it causes the T1 links for example to not work in the comment box but work in commit msg. IE T1 will no longer link, whereas... [10:45:48] 10Wikispeech, 06Wikispeech-STTS, 07Story: Synthesize-Listen-Edit-Synthesize-Listen-Update cycle - https://phabricator.wikimedia.org/T142469#2578594 (10HannaLindgren) [10:45:51] 10Wikispeech, 06Wikispeech-STTS, 07Story: Create new version of CMU dic - https://phabricator.wikimedia.org/T143773#2578593 (10HannaLindgren) 05Open>03Resolved [10:46:34] 06Wikispeech-STTS, 07Story, 10Wikispeech (Sprint 2016-08-10): Create new version of CMU dic - https://phabricator.wikimedia.org/T143773#2578540 (10HannaLindgren) [10:50:10] 10MediaWiki-Parser, 10MobileFrontend: Different behaviour between desktop and mobile with templates returning an initial asterisk - https://phabricator.wikimedia.org/T143775#2578597 (10Rotpunkt) [10:50:47] 10Wikispeech, 06Wikispeech-STTS, 07Story: Add mapper functionality to map with two symbol set files - https://phabricator.wikimedia.org/T143771#2578609 (10HannaLindgren) + re-write and re-run conversion scripts for CMU and NST-sv [10:50:57] 10Wikispeech, 06Wikispeech-STTS, 07Story: Add mapper functionality to map with two symbol set files - https://phabricator.wikimedia.org/T143771#2578610 (10HannaLindgren) [10:51:32] 10Wikispeech, 06Wikispeech-STTS, 07Story: Synthesize-Listen-Edit-Synthesize-Listen-Update cycle - https://phabricator.wikimedia.org/T142469#2578614 (10HannaLindgren) [10:51:35] 06Wikispeech-STTS, 07Story, 10Wikispeech (Sprint 2016-08-10): Define symbol set for CMU lexicon - https://phabricator.wikimedia.org/T143659#2578611 (10HannaLindgren) 05Open>03Resolved [10:53:04] 10Wikispeech, 06Wikispeech-STTS, 07Story: Add mapper functionality to map with two symbol set files - https://phabricator.wikimedia.org/T143771#2578616 (10HannaLindgren) [10:54:22] 10Wikispeech, 06Wikispeech-STTS, 07Story: Add mapper functionality to map with two symbol set files - https://phabricator.wikimedia.org/T143771#2578493 (10HannaLindgren) [10:58:03] (03PS1) 10Robert Vogel: replace jar file with valid signed certificate [extensions/BlueSpiceExtensions] (REL1_23) - 10https://gerrit.wikimedia.org/r/306415 [10:58:12] (03CR) 10Robert Vogel: [C: 032] replace jar file with valid signed certificate [extensions/BlueSpiceExtensions] (REL1_23) - 10https://gerrit.wikimedia.org/r/306415 (owner: 10Robert Vogel) [10:58:31] 10MediaWiki-extensions-ContentTranslation, 10ContentTranslation-non-beta-2016, 05ContentTranslation-Release10, 05Language-Engineering July-September 2016, and 2 others: Citations templates are sometimes translated to useless markup without content - https://phabricator.wikimedia.org/T139268#2578625 (10Amire... [10:58:34] (03Merged) 10jenkins-bot: replace jar file with valid signed certificate [extensions/BlueSpiceExtensions] (REL1_23) - 10https://gerrit.wikimedia.org/r/306415 (owner: 10Robert Vogel) [10:59:08] 10Hovercards, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, 07Spike: [ 8 hours ] Spike: Analyze Hovercards and general instrumentation code for potential sources of unexpected data outcomes - https://phabricator.wikimedia.org/T141922#2578626 (10Jhernandez) We (@jhernandez and @phuedx) think we've f... [11:04:10] 10Gerrit, 13Patch-For-Review: Full Phabricator links are being expanded weirdly in gerrit - https://phabricator.wikimedia.org/T75997#2578634 (10thiemowmde) Again, what exactly do you enter in the comment box? Only the two characters `T1` and nothing else? Other characters or words before or after? What happens... [11:14:14] 10Gerrit, 13Patch-For-Review: Full Phabricator links are being expanded weirdly in gerrit - https://phabricator.wikimedia.org/T75997#2578660 (10Aklapper) @paladox: Noone can look over your shoulder. So as long as you do not provide an absolutely clear list of numbered steps and **avoid any** paraphrasing and v... [11:16:43] (03PS1) 10Ljonka: InsertFile: make compatible with mw1.27 [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/306417 [11:18:11] (03PS1) 10Ljonka: AJAXInterface: quickfix to make ajax functions available [extensions/BlueSpiceFoundation] - 10https://gerrit.wikimedia.org/r/306418 [11:18:27] 10Phabricator, 06Project-Admins: Requests for addition to the #acl*Project-Admins group (in comments) - https://phabricator.wikimedia.org/T706#1818287 (10Mattias_Ostmar-WMSE) I'd like to be added to #project-admins in order to create sub-projects / milestones for the project #connected-open-heritage, please. [11:18:57] 10MediaWiki-extensions-LiquidThreads, 05Language-Engineering July-September 2016, 03Language-Q1-2016-17 Sprint 4, 07Unplanned-Sprint-Work: LiquidThreads does not truncate comments: Data too long for column 'th_change_comment' - https://phabricator.wikimedia.org/T124628#2578679 (10Nikerabbit) p:05Lowest>... [11:20:48] 10MediaWiki-extensions-LiquidThreads, 05Language-Engineering July-September 2016, 03Language-Q1-2016-17 Sprint 4, 07Unplanned-Sprint-Work: LiquidThreads does not truncate comments: Data too long for column 'th_change_comment' - https://phabricator.wikimedia.org/T124628#2578686 (10Nemo_bis) [11:22:20] 10Graphs, 10Technical-Tool-Request: Need a webpage to produce Pie chart according to Graphic Lab's recommendations - https://phabricator.wikimedia.org/T14603#2578692 (10hashar) 05Open>03Resolved Implemented as a Graph piechart template https://en.wikipedia.org/wiki/Template:Graph:PieChart I really love t... [11:24:38] 10EasyTimeline: Provide a stand-alone interactive visual tool to create and edit EasyTimelines (not in the VisualEditor) - https://phabricator.wikimedia.org/T29157#2578696 (10hashar) 05Open>03Resolved a:03hashar I agree with @Jdforrester-WMF lets focus on Graph instead of EasyTimeline which is really legac... [11:25:01] 10MediaWiki-Database, 10Beta-Cluster-Infrastructure, 10Continuous-Integration-Infrastructure, 10DBA, 07WorkType-NewFunctionality: Enable MariaDB/MySQL's Strict Mode - https://phabricator.wikimedia.org/T108255#2578701 (10Nikerabbit) [11:25:04] 10MediaWiki-extensions-LiquidThreads, 05Language-Engineering July-September 2016, 03Language-Q1-2016-17 Sprint 4, 07Unplanned-Sprint-Work: LiquidThreads does not truncate comments: Data too long for column 'th_change_comment' - https://phabricator.wikimedia.org/T124628#2578700 (10Nikerabbit) [11:26:27] 10EasyTimeline: Provide a stand-alone interactive visual tool to create and edit EasyTimelines (not in the VisualEditor) - https://phabricator.wikimedia.org/T29157#2578708 (10hashar) [11:26:31] 10EasyTimeline: EasyTimeline: replace perl script & Ploticus backend with PHP and output via GD or SVG - https://phabricator.wikimedia.org/T29156#2578705 (10hashar) 05Open>03declined This task proposes to migrate EasyTimeline to a new implementation using PHP / Ploticus. The extension is really replaced by t... [11:26:48] (03CR) 10Muehlenhoff: [C: 032] Mention security release and CVE IDs in changelog [debian] - 10https://gerrit.wikimedia.org/r/306397 (owner: 10Muehlenhoff) [11:32:38] (03PS1) 10Nikerabbit: Truncate change comments to avoid database errors [extensions/LiquidThreads] - 10https://gerrit.wikimedia.org/r/306420 (https://phabricator.wikimedia.org/T124628) [11:34:19] (03PS3) 10Mholloway: Restore lead image gradient [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306344 (https://phabricator.wikimedia.org/T135294) [11:39:08] (03CR) 10Florianschmidtwelzow: "That's the error of the first one (https://travis-ci.org/wikimedia/mediawiki-extensions-Lingo/jobs/154583320):" [extensions/Lingo] - 10https://gerrit.wikimedia.org/r/305343 (https://phabricator.wikimedia.org/T141281) (owner: 10Florianschmidtwelzow) [11:41:59] (03CR) 10Florianschmidtwelzow: "Follow up: I8bc10ca7a344390242777c255301d27665862b80" [extensions/Lingo] - 10https://gerrit.wikimedia.org/r/305343 (https://phabricator.wikimedia.org/T141281) (owner: 10Florianschmidtwelzow) [11:46:36] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: Entity selector should show the exact match as first result - https://phabricator.wikimedia.org/T86923#2578715 (10Sjoerddebruin) This is correct now for new statements AFAIK, but the problem still exists at moments when adding qualifiers or sources. [11:47:10] 10Wikispeech: Determine which types of annotation are needed to handle an article in Wikispeech - https://phabricator.wikimedia.org/T143772#2578717 (10Sebastian_Berlin-WMSE) [11:48:58] (03CR) 10Florianschmidtwelzow: "@Nikerabbit: Thanks for merging! *yay* :D Now I can remove the manual applied patch from my production system :P" [core] - 10https://gerrit.wikimedia.org/r/259968 (https://phabricator.wikimedia.org/T121834) (owner: 10Florianschmidtwelzow) [11:49:04] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 07Story: [Story] Add meta description in the header for Wikibase pages - https://phabricator.wikimedia.org/T88475#2578720 (10Sjoerddebruin) How hard is it to do this? It would be a huge improvement to the display of Wikidata items on Google. [11:49:51] 10Wikispeech, 06Wikispeech-WMSE: Determine which types of annotation are needed to handle an article in Wikispeech - https://phabricator.wikimedia.org/T143772#2578721 (10Sebastian_Berlin-WMSE) Story points are based on the two annotated articles in the pilot study. [11:50:44] (03CR) 10Nikerabbit: "I know the feeling :)" [core] - 10https://gerrit.wikimedia.org/r/259968 (https://phabricator.wikimedia.org/T121834) (owner: 10Florianschmidtwelzow) [11:52:45] 06Wikispeech-STTS, 07Story, 10Wikispeech (Sprint 2016-08-10): Meeting 23/8 - https://phabricator.wikimedia.org/T143778#2578727 (10HannaLindgren) [11:52:50] 10Wikispeech, 06Wikispeech-STTS, 07Story: Synthesize-Listen-Edit-Synthesize-Listen-Update cycle - https://phabricator.wikimedia.org/T142469#2536242 (10HannaLindgren) [11:52:52] 06Wikispeech-STTS, 07Story, 10Wikispeech (Sprint 2016-08-10): Meeting 23/8 - https://phabricator.wikimedia.org/T143778#2578727 (10HannaLindgren) 05Open>03Resolved [11:53:43] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 07TestMe: [Bug] Reference section remains the same size when collapsed (after expanding) - https://phabricator.wikimedia.org/T98318#2578746 (10Sjoerddebruin) @Addshore Can you please answer Lydia's question? I never saw this happening and this is more li... [11:55:54] 10Connected-Open-Heritage-Batch-uploads (DecArch 2016-07): Create a metadata field mapping document for DecArch - https://phabricator.wikimedia.org/T140727#2578751 (10Lokal_Profil) [11:58:26] 10MediaWiki-extensions-Diff, 10Wikidata, 07Story: [Story] Mark and show invisible characters in diff - https://phabricator.wikimedia.org/T113370#1663209 (10Sjoerddebruin) I can remember fixing external identifiers containing hidden characters, there is no indication of these in the diff view. [12:00:36] (03CR) 10Dbrant: [C: 04-1] "I may be missing a complicating factor here, but wouldn't it suffice to simply do the following:" [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306344 (https://phabricator.wikimedia.org/T135294) (owner: 10Mholloway) [12:02:38] 10Wikispeech, 06Wikispeech-WMSE: Populate Phase 2 - https://phabricator.wikimedia.org/T143780#2578773 (10Lokal_Profil) [12:03:48] 10MediaWiki-extensions-CentralNotice, 10Fundraising-Backlog, 10Hovercards, 06Reading-Web-Backlog: Measure impact of HoverCards on Central Notice interaction - https://phabricator.wikimedia.org/T131366#2578784 (10ovasileva) Whoo, thank you, @Pcoombe! I agree - we should test on other wikipedias and compare,... [12:09:08] (03CR) 10Glaisher: [C: 032] Truncate change comments to avoid database errors [extensions/LiquidThreads] - 10https://gerrit.wikimedia.org/r/306420 (https://phabricator.wikimedia.org/T124628) (owner: 10Nikerabbit) [12:10:08] (03Merged) 10jenkins-bot: Truncate change comments to avoid database errors [extensions/LiquidThreads] - 10https://gerrit.wikimedia.org/r/306420 (https://phabricator.wikimedia.org/T124628) (owner: 10Nikerabbit) [12:15:19] 10Wikispeech, 10Wikispeech-Player, 06Wikispeech-WMSE, 07Story: [Story] Recitation of marked text (Wikispeech) - https://phabricator.wikimedia.org/T121220#2578790 (10Lokal_Profil) [12:19:26] 10Wikispeech, 06Wikispeech-STTS, 07Story: Synthesize-Listen-Edit-Synthesize-Listen-Update cycle - https://phabricator.wikimedia.org/T142469#2578795 (10HannaLindgren) [12:19:28] 06Wikispeech-STTS, 07Story, 10Wikispeech (Sprint 2016-08-10): Create error when symbol set file doesn't contain the right from/to name - https://phabricator.wikimedia.org/T143774#2578793 (10HannaLindgren) 05Open>03Resolved [12:21:35] 10WikiCon-Francophone-2016: Numériser les dessins de la WikiConFR2016 - https://phabricator.wikimedia.org/T143781#2578797 (10Pyb) [12:22:42] 10Phabricator, 06Project-Admins: Requests for addition to the #acl*Project-Admins group (in comments) - https://phabricator.wikimedia.org/T706#2578808 (10Aklapper) @Mattias_Ostmar-WMSE: I've added you. [12:30:54] (03PS1) 10Mobrovac: Update change-propagation to 94aa503 [services/change-propagation/deploy] - 10https://gerrit.wikimedia.org/r/306422 [12:35:41] (03CR) 10Foxtrott: "> Follow up: I8bc10ca7a344390242777c255301d27665862b80" [extensions/Lingo] - 10https://gerrit.wikimedia.org/r/305343 (https://phabricator.wikimedia.org/T141281) (owner: 10Florianschmidtwelzow) [12:36:06] (03CR) 10Mobrovac: [C: 032 V: 032] Update change-propagation to 94aa503 [services/change-propagation/deploy] - 10https://gerrit.wikimedia.org/r/306422 (owner: 10Mobrovac) [12:40:54] (03PS1) 10DCausse: Add boost-templates setup for relforge and enwiki [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306423 [12:41:18] (03Draft2) 10Florianschmidtwelzow: Follow up php 5.3 issue in I087f13f66 [extensions/Lingo] - 10https://gerrit.wikimedia.org/r/306421 [12:41:32] (03CR) 10Cindy-the-browser-test-bot: [C: 031] "Cindy says good job \o/. Keep it up." [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306169 (owner: 10DCausse) [12:41:49] (03CR) 10Florianschmidtwelzow: "Argh, sorry, I forgot to click "Publish" again :P Made the change in the web UI of gerrit, where you need to click three times publish :D" [extensions/Lingo] - 10https://gerrit.wikimedia.org/r/305343 (https://phabricator.wikimedia.org/T141281) (owner: 10Florianschmidtwelzow) [12:42:08] (03CR) 10Cindy-the-browser-test-bot: "Phan static analysis done without errors!" [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306169 (owner: 10DCausse) [12:45:41] (03PS1) 10Mholloway: Article header: limit serif font to article title [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306424 (https://phabricator.wikimedia.org/T135294) [12:52:07] 10MediaWiki-ResourceLoader: Shibboleth Authentication isn't working due to missing ResourceLoaderUserGroupsModule class - https://phabricator.wikimedia.org/T143782#2578823 (10Urbanecm) [12:53:27] 10MediaWiki-Revision-deletion, 10MediaWiki-extensions-CentralAuth, 06Stewards-and-global-tools: Global suppression does not work properly when the target has already been locally blocked - https://phabricator.wikimedia.org/T25310#2578840 (10MarcoAurelio) >>! In T25310#2578180, @Glaisher wrote: > I have writt... [12:53:38] (03PS1) 10Mobrovac: Restart the nodes after a deploy [services/change-propagation/deploy] - 10https://gerrit.wikimedia.org/r/306426 [12:54:12] (03CR) 10Mobrovac: [C: 032 V: 032] Restart the nodes after a deploy [services/change-propagation/deploy] - 10https://gerrit.wikimedia.org/r/306426 (owner: 10Mobrovac) [13:01:24] (03CR) 10Cindy-the-browser-test-bot: [C: 031] "Cindy says good job \o/. Keep it up." [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306423 (owner: 10DCausse) [13:01:58] (03CR) 10Cindy-the-browser-test-bot: "Phan static analysis done without errors!" [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306423 (owner: 10DCausse) [13:03:08] 10MediaWiki-extensions-SemanticForms: Default Filename, Image Preview for uploadable field not working? - https://phabricator.wikimedia.org/T142979#2578848 (10Yaron_Koren) I doubt that has anything to do with this fix, so, yes, this would require a separate ticket. Though, looking at the JavaScript console for t... [13:04:26] 10Wikispeech, 06Wikispeech-STTS, 07Story: Add mapper functionality to map with two symbol set files - https://phabricator.wikimedia.org/T143771#2578849 (10HannaLindgren) [13:09:47] 10Wikispeech, 06Wikispeech-STTS, 07Story: When symbolset.Mapper finds unknown symbols, it should tell what symbols are incorrect - https://phabricator.wikimedia.org/T143783#2578851 (10HannaLindgren) [13:10:24] 10Wikispeech, 06Wikispeech-WMSE: Populate Phase 2 - https://phabricator.wikimedia.org/T143780#2578865 (10Lokal_Profil) a:03Sebastian_Berlin-WMSE Identified subtasks are: [ ] Find which Phases a Story is part of (can be multiple) [ ] Which parts of a Story contributes to which Phase (if multiple) - document t... [13:10:37] 06Research-and-Data: Request access to data for reader research - https://phabricator.wikimedia.org/T143718#2576804 (10ph_singer) Do we also need to make a ticket requesting access? [13:14:42] (03CR) 10DCausse: "I agree with you using defaults everywhere seems sane," (031 comment) [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) (owner: 10EBernhardson) [13:18:28] (03PS1) 10Mholloway: Restore lead image gradient (alternate version) [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306428 (https://phabricator.wikimedia.org/T135294) [13:25:10] (03CR) 10Dbrant: [C: 04-1] Restore lead image gradient (alternate version) (031 comment) [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306428 (https://phabricator.wikimedia.org/T135294) (owner: 10Mholloway) [13:25:12] 10Wikispeech, 06Wikispeech-STTS: symbolset.Mapper and Mappers should have build-in self-tests at least when loading from files - https://phabricator.wikimedia.org/T143786#2578919 (10HannaLindgren) [13:28:37] 10MediaWiki-Database, 10Wikimedia-Site-requests, 07Wikimedia-log-errors: Notice: Undefined variable: wmfMasterDatacenter in /srv/mediawiki/wmf-config/db-eqiad.php - https://phabricator.wikimedia.org/T143784#2578937 (10jcrespo) Hashar, I change #DBA to #mediawiki-database as this is a mediawiki code issue, bu... [13:31:01] (03PS1) 10Ljonka: InsertCategory: add ucfirst js function to make it compatible with mw 1.27 [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/306432 [13:37:03] 10MediaWiki-Database, 10Wikimedia-Site-requests, 07Wikimedia-log-errors: Notice: Undefined variable: wmfMasterDatacenter in /srv/mediawiki/wmf-config/db-eqiad.php - https://phabricator.wikimedia.org/T143784#2578943 (10jcrespo) This only happens 7 times (on time if you merge several occurrences of the same va... [13:38:48] 10Wikispeech, 06Wikispeech-STTS: Go through BUG and TODO comments in the source code and add tickets where applicable - https://phabricator.wikimedia.org/T143787#2578944 (10HannaLindgren) [13:39:56] (03CR) 10Dbrant: [C: 032] "nice! works on API 15, too." [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306424 (https://phabricator.wikimedia.org/T135294) (owner: 10Mholloway) [13:42:34] (03Merged) 10jenkins-bot: Article header: limit serif font to article title [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306424 (https://phabricator.wikimedia.org/T135294) (owner: 10Mholloway) [13:43:14] (03CR) 10Foxtrott: [C: 032] Follow up php 5.3 issue in I087f13f66 [extensions/Lingo] - 10https://gerrit.wikimedia.org/r/306421 (owner: 10Florianschmidtwelzow) [13:43:36] (03Merged) 10jenkins-bot: Follow up php 5.3 issue in I087f13f66 [extensions/Lingo] - 10https://gerrit.wikimedia.org/r/306421 (owner: 10Florianschmidtwelzow) [13:43:55] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 13Patch-For-Review, 07Story: [Story] Add meta description in the header for Wikibase pages - https://phabricator.wikimedia.org/T88475#2578959 (10Sjoerddebruin) https://yoast.com/meta-descriptions/ has some tips about meta descriptions. Some recommendat... [13:46:31] 04(no projects): Deprecation warning mcrypt_create_iv - https://phabricator.wikimedia.org/T143788#2578963 (10JeroenDeDauw) [13:47:36] 10MediaWiki-General-or-Unknown, 07Regression: Purge asks for "Clear the cache of this page" for logged in users with the "purge" right - https://phabricator.wikimedia.org/T143435#2578974 (10Aklapper) >>! In T143435#2570624, @MZMcBride wrote: > When you say "stopped working", you mean that purging now requires... [13:47:41] (03PS1) 10Dbrant: Fix pronunciation icon color. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306433 [13:53:12] 10Wikimedia-Site-requests, 13Patch-For-Review: Disable local uploads on ms.wikipedia.org (Malay Wikipedia) - https://phabricator.wikimedia.org/T141227#2578980 (10MarcoAurelio) 05Open>03Resolved File uploads are now totally disabled locally as requested. It is however worth noting that people will still hav... [13:54:07] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 13Patch-For-Review: Edits being flagged by review tool on enwiki aren't likely to be damaging - https://phabricator.wikimedia.org/T143738#2577355 (10hashar) Got it deployed. According to eval.php `$wgOresDamagingDefault` is properly set to `soft`... [13:57:39] 10MediaWiki-API, 06Community-Liaisons, 05MW-1.28-release-notes, 13Patch-For-Review, and 3 others: Require POST in ?action=purge - https://phabricator.wikimedia.org/T135170#2578988 (10Aklapper) Summary: * ?action=purge now requires POST (this very T135170; task currently still open for the API part in T1351... [14:00:47] 04(no projects): Deprecation warning mcrypt_create_iv - https://phabricator.wikimedia.org/T143788#2578963 (10TheDJ) https://wiki.php.net/rfc/mcrypt-viking-funeral [14:02:00] (03PS1) 10Jack Phoenix: Add new UsersPagerDoBatchLookups hook to allow for extensions to modify UsersPager::doBatchLookup()'s rigid behavior [core] - 10https://gerrit.wikimedia.org/r/306434 [14:03:54] 10MediaWiki-General-or-Unknown, 07Technical-Debt: Deprecation warnings about mcrypt_* functions in PHP 7.1 - https://phabricator.wikimedia.org/T143788#2578993 (10Aklapper) [14:04:27] 06Wikispeech-STTS, 10Wikispeech (Sprint 2016-08-10): symbolset.Mapper and Mappers should have build-in self-tests at least when loading from files - https://phabricator.wikimedia.org/T143786#2578995 (10HannaLindgren) 05Open>03Resolved [14:04:40] 10Wikispeech, 06Wikispeech-STTS: symbolset.Mapper and Mappers should have build-in self-tests at least when loading from files - https://phabricator.wikimedia.org/T143786#2578919 (10HannaLindgren) 05Resolved>03Open [14:04:58] 10Wikispeech, 06Wikispeech-STTS, 07Story: Add mapper functionality to map with two symbol set files - https://phabricator.wikimedia.org/T143771#2579000 (10HannaLindgren) [14:07:46] 06Research-and-Data: Request access to data for reader research - https://phabricator.wikimedia.org/T143718#2579001 (10leila) No, @ph_singer. I'll add Ops to this task as soon as we're ready for them. :) Thanks for checking though. [14:09:38] (03PS1) 10Robert Vogel: InsertLink: JAR Permissions & new JNLP endpoint [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/306437 [14:13:26] (03PS1) 10Robert Vogel: InsertLink: JAR Permissions & new JNLP endpoint #2 [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/306438 [14:15:33] 10MediaWiki-General-or-Unknown, 07NewPHP, 07Technical-Debt: Deprecation warnings about mcrypt_* functions in PHP 7.1 - https://phabricator.wikimedia.org/T143788#2579004 (10Aklapper) [14:16:42] 10Wikimedia-Site-requests: On wikis where uploads are fully disabled, remove don't show upload rights on any group - https://phabricator.wikimedia.org/T143789#2579006 (10MarcoAurelio) [14:17:12] 10Wikimedia-Site-requests: On wikis where uploads are fully disabled, remove don't show upload rights on any group - https://phabricator.wikimedia.org/T143789#2579019 (10MarcoAurelio) p:05Triage>03Low [14:17:14] (03PS1) 10Halfak: Increments testwiki's model version in the config [services/ores/deploy] - 10https://gerrit.wikimedia.org/r/306439 [14:17:24] https://www.mediawiki.org/wiki/Gerrit/Inactive_projects states "Remove all files in a new commit or make a commit that disables the software contained in it (e.g. disable the entry point if it is an Mediawiki extension)" [14:17:30] Does anyone have an example commit for that? [14:18:29] 10MediaWiki-Authentication-and-authorization: $wgBlockDisablesLogin = true; causes the wiki to be inaccessible - https://phabricator.wikimedia.org/T143790#2579024 (10Kghbln) [14:18:32] (03PS1) 10Robert Vogel: InsertLink: JAR Permissions & new JNLP endpoint #2 [extensions/BlueSpiceExtensions] (REL1_23) - 10https://gerrit.wikimedia.org/r/306441 [14:18:34] (03CR) 10Ladsgroup: [C: 032 V: 032] Increments testwiki's model version in the config [services/ores/deploy] - 10https://gerrit.wikimedia.org/r/306439 (owner: 10Halfak) [14:19:25] 10MediaWiki-Authentication-and-authorization: $wgBlockDisablesLogin = true; causes the wiki to be inaccessible - https://phabricator.wikimedia.org/T143790#2579038 (10Kghbln) [14:20:43] DanielK_WMDE: does https://gerrit.wikimedia.org/r/#/c/302199/5 look ready yet> [14:20:52] 10MediaWiki-Authentication-and-authorization: $wgBlockDisablesLogin = true; causes the wiki to be inaccessible - https://phabricator.wikimedia.org/T143790#2579024 (10Kghbln) @Bawolff I subscribed you since you were the author of the patch set which I think is the most likely the cause of the issue. [14:21:39] 10MediaWiki-Patrolling, 10MediaWiki-extensions-ORES, 10MediaWiki-extensions-Patroller, 10Revision-Scoring-As-A-Service-Backlog, 07Community-consensus-needed: Enable RC patrolling on trwiki - https://phabricator.wikimedia.org/T140475#2579040 (10Halfak) [14:22:11] (03CR) 10Dbrant: [C: 04-1] "Some general observations, from looking into this a bit further:" [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306335 (https://phabricator.wikimedia.org/T143766) (owner: 10Mholloway) [14:23:36] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 13Patch-For-Review, 15User-Ladsgroup: Edits being flagged by review tool on enwiki aren't likely to be damaging - https://phabricator.wikimedia.org/T143738#2579041 (10Ladsgroup) a:03Ladsgroup [14:24:20] 10MediaWiki-Authentication-and-authorization: $wgBlockDisablesLogin = true; causes the wiki to be inaccessible - https://phabricator.wikimedia.org/T143790#2579045 (10Kghbln) [14:27:20] 10MediaWiki-Patrolling, 10MediaWiki-extensions-ORES, 10MediaWiki-extensions-Patroller, 10ORES, 07Community-consensus-needed: Enable RecentChanges patrolling on English Wikipedia - https://phabricator.wikimedia.org/T143791#2579046 (10Halfak) [14:30:09] 10MediaWiki-Authentication-and-authorization: $wgBlockDisablesLogin = true; causes the wiki to be inaccessible for anonymous users - https://phabricator.wikimedia.org/T143790#2579070 (10Kghbln) [14:32:24] 10MediaWiki-Patrolling, 10MediaWiki-extensions-ORES, 10MediaWiki-extensions-Patroller, 10ORES, 07Community-consensus-needed: Enable RecentChanges patrolling on English Wikipedia - https://phabricator.wikimedia.org/T143791#2579046 (10Ladsgroup) One quick note: the patoller is not an extension. It's integr... [14:33:37] 10MediaWiki-Patrolling, 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 07Community-consensus-needed: Enable RecentChanges patrolling on English Wikipedia - https://phabricator.wikimedia.org/T143791#2579077 (10Ladsgroup) [14:33:41] 06Discovery, 10Wikimedia-Portals, 03Discovery-Portal-Sprint, 13Patch-For-Review: Wikipedia.org Portal: add meta description in page source - https://phabricator.wikimedia.org/T143239#2579078 (10JGirault) In my opinion this text is too long (197 characters). The suggested length for the meta description is... [14:35:48] 06Discovery, 10Wikimedia-Portals, 03Discovery-Portal-Sprint, 13Patch-For-Review: Wikipedia.org Portal: add meta description in page source - https://phabricator.wikimedia.org/T143239#2561679 (10Sjoerddebruin) >>! In T143239#2579078, @JGirault wrote: > In my opinion this text is too long (197 characters). T... [14:36:03] 10MediaWiki-extensions-WikibaseRepository, 06Discovery, 10Wikidata, 13Patch-For-Review, 07Story: [Story] Add meta description in the header for Wikibase pages - https://phabricator.wikimedia.org/T88475#2579082 (10Sjoerddebruin) [14:38:35] (03CR) 10Ljonka: [C: 032] InsertLink: JAR Permissions & new JNLP endpoint #2 [extensions/BlueSpiceExtensions] (REL1_23) - 10https://gerrit.wikimedia.org/r/306441 (owner: 10Robert Vogel) [14:39:02] (03Merged) 10jenkins-bot: InsertLink: JAR Permissions & new JNLP endpoint #2 [extensions/BlueSpiceExtensions] (REL1_23) - 10https://gerrit.wikimedia.org/r/306441 (owner: 10Robert Vogel) [14:39:18] (03CR) 10Ljonka: [C: 032] InsertLink: JAR Permissions & new JNLP endpoint #2 [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/306438 (owner: 10Robert Vogel) [14:39:46] (03Merged) 10jenkins-bot: InsertLink: JAR Permissions & new JNLP endpoint #2 [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/306438 (owner: 10Robert Vogel) [14:41:38] 06Wikipedia-Android-App-Backlog, 10Android-app-feature-Navigation (Android-app-feature-Navigation-MVP), 03Mobile-App-Android-Sprint-89-Actinium, 13Patch-For-Review: Move the article title from the lead image - https://phabricator.wikimedia.org/T135294#2579087 (10Mholloway) Hey @Nirzar, the Wikidata descrip... [14:43:29] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Align elements in article footer on beta - https://phabricator.wikimedia.org/T141002#2579088 (10bmansurov) @Nirzar, how should the cases like these be handled? {F4396723} Notice the whitespace after the footer. [14:43:59] (03CR) 10Ljonka: [C: 032] InsertLink: JAR Permissions & new JNLP endpoint [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/306437 (owner: 10Robert Vogel) [14:50:03] 10MediaWiki-General-or-Unknown: Special characters in URL lead to redirect loop under Apache 2.2.22 on Debian 7 - https://phabricator.wikimedia.org/T131414#2579092 (10Kghbln) It will be nice to have a fix in an upcoming version of MediaWiki. I still have no upgrade path for the wikis using 1.25.6 which become in... [14:50:44] (03Merged) 10jenkins-bot: InsertLink: JAR Permissions & new JNLP endpoint [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/306437 (owner: 10Robert Vogel) [14:55:46] (03PS1) 10Dbrant: Bring back parallax scroll of the lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306445 [14:56:04] 10MediaWiki-extensions-ConfirmAccount, 05MW-1.27-release-notes, 05MW-1.28-release-notes: Unable to create an account via "Special:ConfirmAccount" in 1.27 - requires porting to AuthManager - https://phabricator.wikimedia.org/T137843#2579095 (10Kghbln) This is really awesome news that this extension now works... [14:56:49] (03PS19) 10MarcoAurelio: Set fallback language for lzh to zh-hant [core] - 10https://gerrit.wikimedia.org/r/272499 (https://phabricator.wikimedia.org/T125373) [14:58:54] 06Design-Research, 10Reading-UX-Research: Plan study for new Read More algorithm - https://phabricator.wikimedia.org/T142009#2519663 (10Fjalapeno) @pizzzacat Did you work out what platform you want to do this test on? Are you planning on doing the qualitative portion first? [14:59:36] 10WikiCon-Francophone-2016, 06Community-Liaisons (Jul-Sep-2016): WMF presence at the French Wikiconference - https://phabricator.wikimedia.org/T124049#2579099 (10Trizek-WMF) 05Open>03Resolved [15:02:55] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Align elements in article footer on beta - https://phabricator.wikimedia.org/T141002#2579114 (10Nirzar) @bmansurov that should be grey too. think of the grey as the basic background of html and article being white backgr... [15:03:09] (03PS20) 10MarcoAurelio: Set fallback language for lzh to zh-hant [core] - 10https://gerrit.wikimedia.org/r/272499 (https://phabricator.wikimedia.org/T125373) [15:04:22] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Align elements in article footer on beta - https://phabricator.wikimedia.org/T141002#2579119 (10bmansurov) @jdlrobson, to sum up, -1'ed because of T141002#2577837 and T141002#2579114. [15:04:59] 06Design-Research, 10Reading-UX-Research: Plan study for new Read More algorithm - https://phabricator.wikimedia.org/T142009#2579122 (10Fjalapeno) Just so this isn't lost from the email thread: The potential uses for this service identified by the PMs were: 1. Using this service to replace the existing "more... [15:05:31] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Improve user experience in search - https://phabricator.wikimedia.org/T137068#2579138 (10Nirzar) >@Nirzar I'm not sure about getting suggestions after one/two characters do we actually get a good results? Thinking getting... [15:05:50] (03PS1) 10Bmansurov: Beta: clear the loading icon when a new search is started [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) [15:06:02] 10MediaWiki-Internationalization, 07I18n: Language fallback for Lzh (Zh-classical) should be Zh_hant - https://phabricator.wikimedia.org/T125373#2579140 (10MarcoAurelio) 05stalled>03Open a:03MarcoAurelio It seems that the Wikibase issues are now fixed and the patch is passing jenkins-tests, so maybe we c... [15:07:11] (03Abandoned) 10MarcoAurelio: Modify {{int:please-translate}}: added a full stop. [extensions/WikimediaMessages] - 10https://gerrit.wikimedia.org/r/302194 (https://phabricator.wikimedia.org/T141737) (owner: 10MarcoAurelio) [15:07:40] 06Discovery, 06Discovery-Analysis (Current work), 13Patch-For-Review: Wikipedia.org Portal Dashboard: add clicks by language - https://phabricator.wikimedia.org/T140816#2579148 (10debt) Sorry, maybe I'm missing the update, but I'm not seeing the fix for either the size of the font of the languages selected n... [15:08:33] 10MediaWiki-extensions-WikimediaMessages: Modify {{int:please-translate}}: add a full stop - https://phabricator.wikimedia.org/T141737#2579150 (10MarcoAurelio) 05Open>03declined Per patch. See @TTO reasoning. [15:12:45] 10Wikimedia-Site-requests, 13Patch-For-Review: On wikis where uploads are fully disabled remove upload rights on any group - https://phabricator.wikimedia.org/T143789#2579154 (10MarcoAurelio) [15:12:46] 10MediaWiki-General-or-Unknown, 07NewPHP, 07Technical-Debt: Deprecation warnings about mcrypt_* functions in PHP 7.1 - https://phabricator.wikimedia.org/T143788#2579155 (10TheDJ) > The CSPRNG openssl offers is a userspace PRNG based on hash functions, which goes against the advice of Thomas Ptacek to use /de... [15:20:53] (03CR) 10Anomie: [C: 04-1] Add API warnings when upload is same as older versions (034 comments) [core] - 10https://gerrit.wikimedia.org/r/306336 (https://phabricator.wikimedia.org/T141822) (owner: 10Matthias Mullie) [15:21:32] 06Discovery, 06Discovery-Analysis: Add comment to search-related eventlogging in iOS and Android apps to inform Discovery of changes - https://phabricator.wikimedia.org/T143753#2579183 (10debt) p:05Triage>03Low Just as a backstory to this ticket - it was raised during our sprint planning meeting after we t... [15:22:33] (03PS4) 10Bartosz Dziewoński: HTMLFormField: Move 'flatlist' handling to fields that use it and document [core] - 10https://gerrit.wikimedia.org/r/303694 [15:23:20] (03CR) 10Bartosz Dziewoński: [C: 032] "*poke*" [core] - 10https://gerrit.wikimedia.org/r/303694 (owner: 10Bartosz Dziewoński) [15:24:42] (03PS4) 10Bartosz Dziewoński: HTMLFormField: Don't display empty popup in OOUI mode if empty 'help' is given [core] - 10https://gerrit.wikimedia.org/r/304108 [15:24:49] (03CR) 10Anomie: Allow putting the app ID in the password for bot passwords (031 comment) [core] - 10https://gerrit.wikimedia.org/r/306133 (https://phabricator.wikimedia.org/T142304) (owner: 10Gergő Tisza) [15:24:55] 06Discovery, 06Discovery-Analysis: Create a parameterized report template for search team's A/B tests - https://phabricator.wikimedia.org/T131795#2579189 (10debt) Hi @mpopov and @chelsyx - is this something that needs to be done before we start work on T143589 and T143587 [15:25:12] 10Hovercards, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 06Community-Liaisons (Jul-Sep-2016): Start discussion with Italian community about running a/b test for hovercards in October - https://phabricator.wikimedia.org/T143170#2579194 (10CKoerner_WMF) Spoke with @Jseddon from Fundraising to dis... [15:27:12] 10Hovercards, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 06Community-Liaisons (Jul-Sep-2016): Start discussion with Italian community about running a/b test for hovercards in October - https://phabricator.wikimedia.org/T143170#2579200 (10Elitre) Sure thing, been following this for a while now. [15:31:50] (03Merged) 10jenkins-bot: HTMLFormField: Move 'flatlist' handling to fields that use it and document [core] - 10https://gerrit.wikimedia.org/r/303694 (owner: 10Bartosz Dziewoński) [15:32:13] (03PS1) 10Jdlrobson: Copy style attribute to lazy placeholder [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) [15:35:08] (03CR) 10Mholloway: [C: 032] Bring back parallax scroll of the lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306445 (owner: 10Dbrant) [15:36:29] 10MediaWiki-extensions-ConfirmAccount, 05MW-1.27-release-notes, 05MW-1.28-release-notes: Unable to create an account via "Special:ConfirmAccount" in 1.27 - requires porting to AuthManager - https://phabricator.wikimedia.org/T137843#2579217 (10aaron) There is still an annoying bug in core that makes the retur... [15:36:43] 06Wikipedia-Android-App-Backlog, 10Android-app-feature-Navigation (Android-app-feature-Navigation-MVP), 03Mobile-App-Android-Sprint-89-Actinium, 13Patch-For-Review: Move the article title from the lead image - https://phabricator.wikimedia.org/T135294#2579218 (10Mholloway) @Nirzar, Dmitry just put in a fix... [15:37:24] 10MediaWiki-General-or-Unknown, 07HHVM: 1.27-alpha / TitleMoveComplete-closure has invalid call signature - https://phabricator.wikimedia.org/T118683#1806490 (10Yaron_Koren) I, too, don't think this is an HHVM issue - I got a report of it happening with MW 1.27.0 and PHP 5.6.24, no HHVM in use. [15:37:53] (03Merged) 10jenkins-bot: Bring back parallax scroll of the lead image. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306445 (owner: 10Dbrant) [15:37:59] (03CR) 10Physikerwelt: [C: 031] "lgtm." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) (owner: 10Jdlrobson) [15:38:02] (03CR) 10Bartosz Dziewoński: [C: 031] Automatically submit the form on action=purge with JavaScript [core] - 10https://gerrit.wikimedia.org/r/306325 (https://phabricator.wikimedia.org/T143531) (owner: 10Legoktm) [15:40:01] (03CR) 10Kghbln: "Great and thank you for the patch. This should probably also be backported to REl1_27." [core] - 10https://gerrit.wikimedia.org/r/305834 (owner: 10Aaron Schulz) [15:40:18] (03CR) 10Bartosz Dziewoński: "Hmm, I guess this makes the 'purge' user right entirely pointless now. Or did we remove it already?" [core] - 10https://gerrit.wikimedia.org/r/306325 (https://phabricator.wikimedia.org/T143531) (owner: 10Legoktm) [15:43:35] 10MediaWiki-Internationalization, 07I18n: Language fallback for Lzh (Zh-classical) should be Zh_hant - https://phabricator.wikimedia.org/T125373#2579234 (10hoo) [15:43:39] 10MediaWiki-Internationalization, 10Wikidata, 07I18n: Wikibase LanguageFallbackChainFactoryTest makes assumptions on MediaWiki's language fallback chains - https://phabricator.wikimedia.org/T126944#2579232 (10hoo) 05Open>03Resolved [15:45:20] (03CR) 10Hoo man: [C: 031] "The Depends-On in the commit message is no longer needed, as the Wikibase changes has been merged. Should be fine to merge technically (bu" [core] - 10https://gerrit.wikimedia.org/r/272499 (https://phabricator.wikimedia.org/T125373) (owner: 10MarcoAurelio) [15:47:29] 10MediaWiki-API, 10MediaWiki-JobQueue, 05MW-1.27-release-notes, 13Patch-For-Review, 05codfw-rollout: ApiQuerySiteinfo - Exception Caught: No queue partitions available. - https://phabricator.wikimedia.org/T133045#2579239 (10Anomie) 05Open>03Resolved a:03Addshore I'm going to assume this is fixed si... [15:48:06] 06Discovery-Analysis (Current work): EventLogging DB's: add Deb - https://phabricator.wikimedia.org/T143724#2579242 (10debt) a:03debt Thanks - I'll get started soon! :) [15:49:46] (03CR) 10Anomie: [C: 04-1] "Needs rebase." [extensions/EventLogging] - 10https://gerrit.wikimedia.org/r/264494 (https://phabricator.wikimedia.org/T91454) (owner: 10IoannisKydonis) [15:51:39] 10MediaWiki-General-or-Unknown, 06Operations, 06Services, 10Traffic: Investigate query parameter normalization for MW/services - https://phabricator.wikimedia.org/T138093#2579264 (10Jhernandez) AFAIK I don't think that'd be a problem. Maybe @dr0ptp4kt has more insight. [15:53:21] 06Wikipedia-Android-App-Backlog, 10Android-app-feature-Navigation (Android-app-feature-Navigation-MVP), 03Mobile-App-Android-Sprint-89-Actinium, 13Patch-For-Review: Move the article title from the lead image - https://phabricator.wikimedia.org/T135294#2579268 (10Nirzar) @Mholloway great, i will check in ha... [15:58:07] 06Wikipedia-Android-App-Backlog, 13Patch-For-Review, 07Unplanned-Sprint-Work: [Dev] Upgrade to Support Library v24.0.0+ - https://phabricator.wikimedia.org/T138105#2579280 (10Dzahn) [15:58:25] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 07Story: [Story] Versioning in JSON output - https://phabricator.wikimedia.org/T92961#2579283 (10Pfps) > Of course it would by a breaking change. There is no formal spec of the JSON dump beyond the spec for the individual entities, but we have always sai... [15:58:37] 10MediaWiki-extensions-ContentTranslation, 10ContentTranslation-Deployments, 05Language-Engineering July-September 2016, 03Language-Q1-2016-17 Sprint 4, and 2 others: Enable sme-nob language pair for Apertium - https://phabricator.wikimedia.org/T120087#1844599 (10KartikMistry) https://integration.wikimedia... [15:59:28] (03PS1) 10Anomie: API: Some raw message cleanup [core] - 10https://gerrit.wikimedia.org/r/306453 [15:59:36] gwicke: which that was for an hour [16:00:18] gwicke: maybe we could find a meeting room in the office sometime and list out the major outstanding problems and possible workarounds [16:00:43] AaronSchulz: that could be helpful, yes [16:01:23] re one hour: I was hoping that we would make progress on the offline discussion in the meantime, and use the meeting primarily to summarize / agree [16:01:35] but that hasn't happened the way I had hoped [16:02:05] AaronSchulz: will you be in the office today? [16:02:59] 06Front-end-Standards-Group, 05MW-1.28-release-notes, 13Patch-For-Review, 05WMF-deploy-2016-08-16_(1.28.0-wmf.15): Start deprecation process to discourage use of various frontend ResourceLoader JS modules - https://phabricator.wikimedia.org/T142418#2579297 (10Jdlrobson) Hi apologies I forgot to hit send.... [16:04:07] (03CR) 10jenkins-bot: [V: 04-1] API: Some raw message cleanup [core] - 10https://gerrit.wikimedia.org/r/306453 (owner: 10Anomie) [16:04:29] gwicke: how did the hangout go? has a decision been reached? [16:04:48] 10Hovercards, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, 07Spike: [ 8 hours ] Spike: Analyze Hovercards and general instrumentation code for potential sources of unexpected data outcomes - https://phabricator.wikimedia.org/T141922#2579299 (10JKatzWMF) @phuedx @Jhernandez **nice find! ** I did so... [16:05:33] ori: no decision has been reached [16:05:41] sigh [16:05:47] what are the sticking points? [16:05:52] sadly [16:06:32] it's a complicated mix of political & technical things [16:06:51] still unsure about the percentages [16:07:41] 10Cassandra: Investigate lack of recency bias in Cassandra histogram metrics - https://phabricator.wikimedia.org/T137474#2579303 (10Eevans) A Debian package with a backported patch from https://issues.apache.org/jira/browse/CASSANDRA-11752 can be found at https://people.wikimedia.org/~eevans/debian/ [16:08:06] for the most part, Aaron, Eric & I are on the same page re requirements, but ops is still on the fence on that [16:08:24] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, 07Unplanned-Sprint-Work: Update browser tests to reflect absence of watchlist in main menu when anon - https://phabricator.wikimedia.org/T143796#2579304 (10Jdlrobson) [16:08:24] from what i hear, that's not accurate at all :) [16:09:18] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, 05WMF-deploy-2016-08-23_(1.28.0-wmf.16): Add a 'contributions' tab to mobile web stable channel - https://phabricator.wikimedia.org/T141787#2512130 (10Jdlrobson) This has caused a browser test regression which should be fixed before si... [16:09:33] 06Design-Research, 10Reading-UX-Research: Plan study for new Read More algorithm - https://phabricator.wikimedia.org/T142009#2519663 (10Jhernandez) When the S&D team implemented a new param for morelike queries (no boost on popularity) I ran some scripts on articles selected by community and Jon Katz to see th... [16:10:25] 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Spike: [SPIKE: 4hrs] Lazy loaded images data analysis and blog post - https://phabricator.wikimedia.org/T143627#2579326 (10MBinder_WMF) [16:12:10] mark: do tell me [16:12:40] from what I hear, that's a gross misrepresentation of the situation [16:13:18] check out the history of https://wikitech.wikimedia.org/wiki/SessionService/Storage [16:13:29] i have [16:13:50] 10MediaWiki-extensions-WikibaseRepository, 10Wikibase-DataModel, 10Wikidata, 13Patch-For-Review, and 2 others: "The serialization "Q0" is not recognized by the configured id builders" - https://phabricator.wikimedia.org/T143744#2579339 (10greg) p:05Unbreak!>03Normal Reducing priority as the logspam has... [16:15:39] (03CR) 10Aaron Schulz: Add API warnings when upload is same as older versions (031 comment) [core] - 10https://gerrit.wikimedia.org/r/306336 (https://phabricator.wikimedia.org/T141822) (owner: 10Matthias Mullie) [16:18:06] (03PS1) 10Anomie: API: Log when too many values are passed for a multi-valued parameter [core] - 10https://gerrit.wikimedia.org/r/306457 (https://phabricator.wikimedia.org/T41936) [16:18:20] 10Hovercards, 06Reading-Web-Backlog: hovercards shouldn't be an option in commons - https://phabricator.wikimedia.org/T143797#2579355 (10JKatzWMF) [16:18:26] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Spike, 07Technical-Debt: [SPIKE - 2 hrs] Should we remove supportsPositionFixed and Skin#setupPositionFixedEmulation? - https://phabricator.wikimedia.org/T143334#2579367 (10MBinder_WMF) [16:19:12] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Spike, 07Technical-Debt: [SPIKE: 2hrs] Should we remove supportsPositionFixed and Skin#setupPositionFixedEmulation? - https://phabricator.wikimedia.org/T143334#2564902 (10MBinder_WMF) [16:19:24] (03PS1) 10Brian Wolff: Enable $wgResourceLoaderStorageEnabled by default [core] - 10https://gerrit.wikimedia.org/r/306458 [16:19:27] 06Discovery, 06Discovery-Analysis (Current work), 13Patch-For-Review: Wikipedia.org Portal Dashboard: add clicks by language - https://phabricator.wikimedia.org/T140816#2579380 (10mpopov) 1. Would it be better if I were to just hide the title if bottom 50 is selected? 2. The languages **are** in order of hig... [16:20:03] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Spike, 07Technical-Debt: [SPIKE: 2hrs] Should we remove supportsPositionFixed and Skin#setupPositionFixedEmulation? - https://phabricator.wikimedia.org/T143334#2579385 (10Jdlrobson) [16:20:31] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Spike, 07Technical-Debt: [2hrs] Should we remove supportsPositionFixed and Skin#setupPositionFixedEmulation? - https://phabricator.wikimedia.org/T143334#2579388 (10Jhernandez) [16:22:33] 10MediaWiki-extensions-ZeroBanner, 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V: Page actions bar not showing on Wikipedia Zero - https://phabricator.wikimedia.org/T143425#2579394 (10Jdlrobson) This is not a regression. It was purposely done in https://gerrit.wikimedia.org/r/#/c/113655/ [16:22:46] 10MediaWiki-API: API gives spurious "unrecognized parameter" warnings for parameters added by extensions - https://phabricator.wikimedia.org/T49046#2579396 (10Anomie) 05Open>03Invalid This is too vague, there's nothing actionable to do here. If it comes up for a specific extension, we can look at it then. A... [16:23:00] hoo: how is https://gerrit.wikimedia.org/r/#/c/302199/5 coming along? [16:23:17] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 15User-Ladsgroup: Update model versions is badly broken in ORES extension - https://phabricator.wikimedia.org/T143798#2579398 (10Ladsgroup) [16:23:20] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 15User-Ladsgroup: Update model versions is badly broken in ORES extension - https://phabricator.wikimedia.org/T143799#2579413 (10Ladsgroup) [16:24:14] 10MediaWiki-Database, 10Wikimedia-Site-requests, 07Wikimedia-log-errors: Notice: Undefined variable: wmfMasterDatacenter in /srv/mediawiki/wmf-config/db-eqiad.php - https://phabricator.wikimedia.org/T143784#2579446 (10aaron) a:03aaron [16:24:19] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Technical-Debt: Fix Browser#supportsPositionFixed - https://phabricator.wikimedia.org/T143337#2579447 (10Jhernandez) This is fixing `supportsPositionFixed` to work on the codebase, but in any case `this.on('scroll'` in the skin would sti... [16:28:21] 10MediaWiki-Parser, 10MobileFrontend: Different behaviour between desktop and mobile with templates returning an initial asterisk - https://phabricator.wikimedia.org/T143775#2579452 (10Arlolra) > Which is the correct behaviour? Desktop. This is related to T89331 MobileFrontend needs to add, https://gerrit.... [16:29:05] 10MediaWiki-API: Allow extension options to be specified in API calls - https://phabricator.wikimedia.org/T54031#2579456 (10Anomie) To do something about this, I think we'd need: # Put these extension options in ParserOptions, somehow. The extension would also have to depend on the ParserOptions rather than user... [16:31:26] 10MediaWiki-extensions-ZeroBanner, 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V: Page actions bar not showing on Wikipedia Zero - https://phabricator.wikimedia.org/T143425#2579458 (10Jdlrobson) It seems this should only happen when ``` $isZeroTraffic || $this->isZeroSubdomain ``` Som... [16:31:47] 10MediaWiki-extensions-MultimediaViewer, 07Browser-Tests: Download menu.Clicking the image closes the download menu browser test fails in Firefox - https://phabricator.wikimedia.org/T143801#2579459 (10Jdlrobson) [16:32:20] 10MediaWiki-extensions-ZeroBanner, 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Spike: [Spike: 2 hours] Page actions bar not showing on Wikipedia Zero - https://phabricator.wikimedia.org/T143425#2579472 (10dr0ptp4kt) [16:32:31] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 15User-Ladsgroup: Update model versions is badly broken in ORES extension - https://phabricator.wikimedia.org/T143799#2579413 (10Ladsgroup) 05Open>03Invalid [16:35:26] 10MediaWiki-API: Make it easier to get incremental revision data via API - https://phabricator.wikimedia.org/T58848#2579484 (10Anomie) 05Open>03declined > * list=recentchanges should have an rcprop=content option Let's not. list=recentchanges is complex enough already without worrying about the details requ... [16:36:59] 06Design-Research, 10Reading-UX-Research: Plan study for new Read More algorithm - https://phabricator.wikimedia.org/T142009#2579489 (10JKatzWMF) Another thing to consider is how we measure improvement. There was some legitimate questioning on the related pages talk page about whether highest click % was real... [16:38:29] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Spike, 07Technical-Debt: [SPIKE: 2hrs] Should we remove supportsPositionFixed and Skin#setupPositionFixedEmulation? - https://phabricator.wikimedia.org/T143334#2579492 (10MBinder_WMF) [16:40:38] 10MediaWiki-General-or-Unknown, 07Tracking: Bugs affecting translatewiki.net (errors from MediaWiki master) (tracking) - https://phabricator.wikimedia.org/T41480#2579497 (10Nikerabbit) [16:40:44] 10MediaWiki-Database, 10Wikimedia-Site-requests, 07Wikimedia-log-errors: Notice: Undefined variable: wmfMasterDatacenter in /srv/mediawiki/wmf-config/db-eqiad.php - https://phabricator.wikimedia.org/T143784#2579499 (10hashar) >>! In T143784#2578937, @jcrespo wrote: > Hashar, I change #DBA to #mediawiki-datab... [16:40:48] 10WikiCon-Francophone-2016: Remercier Kacie - https://phabricator.wikimedia.org/T143802#2579500 (10Pyb) [16:40:51] (03CR) 10Nikerabbit: "This re-introduces T105890" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/301382 (https://phabricator.wikimedia.org/T140327) (owner: 10Sbisson) [16:41:05] (03CR) 10Bmansurov: [C: 04-1] "Needs more work imo." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) (owner: 10Bmansurov) [16:41:31] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Spike: Spike: Explore methods to move first paragraph above infobox - https://phabricator.wikimedia.org/T143803#2579511 (10Jdlrobson) [16:41:42] (03PS1) 10Subramanya Sastry: Sync up with Parsoid parserTests. [core] - 10https://gerrit.wikimedia.org/r/306461 [16:42:46] 10Mobile-Content-Service: Use summary endpoint data for random inside aggregated endpoint - https://phabricator.wikimedia.org/T143804#2579524 (10bearND) [16:44:26] 06Discovery, 10Wikimedia-Portals, 03Discovery-Portal-Sprint, 13Patch-For-Review: Wikipedia.org Portal: add meta description in page source - https://phabricator.wikimedia.org/T143239#2561679 (10Deskana) >>! In T143239#2579078, @JGirault wrote: > Also, just my 2 cents, I personally feel we should emphasize... [16:46:03] AaronSchulz: I guess so… stuck in review :S [16:46:21] 10MediaWiki-API, 05Security: XSS vulnerability scanner false positives - https://phabricator.wikimedia.org/T36257#2579548 (10Anomie) @dpatrick, @Bawolff: Would a "scan-safe" configuration parameter that disables some formats and changes the behavior of others be useful? Or would it just cause more trouble sinc... [16:47:35] 10MediaWiki-extensions-UniversalLanguageSelector, 10MediaWiki Language Extension Bundle, 05Language-Engineering July-September 2016, 03Language-Q1-2016-17 Sprint 4, 07Unplanned-Sprint-Work: Switch default ULS geoip provider - https://phabricator.wikimedia.org/T143542#2579553 (10Nikerabbit) [16:48:25] 07Design, 10MediaWiki-extensions-Cards, 06Reading-Web-Backlog: Load the cards skeleton and then load the content. - https://phabricator.wikimedia.org/T122212#2579556 (10dr0ptp4kt) Untagging from sprint for the time being, the thinking being that the other tasks that speed up the API and more pre-emptively fe... [16:48:36] 07Design, 10Notifications, 03Collab-Team-Q1-July-Sep-2016, 13Patch-For-Review, 05WMF-deploy-2016-08-23_(1.28.0-wmf.16): Notices tray icon with 99+ needs more space - https://phabricator.wikimedia.org/T142454#2579558 (10Etonkovidova) The changes have been made and they are in UI: `top: 9px; ` `left: 55%;... [16:48:49] 07Design, 10MediaWiki-extensions-Cards, 06Reading-Web-Backlog: Load the cards skeleton and then load the content. - https://phabricator.wikimedia.org/T122212#2579560 (10dr0ptp4kt) p:05High>03Low [16:51:00] 10Hovercards, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V: Fix duplicate logging of click events in Hovercards - https://phabricator.wikimedia.org/T143805#2579567 (10Jhernandez) [16:51:00] hoo: when they all get through, you'll have a beer on me ;) [16:51:57] 10Hovercards, 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V: Fix duplicate logging of click events in Hovercards - https://phabricator.wikimedia.org/T143805#2579594 (10Jhernandez) [16:52:04] hah, thanks :) [16:52:40] 10MobileFrontend, 06Reading-Web-Backlog, 07Easy, 07Regression: Regression: Close icon is hidden and badly positioned in image overlay - https://phabricator.wikimedia.org/T107607#2579595 (10Jdlrobson) [16:55:33] 10MediaWiki-Parser, 10MediaWiki-Unit-tests, 10Parsoid, 07RfC: [RFC]: Split parser tests info multiple files - https://phabricator.wikimedia.org/T111604#2579599 (10ssastry) Possible structure for splitting up the parserTests file: ``` quote list table heading indent_pre paragraph links/ - wikilink - extlink... [16:57:01] (03PS2) 10Kaldari: [WIP] Refactoring job queue logic for PageAssessments [extensions/PageAssessments] - 10https://gerrit.wikimedia.org/r/306314 [16:57:52] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Refactoring job queue logic for PageAssessments [extensions/PageAssessments] - 10https://gerrit.wikimedia.org/r/306314 (owner: 10Kaldari) [16:58:31] (03PS3) 10Kaldari: [WIP] Refactoring job queue logic for PageAssessments [extensions/PageAssessments] - 10https://gerrit.wikimedia.org/r/306314 [16:58:33] (03PS1) 10Sbisson: Don't select echo_event.* [extensions/Echo] - 10https://gerrit.wikimedia.org/r/306465 (https://phabricator.wikimedia.org/T105890) [16:59:14] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Refactoring job queue logic for PageAssessments [extensions/PageAssessments] - 10https://gerrit.wikimedia.org/r/306314 (owner: 10Kaldari) [17:00:24] (03PS1) 10Ladsgroup: Fix CheckModelVersions by changing order of actions [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306468 (https://phabricator.wikimedia.org/T143799) [17:00:53] (03CR) 10Ladsgroup: "Tested in mw-revscoring.wmflabs.org" [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306468 (https://phabricator.wikimedia.org/T143799) (owner: 10Ladsgroup) [17:02:06] 10Design Research tools and infrastructure, 06Design-Research-Backlog: document YouTube Live user study session setup - https://phabricator.wikimedia.org/T143752#2579616 (10Capt_Swing) @Aklapper https://docs.google.com/a/wikimedia.org/document/d/1Ng42qq5tGNyQxo4L3XhGbq8XIqMrfiueBF0MzohEJ-Y/edit?usp=sharing Wh... [17:03:10] 05iOS-app-v5.1.0-Saturn: New screenshots for the 5.1 app store release - https://phabricator.wikimedia.org/T143583#2579624 (10JMinor) a:03Nirzar [17:07:05] 10Hovercards, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, 07Spike: [ 8 hours ] Spike: Analyze Hovercards and general instrumentation code for potential sources of unexpected data outcomes - https://phabricator.wikimedia.org/T141922#2579655 (10Jhernandez) @JKatzWMF Nice, thanks. > Earlier we foun... [17:07:22] 10MediaWiki-Vagrant, 06Release-Engineering-Team, 07Epic: [EPIC] Migrate base image to Debian Jessie - https://phabricator.wikimedia.org/T136429#2579656 (10greg) >>! In T136429#2572477, @greg wrote: > Not there, sadly. There are just only so many SF<->European slots in the week and our team meetings overlap.... [17:07:43] (03CR) 10Arlolra: [C: 032] Sync up with Parsoid parserTests. [core] - 10https://gerrit.wikimedia.org/r/306461 (owner: 10Subramanya Sastry) [17:08:34] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, 07Unplanned-Sprint-Work: Update browser tests to reflect absence of watchlist in main menu when anon - https://phabricator.wikimedia.org/T143796#2579657 (10Jdlrobson) a:05dr0ptp4kt>03None [17:14:36] (03Merged) 10jenkins-bot: Sync up with Parsoid parserTests. [core] - 10https://gerrit.wikimedia.org/r/306461 (owner: 10Subramanya Sastry) [17:14:42] 10MediaWiki-extensions-MultimediaViewer, 06Reading-Web-Backlog, 07Browser-Tests: Browser test "Download menu.Clicking the image closes the download menu browser test" fails in Firefox - https://phabricator.wikimedia.org/T143801#2579672 (10Jdlrobson) [17:14:44] 10MediaWiki-extensions-MultimediaViewer, 06Reading-Web-Backlog, 07Browser-Tests: Browser test "Download menu.Clicking the image closes the download menu browser test" fails in Firefox - https://phabricator.wikimedia.org/T143801#2579459 (10Jdlrobson) [17:14:46] 10MobileFrontend, 06Reading-Web-Backlog: Search failures are not handled - https://phabricator.wikimedia.org/T143741#2579674 (10Jdlrobson) [17:15:25] 10MediaWiki-extensions-MultimediaViewer, 06Reading-Web-Backlog, 07Browser-Tests: Browser test "Download menu.Clicking the image closes the download menu" fails in Firefox - https://phabricator.wikimedia.org/T143801#2579459 (10Jdlrobson) [17:15:36] 05iOS-app-v5.1.0-Saturn: New screenshots for the 5.1 app store release - https://phabricator.wikimedia.org/T143583#2579677 (10JMinor) [17:15:52] 10MediaWiki-extensions-MultimediaViewer, 06Reading-Web-Backlog, 07Browser-Tests: Browser test "Download menu.Clicking the image closes the download menu" fails in Firefox - https://phabricator.wikimedia.org/T143801#2579459 (10Jdlrobson) p:05Triage>03Normal [17:19:58] (03PS2) 10Anomie: API: Some raw message cleanup [core] - 10https://gerrit.wikimedia.org/r/306453 [17:20:14] 07Design, 10MobileFrontend, 06Reading-Web-Backlog: Search failures are not handled - https://phabricator.wikimedia.org/T143741#2579692 (10Nirzar) Let me recreate this, @Jdlrobson @dr0ptp4kt cc @ovasileva what is the priority? [17:21:15] 06Wikipedia-Android-App-Backlog, 10Android-app-feature-Navigation (Android-app-feature-Navigation-MVP), 03Mobile-App-Android-Sprint-89-Actinium, 13Patch-For-Review, 07Unplanned-Sprint-Work: Hide/show the article action toolbar based on scroll actions - https://phabricator.wikimedia.org/T143766#2579695 (10... [17:22:53] (03CR) 10Ori.livneh: "Krinkle, do you have any reservations about this?" [core] - 10https://gerrit.wikimedia.org/r/306458 (owner: 10Brian Wolff) [17:24:04] 10MediaWiki-extensions-Translate: Add feature to show units with no reviews by any user - https://phabricator.wikimedia.org/T143806#2579701 (10Glaisher) [17:25:59] gwicke: not sure about being in the office today [17:26:26] 10MediaWiki-extensions-WikibaseClient, 10ArticlePlaceholder, 10Wikidata: [Task] Implement statement order fetching from other Wikis - https://phabricator.wikimedia.org/T134639#2579717 (10hoo) a:05Lucie>03hoo [17:28:50] 06Discovery, 10Wikimedia-Portals, 06Discovery-Analysis (Current work): Wikipedia.org Portal Visitors' Session Lengths (Redux) - https://phabricator.wikimedia.org/T143137#2579722 (10debt) Would we be able to filter out the "other" page views and do this analysis again? Other pageview treatment is listed in... [17:29:47] (03PS3) 10Anomie: API: Use U+001F (Unit Separator) for separating multi-valued parameters [core] - 10https://gerrit.wikimedia.org/r/305126 (https://phabricator.wikimedia.org/T141960) [17:30:25] AaronSchulz: I am in the office now, and will be tomorrow as well [17:30:37] will be out Friday, Monday & Tuesday [17:31:00] dapatrick: can you look at https://gerrit.wikimedia.org/r/#/c/305834/ ? [17:31:22] e.g. the last comment [17:31:34] Yep, already did. I was just about to +2 it. We discussed it yesterday in team meeting. [17:31:45] AaronSchulz You're referring to MaxSem's comment? [17:31:54] yeah, I wasn't sure what the issue was [17:32:13] I'd like to merge and backport that to improve confirmaccount in 1.27 [17:33:56] 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, 07Unplanned-Sprint-Work: Update browser tests to reflect absence of watchlist in main menu when anon - https://phabricator.wikimedia.org/T143796#2579304 (10jhobs) a:03jhobs [17:34:05] (03Abandoned) 10Jdlrobson: Hide clear icon @ 240px resolutions [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/301171 (owner: 10Jdlrobson) [17:34:11] (03PS1) 10Aaron Schulz: Make Special:CreateAccount respect ?returnto parameter [core] (REL1_27) - 10https://gerrit.wikimedia.org/r/306474 [17:34:28] (03CR) 10jenkins-bot: [V: 04-1] API: Use U+001F (Unit Separator) for separating multi-valued parameters [core] - 10https://gerrit.wikimedia.org/r/305126 (https://phabricator.wikimedia.org/T141960) (owner: 10Anomie) [17:36:17] AaronSchulz Was this feature specifically requested to support redirecting to interwiki sites? If not, how do you feel about restricting it, as in https://phab.wmfusercontent.org/file/data/lamyrmq5tkwh5cqf5ykj/PHID-FILE-f44xkzsg6jngbz4odeij/0001-SECURITY-Don-t-redirect-to-external-sites-after-logi.patch? [17:38:00] (03CR) 10Catrope: [C: 032] Don't select echo_event.* [extensions/Echo] - 10https://gerrit.wikimedia.org/r/306465 (https://phabricator.wikimedia.org/T105890) (owner: 10Sbisson) [17:38:38] (03CR) 10jenkins-bot: [V: 04-1] Add separators between footer links in beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [17:39:02] 10Monitoring, 13Patch-For-Review, 05Prometheus-metrics-monitoring: test prometheus mysqld-exporter - https://phabricator.wikimedia.org/T126757#2579740 (10jcrespo) So the changes on the defaults do not work on trusty, I will revert that to: "ARGS=" I cannot find there log of why it fails, so I cannot debug,... [17:39:08] (03CR) 10Jdlrobson: [C: 031] "What needs more work?" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) (owner: 10Bmansurov) [17:39:10] (03CR) 10jenkins-bot: [V: 04-1] Footer / chrome color changes [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306472 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [17:41:30] (03PS2) 10Jdlrobson: Footer / chrome color changes [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306472 (https://phabricator.wikimedia.org/T141002) [17:41:37] 07Design, 10MobileFrontend, 06Reading-Web-Backlog: Search failures are not handled - https://phabricator.wikimedia.org/T143741#2577415 (10phuedx) @Nirzar: While your thinking about this problem, also bear in mind changes to other UI elements that we might want to make if we detect that the user has lost conn... [17:41:46] (03PS2) 10Jdlrobson: Add separators between footer links in beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) [17:42:15] dapatrick: no, it's just for internal [17:42:25] so I don't care if it's restricted [17:42:37] 10MediaWiki-API, 10MediaWiki-Uploading, 10Beta-Cluster-Infrastructure, 06Commons, and 3 others: internal_api_error_LocalFileLockError while uploading file via API to commons.wikimedia.beta.wmflabs.org - https://phabricator.wikimedia.org/T143655#2575006 (10AlexMonk-WMF) This instance probably crashed or som... [17:42:42] 10MediaWiki-Internationalization, 07Easy, 07I18n: Signatures aren't working correctly at ang.wikipedia.org - https://phabricator.wikimedia.org/T143110#2579755 (10Whatamidoing-WMF) I have just checked, and this problem has been fixed on the site. [17:43:13] AaronSchulz Okay, I'm going to comment about that on the patch. [17:43:46] already amending [17:43:53] (03CR) 10Jhobs: "Aaaaaaand you uploaded a new patch set as I was commenting haha." (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306472 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [17:44:29] (03PS3) 10Aaron Schulz: Make Special:CreateAccount respect ?returnto parameter [core] - 10https://gerrit.wikimedia.org/r/305834 [17:44:37] dapatrick: ^ [17:44:51] (03CR) 10Jforrester: "Add a RELEASE NOTES entry, maybe?" [core] - 10https://gerrit.wikimedia.org/r/306458 (owner: 10Brian Wolff) [17:46:12] (03Merged) 10jenkins-bot: Don't select echo_event.* [extensions/Echo] - 10https://gerrit.wikimedia.org/r/306465 (https://phabricator.wikimedia.org/T105890) (owner: 10Sbisson) [17:47:06] 10Phabricator, 06Operations: migrate RT maint-announce into phabricator - https://phabricator.wikimedia.org/T118176#2579765 (10Dzahn) mailing list removed again by request. i don't know if moving this to phabricator is a thing that we want to happen in the future or never. [17:47:22] 10Monitoring, 13Patch-For-Review, 05Prometheus-metrics-monitoring: test prometheus mysqld-exporter - https://phabricator.wikimedia.org/T126757#2579767 (10jcrespo) BTW, interesting behavior seeing graphs with gaps: https://grafana.wikimedia.org/dashboard/db/mysql?from=1472055221970&to=1472060298229&var-server... [17:47:50] (03CR) 10Dpatrick: [C: 032] "MaxSem, Aaron and I talked about this and I asked him to add a check that the redirect target is internal, a la Lego's patch on https://p" [core] - 10https://gerrit.wikimedia.org/r/305834 (owner: 10Aaron Schulz) [17:48:36] (03CR) 10jenkins-bot: [V: 04-1] Add separators between footer links in beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [17:48:53] 10MediaWiki-General-or-Unknown, 07JavaScript: JS mw.Title does not normalize input to NFC form - https://phabricator.wikimedia.org/T143810#2579774 (10matmarex) [17:49:41] 10MediaWiki-General-or-Unknown, 07JavaScript: JS mw.Title does not normalize input to NFC form - https://phabricator.wikimedia.org/T143810#2579790 (10matmarex) [17:49:43] 10MediaWiki-API: API: add normalized info also for unicode normalization of titles - https://phabricator.wikimedia.org/T29849#2579791 (10matmarex) [17:53:53] (03CR) 10GWicke: "Very nice work. Thank you, @aaron!" [core] - 10https://gerrit.wikimedia.org/r/305943 (owner: 10Aaron Schulz) [17:54:14] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 03Wikidata-Sprint-2016-08-16: Return actual term language in mw.wikibase.label - https://phabricator.wikimedia.org/T143094#2579796 (10hoo) 05Open>03Resolved [17:55:35] (03PS2) 10Aaron Schulz: Make Special:CreateAccount respect ?returnto parameter [core] (REL1_27) - 10https://gerrit.wikimedia.org/r/306474 [17:55:42] (03Merged) 10jenkins-bot: Make Special:CreateAccount respect ?returnto parameter [core] - 10https://gerrit.wikimedia.org/r/305834 (owner: 10Aaron Schulz) [17:56:01] (03Abandoned) 10Aaron Schulz: Make Special:CreateAccount respect ?returnto parameter [core] (REL1_27) - 10https://gerrit.wikimedia.org/r/306474 (owner: 10Aaron Schulz) [17:57:46] (03PS1) 10Aaron Schulz: Make Special:CreateAccount respect ?returnto parameter [core] (REL1_27) - 10https://gerrit.wikimedia.org/r/306477 [17:58:35] (03CR) 10Aaron Schulz: [C: 032] Make Special:CreateAccount respect ?returnto parameter [core] (REL1_27) - 10https://gerrit.wikimedia.org/r/306477 (owner: 10Aaron Schulz) [17:59:05] 06Discovery, 10Wikimedia-Portals, 06Discovery-Analysis (Current work): Wikipedia.org Portal Visitors' Session Lengths (Redux) - https://phabricator.wikimedia.org/T143137#2579799 (10chelsyx) @debt Let's talk about it this afternoon. I think event logging and page views are different. [17:59:10] 10MediaWiki-API: API: add normalized info also for unicode normalization of titles - https://phabricator.wikimedia.org/T29849#331441 (10matmarex) We currently go to great lengths both to prevent any non-NFC text from entering MediaWiki and to prevent any non-NFC text from leaving it. I don't think we should chan... [18:00:06] (03CR) 10Jhobs: [C: 031] "+1, assuming specificity is needed on the .animations rule to override some existing rule." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306472 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [18:00:52] https://www.mediawiki.org/wiki/Gerrit/Inactive_projects states "Remove all files in a new commit or make a commit that disables the software contained in it (e.g. disable the entry point if it is an Mediawiki extension)". Does anyone have an example commit for that? [18:02:57] andre__: yes, let me find it [18:05:28] andre__: https://gerrit.wikimedia.org/r/#/c/87029/ [18:05:34] MatmaRex: Thanks! That will allow me to close one unmaintained repository, finally. :) [18:06:00] andre__: this is actually kind of old, might be one of the first cases where we did this. but i can't seem to find any recent ones [18:07:07] andre__: a bit less ancient: https://gerrit.wikimedia.org/r/144988 [18:07:23] (03PS5) 10BearND: Change way to get title from links [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/303942 (https://phabricator.wikimedia.org/T136223) [18:07:25] (03Merged) 10jenkins-bot: Make Special:CreateAccount respect ?returnto parameter [core] (REL1_27) - 10https://gerrit.wikimedia.org/r/306477 (owner: 10Aaron Schulz) [18:07:34] (03PS6) 10BearND: Change way to get title from links [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/303942 (https://phabricator.wikimedia.org/T136223) [18:08:06] MatmaRex: Still, better than nothing to link to in https://www.mediawiki.org/wiki/Gerrit/Inactive_projects#Actions_to_deactivate_an_unused_repository [18:08:07] added [18:08:08] thanks [18:09:30] (03CR) 10Smalyshev: [C: 031] Call ssl_set() in DatabaseMysqli if DBO_SSL is set [core] - 10https://gerrit.wikimedia.org/r/306026 (https://phabricator.wikimedia.org/T136218) (owner: 10Aaron Schulz) [18:10:17] 10MediaWiki-Patrolling, 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 07Community-consensus-needed: Enable RecentChanges patrolling on English Wikipedia - https://phabricator.wikimedia.org/T143791#2579840 (10Halfak) [18:10:53] (03PS2) 10Bmansurov: Beta: clear the loading icon when a new search is started [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) [18:12:31] (03CR) 10Bmansurov: "> What needs more work?" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) (owner: 10Bmansurov) [18:14:51] (03PS3) 10Esanders: Move originalDomElements to IV store and use new hash [extensions/Cite] - 10https://gerrit.wikimedia.org/r/305328 [18:14:53] 10MediaWiki-extensions-Scribunto, 10Wikidata: Lua should have access to get Q-ID from wiki:page pair - https://phabricator.wikimedia.org/T135442#2579852 (10hoo) [18:14:57] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: [Task] Add mw.wikibase.getEntityObject by site link (title) Lua function - https://phabricator.wikimedia.org/T74815#2579855 (10hoo) [18:15:05] (03CR) 10Jforrester: [C: 032] Move originalDomElements to IV store and use new hash [extensions/Cite] - 10https://gerrit.wikimedia.org/r/305328 (owner: 10Esanders) [18:15:38] (03CR) 10jenkins-bot: [V: 04-1] Beta: clear the loading icon when a new search is started [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) (owner: 10Bmansurov) [18:16:44] (03PS3) 10Jdlrobson: Add separators between footer links in beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) [18:16:56] (03PS3) 10Bmansurov: Beta: clear the loading icon when a new search is started [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) [18:17:40] (03CR) 10Jforrester: [V: 032] Move originalDomElements to IV store and use new hash [extensions/Cite] - 10https://gerrit.wikimedia.org/r/305328 (owner: 10Esanders) [18:19:11] (03CR) 10Jdlrobson: [C: 032] Beta: clear the loading icon when a new search is started [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) (owner: 10Bmansurov) [18:20:27] (03Abandoned) 10Jdlrobson: POC: Refactor performSearch method [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306058 (owner: 10Jdlrobson) [18:27:49] 10MobileFrontend, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, and 2 others: [SPIKE: 1 hour] Avoid unintended wrapping on en.m.wikipedia.org "2016 Formula One season" article - https://phabricator.wikimedia.org/T143558#2579879 (10Jdlrobson) Given `.flagicon` is an e... [18:28:44] (03CR) 10jenkins-bot: [V: 04-1] Beta: clear the loading icon when a new search is started [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) (owner: 10Bmansurov) [18:28:51] (03CR) 10Jhobs: [C: 031] "Assuming Jenkins passes." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [18:29:16] (03CR) 10jenkins-bot: [V: 04-1] Add separators between footer links in beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [18:30:03] (03PS1) 10Jdlrobson: Don't treat certain image placeholders as block [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306479 (https://phabricator.wikimedia.org/T143558) [18:30:07] 10MobileFrontend, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 13Patch-For-Review, and 3 others: [SPIKE: 1 hour] Avoid unintended wrapping on en.m.wikipedia.org "2016 Formula One season" article - https://phabricator.wikimedia.org/T143558#2579898 (10Jdlrobson) a:03Jdlrobson [18:31:22] (03CR) 10Jdlrobson: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [18:31:38] RoanKattouw: does https://gerrit.wikimedia.org/r/#/c/306334/ look sane? [18:31:41] (03CR) 10Bmansurov: [C: 04-1] "We also need to apply the style data attribute to the image in the front end." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) (owner: 10Jdlrobson) [18:32:10] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 13Patch-For-Review, 15User-Ladsgroup: Edits being flagged by review tool on enwiki aren't likely to be damaging - https://phabricator.wikimedia.org/T143738#2577355 (10Ladsgroup) 05Open>03Resolved [18:32:46] (03PS2) 10Jdlrobson: Copy style attribute to lazy placeholder [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) [18:32:56] (03PS3) 10Jdlrobson: Copy style attribute to lazy placeholder [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) [18:34:29] (03CR) 10Catrope: [C: 032] Fix CheckModelVersions by changing order of actions [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306468 (https://phabricator.wikimedia.org/T143799) (owner: 10Ladsgroup) [18:35:38] (03Merged) 10jenkins-bot: Fix CheckModelVersions by changing order of actions [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306468 (https://phabricator.wikimedia.org/T143799) (owner: 10Ladsgroup) [18:35:47] (03CR) 10jenkins-bot: [V: 04-1] Don't treat certain image placeholders as block [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306479 (https://phabricator.wikimedia.org/T143558) (owner: 10Jdlrobson) [18:36:14] 07Design, 06Discovery, 10Kartographer, 06Maps, 03Maps-Sprint: and/or : Image size is not fitted to the width of the marker tool tip window - https://phabricator.wikimedia.org/T140213#2579912 (10JGirault) a:03JGirault [18:37:31] (03CR) 10Bmansurov: [C: 032] Copy style attribute to lazy placeholder [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) (owner: 10Jdlrobson) [18:37:51] AaronSchulz: Yeah, looks sane. I don't know the code well enough to know if return; is the right thing to do in the new place too, but I guess testing will tell you [18:38:33] AaronSchulz: My main concern with making it earlier is that extensions hook into OutputPageLastModified, so you could potentially break an extension hook if it's trying to use things that aren't available yet [18:38:46] But I don't know if that's actually gonna happen because I haven't looked in detail [18:38:56] (03CR) 10Bmansurov: [C: 032] "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) (owner: 10Bmansurov) [18:39:08] Probably best to test it locally a bit, merge it, and then test it in beta labs for a week [18:39:10] (03PS1) 10Sbisson: Get the overfetched before bundling to avoid removing the base [extensions/Echo] - 10https://gerrit.wikimedia.org/r/306481 (https://phabricator.wikimedia.org/T143813) [18:39:21] (03CR) 10jenkins-bot: [V: 04-1] Copy style attribute to lazy placeholder [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) (owner: 10Jdlrobson) [18:40:19] (03PS1) 10Bearloga: Make title smaller & sort langs alphabetically after sorting by clicks [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306482 [18:41:02] (03CR) 10Bearloga: [C: 032 V: 032] Make title smaller & sort langs alphabetically after sorting by clicks [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306482 (owner: 10Bearloga) [18:42:21] (03CR) 10Smalyshev: [C: 032] Add boost-templates setup for relforge and enwiki [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306423 (owner: 10DCausse) [18:44:07] (03PS1) 10Mooeypoo: Followup on I75d0098e: Use getTopicTitleObj properly [extensions/Flow] - 10https://gerrit.wikimedia.org/r/306483 (https://phabricator.wikimedia.org/T143757) [18:44:16] RoanKattouw: everything in /extensions looks fine [18:44:33] the looks at user/session stuff [18:44:52] *they [18:45:06] (03CR) 10Catrope: [C: 032] Move Article checkLastModified() up to MediaWiki::performRequest [core] - 10https://gerrit.wikimedia.org/r/306334 (owner: 10Aaron Schulz) [18:45:17] 07German-Community-Wishlist-Main-Wishes, 06TCB-Team (Oct2016-March2017): When saving, be informed about mentions you sent (#9) - https://phabricator.wikimedia.org/T139962#2579953 (10Lea_WMDE) >I'm sorry, but I think we discovered complications with that idea. When we started working on the wish, we initially p... [18:45:42] AaronSchulz: Alright let's give it a shot then [18:46:15] too bad chrome doesn't send IMS to pages :( [18:46:20] (03PS1) 10Aklapper: Remove everything except for short information in PdfBook.php [extensions/PdfBook] - 10https://gerrit.wikimedia.org/r/306484 (https://phabricator.wikimedia.org/T124978) [18:46:28] RoanKattouw: if someone could figure out how to do that I'd get them a beer [18:46:47] It does [18:46:49] I think [18:46:57] 06WMF-Legal, 07Tracking: Copyright license and attribution issues (tracking) - https://phabricator.wikimedia.org/T87268#2579963 (10JMinor) [18:47:08] But [x] Disable cache is enabled by default when the inspector is open [18:47:23] that feature is not checked for me [18:47:52] (03Merged) 10jenkins-bot: Beta: clear the loading icon when a new search is started [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306446 (https://phabricator.wikimedia.org/T137068) (owner: 10Bmansurov) [18:47:52] I can hack max-age=100 to get (from cache) hits for a while [18:47:58] (03CR) 10Aklapper: [C: 04-1] "Not to be merged until T143684 has been sorted out." [extensions/PdfBook] - 10https://gerrit.wikimedia.org/r/306484 (https://phabricator.wikimedia.org/T124978) (owner: 10Aklapper) [18:48:03] but what I want is always-IMS [18:48:14] like FF [18:48:44] Hmm [18:48:57] opera is the same, since it steals the code [18:49:01] (03PS2) 10Mooeypoo: [wip] Followup on I75d0098e: Use getTopicTitleObj properly [extensions/Flow] - 10https://gerrit.wikimedia.org/r/306483 (https://phabricator.wikimedia.org/T143757) [18:49:07] 06Developer-Relations (Jul-Sep-2016), 13Patch-For-Review: Sort out PdfBook extension fork situation: Disable Wikimedia repo; link to GitHub repo - https://phabricator.wikimedia.org/T124978#2579992 (10Aklapper) >>! In T124978#2575674, @Aklapper wrote: > * No idea how to handle "Remove all files in a new commit... [18:49:09] sucks how slow it is in vagrant compared to FF [18:49:13] (03CR) 10Catrope: [C: 032] Get the overfetched before bundling to avoid removing the base [extensions/Echo] - 10https://gerrit.wikimedia.org/r/306481 (https://phabricator.wikimedia.org/T143813) (owner: 10Sbisson) [18:50:24] (03PS3) 10Mooeypoo: [wip] Followup on I75d0098e: Use getTopicTitleObj properly [extensions/Flow] - 10https://gerrit.wikimedia.org/r/306483 (https://phabricator.wikimedia.org/T143757) [18:52:55] 10MediaWiki-General-or-Unknown, 06Front-end-Standards-Group, 05MW-1.28-release-notes, 13Patch-For-Review, 05WMF-deploy-2016-08-16_(1.28.0-wmf.15): Start deprecation process to discourage use of various frontend ResourceLoader JS modules - https://phabricator.wikimedia.org/T142418#2580006 (10Aklapper) [18:54:16] (03PS2) 10Mholloway: Restore lead image gradient (alternate version) [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/306428 (https://phabricator.wikimedia.org/T135294) [18:54:25] (03PS1) 10Bearloga: Hide zero values [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306485 [18:54:31] (03CR) 10Bearloga: [C: 032 V: 032] Hide zero values [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306485 (owner: 10Bearloga) [18:54:44] (03PS1) 10Chad: Remove $wgTitle, unused [extensions/ErrorHandler] - 10https://gerrit.wikimedia.org/r/306486 [18:55:14] (03Merged) 10jenkins-bot: Add boost-templates setup for relforge and enwiki [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306423 (owner: 10DCausse) [18:55:16] (03Merged) 10jenkins-bot: Move Article checkLastModified() up to MediaWiki::performRequest [core] - 10https://gerrit.wikimedia.org/r/306334 (owner: 10Aaron Schulz) [18:55:36] (03PS1) 10Mattflaschen: Use psysh for Zend shell instead of phpsh [vagrant] - 10https://gerrit.wikimedia.org/r/306487 (https://phabricator.wikimedia.org/T123951) [18:55:45] (03Merged) 10jenkins-bot: Get the overfetched before bundling to avoid removing the base [extensions/Echo] - 10https://gerrit.wikimedia.org/r/306481 (https://phabricator.wikimedia.org/T143813) (owner: 10Sbisson) [18:57:37] (03PS1) 10Mattflaschen: Memcached PECL: Allow using binary protocol [core] - 10https://gerrit.wikimedia.org/r/306488 (https://phabricator.wikimedia.org/T143464) [18:59:24] (03CR) 10Jhobs: Add separators between footer links in beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [19:01:04] (03PS1) 10Mattflaschen: Use binary protocol with memcached [vagrant] - 10https://gerrit.wikimedia.org/r/306490 (https://phabricator.wikimedia.org/T143464) [19:02:13] (03CR) 10jenkins-bot: [V: 04-1] Use binary protocol with memcached [vagrant] - 10https://gerrit.wikimedia.org/r/306490 (https://phabricator.wikimedia.org/T143464) (owner: 10Mattflaschen) [19:02:33] (03PS4) 10Anomie: API: Use U+001F (Unit Separator) for separating multi-valued parameters [core] - 10https://gerrit.wikimedia.org/r/305126 (https://phabricator.wikimedia.org/T141960) [19:02:56] (03PS1) 10Anomie: API: Warn when input parameters are normalized [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) [19:04:00] (03CR) 10Mattflaschen: "After checking out this and the associated MW change, you can test by:" [vagrant] - 10https://gerrit.wikimedia.org/r/306490 (https://phabricator.wikimedia.org/T143464) (owner: 10Mattflaschen) [19:04:08] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 07Story: [Story] Versioning in JSON output - https://phabricator.wikimedia.org/T92961#2580052 (10Smalyshev) I'm looking at https://www.wikidata.org/wiki/Special:EntityData/Q42.json structure, and I see it is: ``` { "entities": { "Q42": { ...stuff.... [19:05:12] (03CR) 10Bmansurov: "Haven't been able to test locally yet." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306479 (https://phabricator.wikimedia.org/T143558) (owner: 10Jdlrobson) [19:06:17] (03CR) 10Jdlrobson: "any reason you dropped your -1? CI seems to be having issues right now and if you view the log it's not related." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [19:06:28] (03CR) 10Jdlrobson: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [19:07:38] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: Create a {{#statements:…}} parser function for human readable access to statements - https://phabricator.wikimedia.org/T142941#2580057 (10hoo) [19:07:53] 10MediaWiki-extensions-WikibaseClient, 10ArticlePlaceholder, 10Wikidata: {{#statements}} parser function should output EntityIdValues as local wiki links, once we have article placeholders - https://phabricator.wikimedia.org/T99897#2580058 (10hoo) [19:08:04] (03CR) 10jenkins-bot: [V: 04-1] API: Use U+001F (Unit Separator) for separating multi-valued parameters [core] - 10https://gerrit.wikimedia.org/r/305126 (https://phabricator.wikimedia.org/T141960) (owner: 10Anomie) [19:08:24] (03PS2) 10Mattflaschen: Use psysh for Zend shell instead of phpsh [vagrant] - 10https://gerrit.wikimedia.org/r/306487 (https://phabricator.wikimedia.org/T123951) [19:08:26] (03PS2) 10Mattflaschen: Use binary protocol with memcached [vagrant] - 10https://gerrit.wikimedia.org/r/306490 (https://phabricator.wikimedia.org/T143464) [19:08:33] 10ProofreadPage, 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wmf.16/extensions/ProofreadPage/ProofreadPage.body.php on line 444 - https://phabricator.wikimedia.org/T143817#2580059 (... [19:09:02] matt_flaschen: that's cool, I use psysh for Symfony projects, or when I need an out-of-framework REPL shell, I like it [19:09:36] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Improve user experience in search - https://phabricator.wikimedia.org/T137068#2580071 (10bmansurov) To test please visit https://en.m.wikipedia.beta.wmflabs.org/wiki/Main_Page?mobileaction=beta and click on the search bar... [19:09:52] Dereckson, yeah, I like it so far. tgr's suggestion. [19:12:03] (03CR) 10jenkins-bot: [V: 04-1] API: Warn when input parameters are normalized [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) (owner: 10Anomie) [19:12:17] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Improve user experience in search - https://phabricator.wikimedia.org/T137068#2580072 (10phuedx) {F4397570} [19:13:18] 10MediaWiki-extensions-General-or-Unknown, 10Revision-Slider, 06TCB-Team, 03TCB-Team-Sprint-2016-08-11, 03TCB-Team-Sprint-2016-08-25: Revision slider disconnects wikEdDiff - https://phabricator.wikimedia.org/T143199#2580073 (10Lea_WMDE) [19:14:03] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: Create mw.wikibase.entity:formatStatements for human readable access to statements - https://phabricator.wikimedia.org/T142942#2580074 (10hoo) [19:14:26] (03CR) 10Bmansurov: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) (owner: 10Jdlrobson) [19:14:30] (03CR) 10Bmansurov: [C: 032] Copy style attribute to lazy placeholder [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) (owner: 10Jdlrobson) [19:14:41] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: Create client functionality for getting human readable wikitext from Wikibase statements - https://phabricator.wikimedia.org/T142940#2580087 (10hoo) Updated the names for the new functionality listed in the sub-tickets according to a discussion we just had. [19:15:42] 07Design, 10Revision-Slider, 06TCB-Team, 06WMDE-Design, and 2 others: Mobile behavior: Revision slider may always only have one tooltip - https://phabricator.wikimedia.org/T143090#2580088 (10Lea_WMDE) [19:15:46] 06WMF-NDA-Requests: Develop data usage code - https://phabricator.wikimedia.org/T143819#2580089 (10I9606) [19:16:41] 10MediaWiki-Database, 10Wikidata, 07Wikimedia-log-errors: Wikibase\SqlIdGenerator::generateNewId: Explicit commit of implicit transaction. - https://phabricator.wikimedia.org/T143818#2580075 (10hashar) [19:17:31] 10MediaWiki-Database, 10Wikidata, 07Wikimedia-log-errors: Wikibase\SqlIdGenerator::generateNewId: Explicit commit of implicit transaction. - https://phabricator.wikimedia.org/T143818#2580075 (10hashar) [19:18:01] 10Wikimedia-Blog: Wikimedia Blog should provide full article syndication feeds (RSS) - https://phabricator.wikimedia.org/T143450#2568653 (10EdErhart-WMF) @CKoerner_WMF how does it look now? [19:21:07] (03Merged) 10jenkins-bot: Copy style attribute to lazy placeholder [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306451 (https://phabricator.wikimedia.org/T143768) (owner: 10Jdlrobson) [19:22:25] 10MediaWiki-extensions-LiquidThreads, 07I18n: Add missing documentation messages to LiquidThread - https://phabricator.wikimedia.org/T120129#2580128 (10Nemo_bis) p:05Lowest>03Normal [19:22:43] (03CR) 10Nemo bis: "check experimental" [extensions/LiquidThreads] - 10https://gerrit.wikimedia.org/r/306367 (owner: 10L10n-bot) [19:24:12] (03CR) 10Bmansurov: Footer / chrome color changes (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306472 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [19:25:12] (03CR) 10Nemo bis: "check experimental" [extensions/LiquidThreads] - 10https://gerrit.wikimedia.org/r/212819 (https://phabricator.wikimedia.org/T87935) (owner: 10Paladox) [19:25:21] 10ProofreadPage, 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wmf.16/extensions/ProofreadPage/ProofreadPage.body.php on line 444 - https://phabricator.wikimedia.org/T143817#2580132 (... [19:26:27] 10MediaWiki-extensions-LiquidThreads, 07I18n: Add missing documentation messages to LiquidThread - https://phabricator.wikimedia.org/T120129#2580150 (10Nemo_bis) [19:27:18] (03CR) 10Hashar: "Causes T143817" (031 comment) [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/305796 (owner: 10Dereckson) [19:27:22] 10MediaWiki-extensions-LiquidThreads: LQT: Call to a member function getText() on a non-object at Message.php - https://phabricator.wikimedia.org/T43186#2580151 (10Nemo_bis) p:05Lowest>03Normal [19:28:04] 10MediaWiki-extensions-LiquidThreads: LQT: Call to a member function subject() on a non-object - https://phabricator.wikimedia.org/T42798#2580159 (10Nemo_bis) p:05Lowest>03Normal [19:28:05] (03CR) 10Bmansurov: [C: 04-1] "According to the mock-up the separator should be a disc not a square." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [19:28:21] 10MediaWiki-extensions-LiquidThreads: LQT: Keep talk refactoring possible - https://phabricator.wikimedia.org/T31711#2580165 (10Nemo_bis) [19:28:23] 10MediaWiki-extensions-LiquidThreads: Warn when users reply to another person's post within the post itself by modifying it - https://phabricator.wikimedia.org/T35411#2580162 (10Nemo_bis) 05Open>03declined p:05Lowest>03Low > In the real world, this bug can be WONTIFIXed if you want. :) [19:28:47] hashar: I'm preparing a fix [19:28:56] Dereckson: nice [19:29:11] Dereckson: from the wmf logs, only the onArticlePurge has a mismatch [19:29:16] 10MediaWiki-extensions-LiquidThreads, 07I18n: "Restore deleted thread" page lacks subject - https://phabricator.wikimedia.org/T30746#2580166 (10Nemo_bis) p:05Lowest>03Low [19:29:20] so maybe that is the only one impacted [19:29:31] https://www.mediawiki.org/wiki/Manual:Hooks/ArticlePurge [19:29:43] but maybe other methods haven't been invoked yet and would suffer from the same mismatch of WikiPage vs Article [19:29:57] documentation is indeed Article (MW < 1.18) or WikiPage (MW 1.18+) [19:30:10] neat [19:30:16] ah [19:30:34] there is a wikisource on wmflabs? [19:31:11] if we really did something like ProofReadPage\onArticleUpdate implements MediaWiki\Hook\ArticleUpdate [19:31:18] we would have the mismatch caught earlier :] [19:31:23] 10ProofreadPage, 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wmf.16/extensions/ProofreadPage/ProofreadPage.body.php on line 444 - https://phabricator.wikimedia.org/T143817#2580176 (... [19:31:24] Dereckson: that is on prod [19:31:29] oh on beta [19:31:30] yeah en [19:31:42] https://en.wikisource.beta.wmflabs.org/wiki/Main_Page [19:31:48] yes I know, but it has been merged before to be in prod, so a check on beta should occur next time [19:31:51] ok [19:32:39] not sure whether logs on beta are properly tracked / triaiged [19:32:48] part of them certainly are but that is not exhaustive [19:33:17] idling for a few minutes will be back soon [19:33:32] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Improve user experience in search - https://phabricator.wikimedia.org/T137068#2580179 (10bmansurov) ^ 😄 [19:33:41] if you have a patch #mediawiki-core me or #wikimedia-releng can surely get it +2ed [19:33:43] Indeed, checked *onArticlePurge* on https://logstash-beta.wmflabs.org/ nothing [19:33:44] (03CR) 10Jdlrobson: "It is a disc on my OSX laptop." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [19:33:48] 10MediaWiki-extensions-LiquidThreads: LQT: Keep talk refactoring possible - https://phabricator.wikimedia.org/T31711#2580184 (10Nemo_bis) [19:33:57] I guess nothing is hitting proofreadpage :( [19:34:04] brb [19:34:33] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Align elements in article footer on beta - https://phabricator.wikimedia.org/T141002#2580186 (10Jdlrobson) @nirzar what symbol should i use for the separator in the footer links? I'm using `𐤟` but apparently this renders a... [19:34:41] 10MediaWiki-extensions-ConfirmAccount, 13Patch-For-Review: Deprecation warnings about Linker::makeKnownLinkObj, Linker::link, Linker::linkKnown, User::createNew and User::crypt - https://phabricator.wikimedia.org/T121145#2580187 (10Paladox) looks like it is all resolved now. [19:34:48] (03PS5) 10Anomie: API: Use U+001F (Unit Separator) for separating multi-valued parameters [core] - 10https://gerrit.wikimedia.org/r/305126 (https://phabricator.wikimedia.org/T141960) [19:35:03] 10MediaWiki-extensions-ConfirmAccount, 13Patch-For-Review: Deprecation warnings about Linker::makeKnownLinkObj, Linker::link, Linker::linkKnown, User::createNew and User::crypt - https://phabricator.wikimedia.org/T121145#2580189 (10Paladox) 05Open>03Resolved a:03Paladox [19:35:08] (03PS2) 10Anomie: API: Warn when input parameters are normalized [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) [19:35:30] (03PS1) 10Bartosz Dziewoński: mw.Title: Correct handling of Unicode whitespace and bidi control characters [core] - 10https://gerrit.wikimedia.org/r/306493 (https://phabricator.wikimedia.org/T143759) [19:36:32] (03CR) 10BryanDavis: [C: 032] Call ssl_set() in DatabaseMysqli if DBO_SSL is set [core] - 10https://gerrit.wikimedia.org/r/306026 (https://phabricator.wikimedia.org/T136218) (owner: 10Aaron Schulz) [19:37:08] (03PS1) 10Ladsgroup: Fix for purging scores [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306494 (https://phabricator.wikimedia.org/T143798) [19:37:38] 10MediaWiki-extensions-ConfirmAccount: Deprecation warnings about Linker::makeKnownLinkObj, Linker::link, Linker::linkKnown, User::createNew and User::crypt - https://phabricator.wikimedia.org/T121145#2580197 (10Paladox) [19:39:23] (03CR) 10Legoktm: "Well, it was mostly made useless in c2ce6a1b6085cf0cbf0fc2c3542fd36096bf2948, and I noted that in my CR that a followup was needed on that" [core] - 10https://gerrit.wikimedia.org/r/306325 (https://phabricator.wikimedia.org/T143531) (owner: 10Legoktm) [19:39:39] 06Discovery, 10Wikimedia-Portals, 03Discovery-Portal-Sprint, 13Patch-For-Review: Wikipedia.org Portal: add meta description in page source - https://phabricator.wikimedia.org/T143239#2580201 (10debt) The unfortunate part of adding in specific numbers > with more than 41,000,000 articles in 294 languages... [19:39:41] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 13Patch-For-Review, 15User-Ladsgroup: Update model versions is badly broken in ORES extension - https://phabricator.wikimedia.org/T143798#2579398 (10Ladsgroup) [19:39:43] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 13Patch-For-Review, 15User-Ladsgroup, 05WMF-deploy-2016-08-30_(1.28.0-wmf.17): Update model versions is badly broken in ORES extension - https://phabricator.wikimedia.org/T143799#2580205 (10Ladsgroup) [19:39:57] 10MediaWiki-extensions-ORES, 10ORES: Unclear header for ORES options in Preferences - https://phabricator.wikimedia.org/T143822#2580206 (10kaldari) [19:40:26] 10MediaWiki-extensions-ORES, 10ORES: Unclear header for ORES options in Preferences - https://phabricator.wikimedia.org/T143822#2580218 (10kaldari) [19:41:00] (03CR) 10jenkins-bot: [V: 04-1] API: Use U+001F (Unit Separator) for separating multi-valued parameters [core] - 10https://gerrit.wikimedia.org/r/305126 (https://phabricator.wikimedia.org/T141960) (owner: 10Anomie) [19:41:30] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service: Unclear header for ORES options in Preferences - https://phabricator.wikimedia.org/T143822#2580219 (10Ladsgroup) [19:41:31] SMalyshev: interesting in taking over https://gerrit.wikimedia.org/r/#/c/288462/ ? E.g. following lego's comments? [19:42:00] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service: Unclear header for ORES options in Preferences - https://phabricator.wikimedia.org/T143822#2580206 (10Ladsgroup) Hey, do you mean in preferences or beta features part? [19:42:01] 10ProofreadPage, 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wmf.16/extensions/ProofreadPage/ProofreadPage.body.php on line 444 - https://phabricator.wikimedia.org/T143817#2580221 (... [19:42:37] (03CR) 10Legoktm: [C: 032] Fix for purging scores [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306494 (https://phabricator.wikimedia.org/T143798) (owner: 10Ladsgroup) [19:43:26] AaronSchulz: I started implementing your SecondaryDataUpdates suggestion at https://gerrit.wikimedia.org/r/#/c/306314/. I'm a bit lost on how to use EnqueueableDataUpdate though (especially getAsJobSpecification). There's pretty much no documentation for any of it, so I've just been cobbling it together based mostly on what's in LinksUpdate. If you could [19:43:26] take a look and give me some advice on getting that working, it would be much appreciated. [19:43:31] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 07Story: [Story] Versioning in JSON output - https://phabricator.wikimedia.org/T92961#2580224 (10daniel) @Smalyshev Oh right, I forgot that we do already output an envelope from Special:EntityData! That makes things easier. For the dump, yea, we have th... [19:44:01] (03PS1) 10Mobrovac: Update change-propagation to 94aa503 [services/change-propagation/deploy] - 10https://gerrit.wikimedia.org/r/306495 [19:44:03] (03CR) 10jenkins-bot: [V: 04-1] API: Warn when input parameters are normalized [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) (owner: 10Anomie) [19:44:26] kaldari: so you took my less preferred, more complicated, solution? ;) [19:44:44] (03Abandoned) 10Mobrovac: Update change-propagation to 94aa503 [services/change-propagation/deploy] - 10https://gerrit.wikimedia.org/r/306495 (owner: 10Mobrovac) [19:44:55] (03CR) 10Phuedx: "You're right that it's overkill but it's a stepping stone to the MobileContext god object, which can then be split apart more easily yes, although I also started on the simpler solution in a separate branch in case I can't figure out the more complicated solution :) [19:45:12] 10MediaWiki-extensions-EducationProgram, 10GlobalUserPage, 10VisualEditor, 10VisualEditor-MediaWiki: VisualEditor redlink detection should obey Title::isAlwaysKnown (so that links to Special pages, EducationProgram pages and GlobalUserPages render correctly) - https://phabricator.wikimedia.org/T72123#2580228... [19:45:32] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 07Story: [Story] Versioning in JSON output - https://phabricator.wikimedia.org/T92961#2580231 (10Smalyshev) Metadata file maybe would be fine too, then it should look the same as the API format without `entities`, I think, e.g. we'd have: ``` { "@meta... [19:45:50] 10ProofreadPage, 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wmf.16/extensions/ProofreadPage/ProofreadPage.body.php on line 444 - https://phabricator.wikimedia.org/T143817#2580232 (... [19:45:53] 10MediaWiki-General-or-Unknown, 06translatewiki.net, 07I18n, 05MW-1.27-release-notes: Translation bot broke core tests - https://phabricator.wikimedia.org/T131981#2185018 (10Anomie) Tests are broken again today. This time because `userjsispublic` and `usercssispublic` have gone missing in {b3108317}. [19:45:59] 10MediaWiki-General-or-Unknown, 06translatewiki.net, 07I18n, 05MW-1.27-release-notes: Translation bot broke core tests - https://phabricator.wikimedia.org/T131981#2580236 (10Anomie) 05Resolved>03Open [19:46:01] Dereckson: should we revert the type hints patch and revisit later ? [19:46:03] (03Merged) 10jenkins-bot: Call ssl_set() in DatabaseMysqli if DBO_SSL is set [core] - 10https://gerrit.wikimedia.org/r/306026 (https://phabricator.wikimedia.org/T136218) (owner: 10Aaron Schulz) [19:46:07] (03Merged) 10jenkins-bot: Fix for purging scores [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306494 (https://phabricator.wikimedia.org/T143798) (owner: 10Ladsgroup) [19:46:31] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 07Story: [Story] Versioning in JSON output - https://phabricator.wikimedia.org/T92961#2580240 (10daniel) >>! In T92961#2579283, @Pfps wrote: > Well I think that it should be a breaking change, but I read the stable interface policy as saying that it isn'... [19:46:36] (03PS1) 10Mobrovac: Update change-propagation to 85aa806 [services/change-propagation/deploy] - 10https://gerrit.wikimedia.org/r/306496 [19:47:06] (03CR) 10Mobrovac: [C: 032 V: 032] Update change-propagation to 85aa806 [services/change-propagation/deploy] - 10https://gerrit.wikimedia.org/r/306496 (owner: 10Mobrovac) [19:47:08] (03PS1) 10Dereckson: Fix hooks signatures [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306497 [19:47:11] AaronSchulz: I'm a bit lost on what EnqueueableDataUpdate::doUpdate and EnqueueableDataUpdate::getAsJobSpecification are for and how they work [19:48:42] (03CR) 10jenkins-bot: [V: 04-1] mw.Title: Correct handling of Unicode whitespace and bidi control characters [core] - 10https://gerrit.wikimedia.org/r/306493 (https://phabricator.wikimedia.org/T143759) (owner: 10Bartosz Dziewoński) [19:48:44] do I need to have both of those or just one? [19:48:59] doUpdate() runs in DeferredUpates::doUpdates( 'run' ) [19:49:05] (03PS2) 10Dereckson: Fix hooks signatures [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306497 (https://phabricator.wikimedia.org/T143817) [19:49:29] getAsJobSpecification() causes jobs to be pushed in DeferredUpates::doUpdates( 'enqueue') [19:49:50] the jobs end up running DeferredUpates::doUpdates( 'run' ) [19:49:54] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 13Patch-For-Review, 15User-Ladsgroup: Edits being flagged by review tool on enwiki aren't likely to be damaging - https://phabricator.wikimedia.org/T143738#2580263 (10hashar) Well done @Ladsgroup [19:50:04] AaronSchulz: ah ok [19:50:37] Dereckson: will merge [19:50:52] kaldari: it's kind of a result of legacy though [19:50:55] (03PS1) 10Hashar: Fix hooks signatures [extensions/ProofreadPage] (wmf/1.28.0-wmf.16) - 10https://gerrit.wikimedia.org/r/306498 (https://phabricator.wikimedia.org/T143817) [19:51:15] 06Wikipedia-Android-App-Backlog, 07Mobile: Some users can't login or edit without proxy in Iran - https://phabricator.wikimedia.org/T142309#2530722 (10Dbrant) Is the user able to log in and/or edit on mobile web (instead of the app) without a proxy? We're not sure that this is an issue specifically with the app. [19:51:31] (03PS3) 10Dereckson: Fix hooks signatures [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306497 [19:51:41] I mean it could have been where Jobs classes were made and run() called in defferedupdates or JobRunner (if registered in $wgJobClasses for the later) [19:52:47] instead there is this round about interaction between two classes that both represent jobs [19:52:51] 10MediaWiki-Page-deletion, 03Collab-Team-Q1-July-Sep-2016, 05MW-1.28-release-notes, 13Patch-For-Review, 05WMF-deploy-2016-08-16_(1.28.0-wmf.15): Failed restore and data loss on ar_rev_id collisions - https://phabricator.wikimedia.org/T135852#2580299 (10Etonkovidova) Re-checked both scenarios - the resul... [19:53:17] (03PS3) 10Anomie: API: Warn when input parameters are normalized [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) [19:53:18] historically DeferrableUpdates are not serializable where as jobs are [19:53:42] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 13Patch-For-Review, 15User-Ladsgroup: Edits being flagged by review tool on enwiki aren't likely to be damaging - https://phabricator.wikimedia.org/T143738#2580318 (10Ladsgroup) Thanks @hashar :) [19:54:06] (03PS2) 10Hashar: Fix hooks signatures [extensions/ProofreadPage] (wmf/1.28.0-wmf.16) - 10https://gerrit.wikimedia.org/r/306498 (https://phabricator.wikimedia.org/T143817) [19:54:07] jobs can start with arrays of params and use the service singleton(s) to get the stores needed to do things [19:54:43] AaronSchulz: Wouldn't it be simpler for me to just create some jobs directly from the hook. Seems like a lot of extra machinations. [19:54:48] (03CR) 10Hashar: "Synced with change in master https://gerrit.wikimedia.org/r/#/c/306497/3 (PS3) which adds onArticleDelete" [extensions/ProofreadPage] (wmf/1.28.0-wmf.16) - 10https://gerrit.wikimedia.org/r/306498 (https://phabricator.wikimedia.org/T143817) (owner: 10Hashar) [19:55:28] kaldari: my first suggestion was to just hook into refreshlinks with some methods that do the updates right there [19:55:45] onLinksUpdateComplete( ... ) {} [19:55:59] (03PS1) 10Ladsgroup: Remove deprecated config [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306500 [19:56:06] each one will change like 1 row or something, right? [19:56:12] 10MediaWiki-extensions-WikibaseRepository, 06Commons, 10Wikidata, 07Story, 15User-Ladsgroup: [Story] Add a new datatype for linking to creators of artwork and more (smart URI) - https://phabricator.wikimedia.org/T127929#2580320 (10daniel) >>! In T127929#2567914, @Izno wrote: > Are we trying to avoid intr... [19:56:25] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 07Story: [Story] Versioning in JSON output - https://phabricator.wikimedia.org/T92961#2580325 (10daniel) >>! In T92961#2580231, @Smalyshev wrote: > Metadata file maybe would be fine too, then it should look the same as the API format without `entities`,... [19:56:28] and JobRunner has lots of lag checks [19:56:29] AaronSchulz: yeah, or 2 rows at worst [19:56:35] that seems very simple to me [19:56:40] 10MediaWiki-General-or-Unknown, 06translatewiki.net, 07I18n, 05MW-1.27-release-notes: Translation bot broke core tests - https://phabricator.wikimedia.org/T131981#2580329 (10Raymond) Fixed now https://gerrit.wikimedia.org/r/#/c/306499/1/languages/i18n/qqq.json with as latest regular export. [19:56:55] (03CR) 10Bartosz Dziewoński: "I am still unhappy about this and the parent commit. But assuming I lost this battle, I left some comments inline." (032 comments) [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) (owner: 10Anomie) [19:57:06] Dereckson: good to go with the 3 signature updates? [19:57:14] * Dereckson nods [19:57:24] (03CR) 10Hashar: [C: 032] Fix hooks signatures [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306497 (owner: 10Dereckson) [19:57:34] and the wmf one is https://gerrit.wikimedia.org/r/#/c/306498/2 [19:57:48] (03CR) 10Hashar: [C: 032] Fix hooks signatures [extensions/ProofreadPage] (wmf/1.28.0-wmf.16) - 10https://gerrit.wikimedia.org/r/306498 (https://phabricator.wikimedia.org/T143817) (owner: 10Hashar) [19:57:55] will deploy it in a few [19:58:39] AaronSchulz: So maybe just spawning some jobs off of LinksUpdateComplete hook would be best? [19:58:46] (03Merged) 10jenkins-bot: Fix hooks signatures [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306497 (owner: 10Dereckson) [19:58:57] (03Merged) 10jenkins-bot: Fix hooks signatures [extensions/ProofreadPage] (wmf/1.28.0-wmf.16) - 10https://gerrit.wikimedia.org/r/306498 (https://phabricator.wikimedia.org/T143817) (owner: 10Hashar) [20:00:30] kaldari: ??? [20:00:58] AaronSchulz: I still prefer using jobs since these are super low-priority data updates [20:01:00] I'm saying to just have a function fired by that hook that get the DB and updates it [20:01:15] I don't think it's worth the extra code [20:01:42] hmmm, maybe you're right [20:01:48] it's basically a one-off big edit [20:02:21] yes [20:03:14] OK, I'm going to try that approach and keep my fingers crossed [20:03:46] 10ProofreadPage, 13Patch-For-Review, 05WMF-deploy-2016-08-23_(1.28.0-wmf.16), 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wm... - https://phabricator.wikimedia.org/T143817#2580355 [20:03:49] If you ever feel like writing some documentation for how to use EnqueueableDataUpdate though, that might be useful for the future ;) [20:03:50] think about when people add a category to a big template, it's like the same thing, and extra row per a crap load of backlinks [20:04:18] AaronSchulz: but isn't that handled via a job? [20:05:10] I actually don't know, I'm just guessing [20:05:40] refreshLinksJob [20:06:20] AaronSchulz: but you're saying that even if it wasn't handled by a job, it wouldn't cause any problems? [20:06:21] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Align elements in article footer on beta - https://phabricator.wikimedia.org/T141002#2580359 (10bmansurov) Can confirm the above: {F4397722} [20:06:35] 06Reading-Web-Backlog, 07Epic: [EPIC] Mobile language switching improvement work II - https://phabricator.wikimedia.org/T137932#2580361 (10JKatzWMF) [20:06:37] 07Design, 10MobileFrontend, 06Reading-Web-Backlog: When switching languages on mobile web, allow search for a language in any language - https://phabricator.wikimedia.org/T137680#2580360 (10JKatzWMF) [20:07:43] (03PS2) 10Bartosz Dziewoński: mw.Title: Correct handling of Unicode whitespace and bidi control characters [core] - 10https://gerrit.wikimedia.org/r/306493 (https://phabricator.wikimedia.org/T143759) [20:07:52] (03PS6) 10Anomie: API: Use U+001F (Unit Separator) for separating multi-valued parameters [core] - 10https://gerrit.wikimedia.org/r/305126 (https://phabricator.wikimedia.org/T141960) [20:07:59] (03PS3) 10Bartosz Dziewoński: mw.Title: Correct handling of Unicode whitespace and bidi control characters [core] - 10https://gerrit.wikimedia.org/r/306493 (https://phabricator.wikimedia.org/T143759) [20:09:46] kaldari: well hooking into refreshLinks basically means that refreshLinksJob is handling it [20:10:23] (03CR) 10Krinkle: [C: 032] Set right MessagesDirs [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306403 (https://phabricator.wikimedia.org/T143770) (owner: 10Phedenskog) [20:10:38] (03CR) 10Ladsgroup: [C: 032] Remove deprecated config [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306500 (owner: 10Ladsgroup) [20:10:42] (03CR) 10Krinkle: [C: 032] Remove unused message keys in mustache [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306406 (https://phabricator.wikimedia.org/T143770) (owner: 10Phedenskog) [20:12:24] 04(no projects): ResearchSpike: Pivot UI: react, playwood - https://phabricator.wikimedia.org/T143828#2580378 (10Nuria) [20:12:45] (03Merged) 10jenkins-bot: Set right MessagesDirs [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306403 (https://phabricator.wikimedia.org/T143770) (owner: 10Phedenskog) [20:12:47] (03Merged) 10jenkins-bot: Remove deprecated config [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306500 (owner: 10Ladsgroup) [20:12:52] AaronSchulz: I'm still confused. Don't I need my own job class to define what the job actually does? Any chance I could sit down with you Friday or next week in person and discuss in more detail? [20:13:33] I don't think you need a job class, just a regular old hook handler for that hook in LinksUpdate [20:14:13] (03CR) 10Krinkle: [C: 032] Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds [core] - 10https://gerrit.wikimedia.org/r/305952 (owner: 10Aaron Schulz) [20:14:38] 06Reading-Web-Backlog, 07Epic: Remove bottom button - https://phabricator.wikimedia.org/T143829#2580393 (10JKatzWMF) [20:14:54] 06Reading-Web-Backlog, 07Epic: [EPIC] Mobile language switching improvement work II - https://phabricator.wikimedia.org/T137932#2384157 (10JKatzWMF) On reflection, T130849 seems to be something that should happen as a last step before closing this Epic--but probably can't happen. And we will need to remove th... [20:15:13] (03PS1) 10Bearloga: Note language order in legend [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306517 (https://phabricator.wikimedia.org/T140816) [20:15:20] (03CR) 10Bearloga: [C: 032 V: 032] Note language order in legend [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306517 (https://phabricator.wikimedia.org/T140816) (owner: 10Bearloga) [20:15:35] (03Merged) 10jenkins-bot: Remove unused message keys in mustache [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306406 (https://phabricator.wikimedia.org/T143770) (owner: 10Phedenskog) [20:16:02] (03PS2) 10Krinkle: Use i18n messages for size [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306411 (https://phabricator.wikimedia.org/T143770) (owner: 10Phedenskog) [20:16:11] 10ProofreadPage, 13Patch-For-Review, 05WMF-deploy-2016-08-23_(1.28.0-wmf.16), 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wm... - https://phabricator.wikimedia.org/T143817#2580418 [20:16:33] (03CR) 10Anomie: API: Warn when input parameters are normalized (032 comments) [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) (owner: 10Anomie) [20:16:36] (03CR) 10Krinkle: "Manual rebase (no conflict, but Gerrit wasn't able to for some reason)." [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306411 (https://phabricator.wikimedia.org/T143770) (owner: 10Phedenskog) [20:17:06] (03PS4) 10Anomie: API: Warn when input parameters are normalized [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) [20:17:46] AaronSchulz: So is that hook itself fired from a job? [20:17:52] (03CR) 10Krinkle: "There's one more for the local storage size." [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306411 (https://phabricator.wikimedia.org/T143770) (owner: 10Phedenskog) [20:18:46] (03PS5) 10Anomie: API: Warn when input parameters are normalized [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) [20:20:39] (03CR) 10Ladsgroup: Factor out Cache::$classMap into $wgOresModelClasses (033 comments) [extensions/ORES] - 10https://gerrit.wikimedia.org/r/306316 (owner: 10Catrope) [20:20:52] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service: Unclear header for ORES options in Preferences - https://phabricator.wikimedia.org/T143822#2580422 (10kaldari) In Recent Changes preferences (and just the header). [20:21:15] (03Merged) 10jenkins-bot: Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds [core] - 10https://gerrit.wikimedia.org/r/305952 (owner: 10Aaron Schulz) [20:22:52] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 07Easy, 15User-Ladsgroup: Unclear header for ORES options in Preferences - https://phabricator.wikimedia.org/T143822#2580423 (10Ladsgroup) p:05Triage>03Normal a:03Ladsgroup [20:23:00] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 07Easy, 15User-Ladsgroup: Unclear header for ORES options in Preferences - https://phabricator.wikimedia.org/T143822#2580428 (10Ladsgroup) I do it :) [20:23:44] (03CR) 10jenkins-bot: [V: 04-1] API: Warn when input parameters are normalized [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) (owner: 10Anomie) [20:25:24] kaldari: yep [20:25:39] AaronSchulz: AH! Everything makes sense now :) [20:25:51] the job gets the secondary updates for the wikipage and then calls DataUpdates::runUpdates [20:27:23] (03PS6) 10Anomie: API: Warn when input parameters are normalized [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) [20:28:27] AaronSchulz: OK, I'll just follow you're original suggestion then. I didn't realize that LinksUpdateComplete was fired from the LinkUpdate jobs. I think I was confused by the fact that LinksUpdate::doUpdate() spawns it's own jobs, but I guess that's because there's some job recursion going on. [20:31:19] 10MediaWiki-Database, 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 07Wikimedia-log-errors: Wikibase\SqlIdGenerator::generateNewId: Explicit commit of implicit transaction. - https://phabricator.wikimedia.org/T143818#2580447 (10greg) [20:33:56] 10MediaWiki-Database, 10MediaWiki-extensions-WikibaseClient, 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, and 2 others: Wikibase\SqlIdGenerator::generateNewId: Explicit commit of implicit transaction. - https://phabricator.wikimedia.org/T143818#2580453 (10hoo) a:03hoo [20:39:15] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Align elements in article footer on beta - https://phabricator.wikimedia.org/T141002#2580461 (10Nirzar) • ^ has the most support. if not then you can use this http://www.fileformat.info/info/unicode/char/2022/index.... [20:39:36] 06Wikipedia-Android-App-Backlog, 07Mobile: Some users can't login or edit without proxy in Iran - https://phabricator.wikimedia.org/T142309#2580462 (10Ladsgroup) The user says the web view also have issues: {F4397803} He says without proxy it stuck at this stage but when proxy enabled. It works without any iss... [20:40:20] (03CR) 10Bartosz Dziewoński: "On actual testing, this behaves unexpectedly when the input title is both non-NFC and non-canonical. There is one entry for NFC normalizat" [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) (owner: 10Anomie) [20:42:02] 10MediaWiki-ResourceLoader: Shibboleth Authentication isn't working due to missing ResourceLoaderUserGroupsModule class - https://phabricator.wikimedia.org/T143782#2578823 (10Krinkle) @Urbanecm It looks like you're running an alpha version of MediaWiki 1.28 with a version of the Shibboleth Authentication extensi... [20:42:16] 10MediaWiki-ResourceLoader: Shibboleth Authentication isn't working due to missing ResourceLoaderUserGroupsModule class - https://phabricator.wikimedia.org/T143782#2580471 (10Krinkle) p:05Triage>03Normal [20:42:46] (03PS1) 10Dereckson: Fix ProofreadPage::updatePrIndex signature [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306545 [20:43:30] (03PS2) 10Dereckson: Fix ProofreadPage::updatePrIndex signature [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306545 [20:44:48] kaldari: those jobs are for backlinks, the rest of the code is for the title that explicitly changed [20:45:13] 10MediaWiki-extensions-CentralNotice, 10Fundraising-Backlog, 03Fundraising Sprint Octopus Untangling, 13Patch-For-Review: Convert CentralNotice to use extension registration - https://phabricator.wikimedia.org/T108721#2580474 (10DStrine) [20:45:17] so it makes jobs for the backlinks that work on each of those titles that are *not* themselves recursive [20:45:39] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Improve user experience in search - https://phabricator.wikimedia.org/T137068#2580475 (10Nirzar) RIP flicker :) The spinner has an issue in the implementation. partially i didn't cover this in my mocks. The spinner shou... [20:45:41] 06Discovery, 03Discovery-Search-Sprint: Search Results page: re-design page display of search results across languages - https://phabricator.wikimedia.org/T139310#2580476 (10debt) [20:47:19] 10MediaWiki-extensions-CentralNotice, 10Fundraising-Backlog, 03Fundraising Sprint Octopus Untangling, 13Patch-For-Review, 07Spike: Spike: Monitor deployment rolling back our "googleoff" tag - https://phabricator.wikimedia.org/T137761#2580482 (10DStrine) [20:47:20] (03CR) 10Anomie: "> There is one entry for NFC normalization and one for canonical title normalization. One can still make use of it (it just required follo" [core] - 10https://gerrit.wikimedia.org/r/306491 (https://phabricator.wikimedia.org/T29849) (owner: 10Anomie) [20:47:34] 10WMUA-Partnerships: Wikipedia Education Program Training in Kyiv - https://phabricator.wikimedia.org/T141872#2580484 (10Yury_Bulka) [20:48:23] 10MediaWiki-ResourceLoader: Shibboleth Authentication isn't working due to missing ResourceLoaderUserGroupsModule class - https://phabricator.wikimedia.org/T143782#2580485 (10Krinkle) @Urbanecm I took a look at the Shibboleth Authentication extension, but I do not see any reference to the `ResourceLoaderUserGrou... [20:49:48] (03PS1) 10Bartosz Dziewoński: mw.widgets.CategoryCapsuleItemWidget: Handle titles not normalized in JS code [core] - 10https://gerrit.wikimedia.org/r/306547 (https://phabricator.wikimedia.org/T139130) [20:49:54] 10MediaWiki-extensions-DonationInterface, 10Fundraising-Backlog, 03Fundraising Sprint Octopus Untangling, 13Patch-For-Review: Donations message with empty body! - https://phabricator.wikimedia.org/T140515#2580491 (10DStrine) [20:50:04] (03PS2) 10Bartosz Dziewoński: mw.widgets.CategoryCapsuleItemWidget: Handle titles not normalized in JS code [core] - 10https://gerrit.wikimedia.org/r/306547 (https://phabricator.wikimedia.org/T139130) [20:51:23] 10MediaWiki-General-or-Unknown, 07JavaScript: JS mw.Title does not normalize input to NFC form - https://phabricator.wikimedia.org/T143810#2580494 (10matmarex) p:05Triage>03Lowest [20:51:32] 10MediaWiki-General-or-Unknown, 07JavaScript, 13Patch-For-Review: JS mw.Title does not strip Unicode bidi control characters from input, unlike PHP MediaWikiTitleCodec - https://phabricator.wikimedia.org/T143759#2580495 (10matmarex) p:05Triage>03Normal a:03matmarex [20:51:38] 10WMUA-Partnerships: Interviews with Wikipedians who teach - https://phabricator.wikimedia.org/T143830#2580497 (10Yury_Bulka) [20:52:15] (03PS3) 10EBernhardson: Ensure timeouts are always used [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) [20:52:34] 10MediaWiki-extensions-DonationInterface, 10FR-Smashpig, 10Fundraising-Backlog, 07Spike: Spike: Reconcile SmashPig and DonationInterface configuration - https://phabricator.wikimedia.org/T143831#2580512 (10awight) [20:52:48] 06Discovery, 06Discovery-Analysis (Current work): Wikipedia.org Portal Dashboard: add "other" to pageviews page - https://phabricator.wikimedia.org/T143605#2580526 (10mpopov) @ksmith mentioned that if we could see a histogram of pageviews per IP address, that would help us come to consensus on the appropriate... [20:53:44] 10MediaWiki-extensions-WikibaseClient, 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: [bug] sitelink on Wikidata not removed when page on client is deleted - https://phabricator.wikimedia.org/T143485#2580529 (10hoo) This indeed happened because [[https://meta.wikimedia.org/wiki/Special:CentralAuth/Pe... [20:56:02] (03PS4) 10Jdlrobson: Add separators between footer links in beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) [20:57:42] (03CR) 10jenkins-bot: [V: 04-1] Ensure timeouts are always used [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) (owner: 10EBernhardson) [20:58:08] 10MediaWiki-extensions-WikibaseRepository, 06Commons, 10Wikidata, 07Story, 15User-Ladsgroup: [Story] Add a new datatype for linking to creators of artwork and more (smart URI) - https://phabricator.wikimedia.org/T127929#2580534 (10Izno) >>! In T127929#2580320, @daniel wrote: > daniel added a subscriber:... [20:58:19] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 03Reading-Web-Sprint-79-Uh-oh, 05WMF-deploy-2016-08-30_(1.28.0-wmf.17): Increase the size of Wikipedia wordmark in footer - https://phabricator.wikimedia.org/T143832#2580535 (10Nirzar) [20:58:26] (03CR) 10EBernhardson: [C: 032] Use the UserTesting framework in maint scripts [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306169 (owner: 10DCausse) [20:59:27] (03PS4) 10EBernhardson: Ensure timeouts are always used [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) [21:00:00] (03CR) 10Jdlrobson: Footer / chrome color changes (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306472 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [21:00:11] (03PS5) 10Jdlrobson: Add separators between footer links in beta [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) [21:00:52] 10MediaWiki-extensions-MapSources, 06Discovery, 10Kartographer, 06Maps, 03Maps-Sprint: Deprecate provided by MapSources in favor of Kartographer's - https://phabricator.wikimedia.org/T136768#2580559 (10MaxSem) I've attempted it on en: https://en.wikivoyage.org/w/index.php?title=Wik... [21:00:56] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Align elements in article footer on beta - https://phabricator.wikimedia.org/T141002#2580560 (10Jdlrobson) I've used the bull: https://gerrit.wikimedia.org/r/#/c/306473/ Also replied to Baha's comment on other patch: http... [21:03:37] (03CR) 10jenkins-bot: [V: 04-1] Ensure timeouts are always used [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) (owner: 10EBernhardson) [21:04:25] 10MediaWiki-Configuration, 05MW-1.28-release-notes, 13Patch-For-Review, 07Upstream, 05WMF-deploy-2016-08-30_(1.28.0-wmf.17): ExtensionRegistration schema: An array of valid value types is not respected in error message - https://phabricator.wikimedia.org/T141281#2580568 (10Florian) 05Open>03Resolved [21:07:13] (03Merged) 10jenkins-bot: Use the UserTesting framework in maint scripts [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306169 (owner: 10DCausse) [21:09:15] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 03Wikidata-Sprint-2016-08-16: Data access in user language doesn't obey the uselang get parameter - https://phabricator.wikimedia.org/T142906#2580575 (10hoo) 05Open>03Resolved [21:10:03] (03CR) 10Cindy-the-browser-test-bot: [C: 031] "Cindy says good job \o/. Keep it up." [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) (owner: 10EBernhardson) [21:10:36] (03CR) 10Cindy-the-browser-test-bot: "Phan static analysis done without errors!" [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) (owner: 10EBernhardson) [21:13:26] (03CR) 10Anomie: mw.widgets.CategoryCapsuleItemWidget: Handle titles not normalized in JS code (032 comments) [core] - 10https://gerrit.wikimedia.org/r/306547 (https://phabricator.wikimedia.org/T139130) (owner: 10Bartosz Dziewoński) [21:20:06] (03CR) 10Aaron Schulz: [C: 032] Use binary protocol with memcached [vagrant] - 10https://gerrit.wikimedia.org/r/306490 (https://phabricator.wikimedia.org/T143464) (owner: 10Mattflaschen) [21:21:39] (03PS5) 10EBernhardson: Ensure timeouts are always used [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) [21:22:33] (03PS4) 10Mooeypoo: Followup on I75d0098e: Use getTopicTitleObj properly [extensions/Flow] - 10https://gerrit.wikimedia.org/r/306483 (https://phabricator.wikimedia.org/T143757) [21:23:19] 10MobileFrontend, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 13Patch-For-Review, and 3 others: [SPIKE: 1 hour] Avoid unintended wrapping on en.m.wikipedia.org "2016 Formula One season" article - https://phabricator.wikimedia.org/T143558#2580603 (10TheDJ) > Apparently there is a 1 pixel border... [21:27:08] 10MediaWiki-extensions-ORES, 10Revision-Scoring-As-A-Service-Backlog: Inject ores scores to rc-data - https://phabricator.wikimedia.org/T143611#2573377 (10Legoktm) Errrr, I'm confused. Can you explain the problem that needs fixing? [21:27:26] 10MobileFrontend, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 13Patch-For-Review, and 3 others: [SPIKE: 1 hour] Avoid unintended wrapping on en.m.wikipedia.org "2016 Formula One season" article - https://phabricator.wikimedia.org/T143558#2580606 (10TheDJ) In general however, messing with the pos... [21:32:35] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Align elements in article footer on beta - https://phabricator.wikimedia.org/T141002#2580608 (10Nirzar) >@MBinder_WMF this has easily morphed into a 5 point card at least. I agree Retro Item: can we analyze the reason be... [21:32:41] (03PS1) 10EBernhardson: Add flag for disabling phrase suggester [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306551 (https://phabricator.wikimedia.org/T143260) [21:33:05] 07Design, 10MobileFrontend, 06Reading-Web-Backlog: Increase the size of Wikipedia wordmark in footer - https://phabricator.wikimedia.org/T143832#2580612 (10Nirzar) [21:33:22] 07Design, 10MobileFrontend, 06Reading-Web-Backlog: Increase the size of Wikipedia wordmark in footer - https://phabricator.wikimedia.org/T143832#2580535 (10Nirzar) p:05High>03Normal [21:34:04] 10CirrusSearch, 06Discovery, 03Discovery-Search-Sprint: investigate rise in cirrusSearchLinksUpdatePrioritized job abandonment - https://phabricator.wikimedia.org/T143274#2562584 (10EBernhardson) a:03EBernhardson [21:37:27] (03CR) 10Gergő Tisza: "Is there a reason it's Zend-only? PsySH works with HHVM now: https://github.com/bobthecow/psysh/issues/81" [vagrant] - 10https://gerrit.wikimedia.org/r/306487 (https://phabricator.wikimedia.org/T123951) (owner: 10Mattflaschen) [21:38:35] 07Design, 10Revision-Slider, 06TCB-Team, 06WMDE-Design, and 2 others: Mobile behavior: Revision slider may always only have one tooltip - https://phabricator.wikimedia.org/T143090#2580622 (10Lea_WMDE) update from extension talk pae: Please check the behavior of the revision slider on a notebook with a touc... [21:38:38] 06Design-Research, 10Reading-UX-Research: Plan study for new Read More algorithm - https://phabricator.wikimedia.org/T142009#2580623 (10Tnegrin) Don't know how useful this will be but Discovery has a tool that can be used for humans to test search results: https://discernatron.wmflabs.org/login Code is here:... [21:39:16] 10Wikimedia-General-or-Unknown: Investigate replacements for Google Hangouts - https://phabricator.wikimedia.org/T143247#2580626 (10Pine) Brendan's previous explanation was about audio echoes. My question from my previous post is about dropped audio. [21:40:11] 07Design, 06Discovery, 10Kartographer, 06Maps, 03Maps-Sprint: Override leaflet style to match wikimedia style - https://phabricator.wikimedia.org/T143833#2580627 (10JGirault) [21:44:07] (03CR) 10Cindy-the-browser-test-bot: [C: 031] "Cindy says good job \o/. Keep it up." [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) (owner: 10EBernhardson) [21:44:38] (03CR) 10Cindy-the-browser-test-bot: "Phan static analysis done without errors!" [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306306 (https://phabricator.wikimedia.org/T143006) (owner: 10EBernhardson) [21:45:06] (03PS2) 10Aaron Schulz: Optimize ChronologyProtector writes for multi-DC case [core] - 10https://gerrit.wikimedia.org/r/305944 [21:45:10] (03PS1) 10Aaron Schulz: Cleanups to SqlBagOStuff [core] - 10https://gerrit.wikimedia.org/r/306553 [21:45:12] (03PS1) 10Aaron Schulz: [WIP] Add support for masking the WRITE_SYNC latency from ChronologyProtector [core] - 10https://gerrit.wikimedia.org/r/306554 [21:45:46] (03PS4) 10Hoo man: Make Scribunto_LuaSandboxTests::testArgumentParsingTime more robust [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/305649 (https://phabricator.wikimedia.org/T143389) [21:46:48] Krinkle: did you see my comments on https://gerrit.wikimedia.org/r/#/c/305933/ ? [21:47:05] (03CR) 10Hoo man: "Also taking system time into account now, per Anomie." [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/305649 (https://phabricator.wikimedia.org/T143389) (owner: 10Hoo man) [21:49:49] 10MobileFrontend, 10Reading-Community-Engagement, 06Reading-Web-Backlog, 13Patch-For-Review, and 3 others: [SPIKE: 1 hour] Avoid unintended wrapping on en.m.wikipedia.org "2016 Formula One season" article - https://phabricator.wikimedia.org/T143558#2580674 (10Jdlrobson) We used to replace .lazy-image-place... [21:50:56] (03CR) 10jenkins-bot: [V: 04-1] Cleanups to SqlBagOStuff [core] - 10https://gerrit.wikimedia.org/r/306553 (owner: 10Aaron Schulz) [21:51:01] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Add support for masking the WRITE_SYNC latency from ChronologyProtector [core] - 10https://gerrit.wikimedia.org/r/306554 (owner: 10Aaron Schulz) [21:54:09] (03CR) 10Krinkle: Cleanups to SqlBagOStuff (031 comment) [core] - 10https://gerrit.wikimedia.org/r/306553 (owner: 10Aaron Schulz) [21:54:28] 07Design, 06Discovery, 10Kartographer, 06Maps, and 2 others: Override leaflet style to match wikimedia style - https://phabricator.wikimedia.org/T143833#2580676 (10JGirault) Patch screenshot: {F4397962} [21:54:35] (03CR) 10Krinkle: [C: 04-1] Use i18n messages for size [extensions/PerformanceInspector] - 10https://gerrit.wikimedia.org/r/306411 (https://phabricator.wikimedia.org/T143770) (owner: 10Phedenskog) [21:55:08] (03PS9) 10Krinkle: Various fixes to DeferrableUpdates related classes [core] - 10https://gerrit.wikimedia.org/r/305953 (owner: 10Aaron Schulz) [21:56:46] 10MediaWiki-extensions-ORES, 10Revision-Scoring-As-A-Service-Backlog: Request scores when someone checks out edits that are not stored in ores_classification - https://phabricator.wikimedia.org/T143612#2573392 (10Legoktm) Is this a duplicate of {T131785}? Or subtask? [21:58:11] (03PS1) 10Mooeypoo: Append mw.notify div to the body so it appears on top of everything [core] - 10https://gerrit.wikimedia.org/r/306560 [21:59:29] (03PS4) 10Mooeypoo: [wip] Add 'mark all read' button to the notification overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306288 (https://phabricator.wikimedia.org/T141404) [22:00:18] (03PS1) 10Bmansurov: Beta: vertically center-align the spinner of the search results [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306561 (https://phabricator.wikimedia.org/T137068) [22:01:14] 07Design, 10UI-Standardization, 06Collaboration-Team-Triage, 10Flow, 10VisualEditor: Come up with a better re-usable UI concept for a button to switch editor-mode; the current one is confusing, and hard to discover - https://phabricator.wikimedia.org/T116417#2580702 (10Quiddity) Big +1 for ButtonSelectWi... [22:01:38] 07Design, 10MobileFrontend, 06Reading-Web-Backlog, 13Patch-For-Review, and 2 others: Align elements in article footer on beta - https://phabricator.wikimedia.org/T141002#2580703 (10MBinder_WMF) @Nirzar @Jdlrobson added to retro [22:02:01] (03CR) 10Aaron Schulz: Cleanups to SqlBagOStuff (031 comment) [core] - 10https://gerrit.wikimedia.org/r/306553 (owner: 10Aaron Schulz) [22:04:56] 10MediaWiki-Parser, 07Regression: wgPageParseReport is empty on preview / section preview - https://phabricator.wikimedia.org/T142210#2580725 (10Johan) [22:05:23] (03CR) 10jenkins-bot: [V: 04-1] [wip] Add 'mark all read' button to the notification overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306288 (https://phabricator.wikimedia.org/T141404) (owner: 10Mooeypoo) [22:05:35] (03CR) 10Bmansurov: Add separators between footer links in beta (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [22:06:54] (03CR) 10Smalyshev: [C: 032] Add flag for disabling phrase suggester [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306551 (https://phabricator.wikimedia.org/T143260) (owner: 10EBernhardson) [22:06:59] (03PS5) 10Mooeypoo: [wip] Add 'mark all read' button to the notification overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306288 (https://phabricator.wikimedia.org/T141404) [22:07:41] (03CR) 10Gergő Tisza: Use psysh for Zend shell instead of phpsh (032 comments) [vagrant] - 10https://gerrit.wikimedia.org/r/306487 (https://phabricator.wikimedia.org/T123951) (owner: 10Mattflaschen) [22:08:31] (03CR) 10Cindy-the-browser-test-bot: [C: 031] "Cindy says good job \o/. Keep it up." [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306551 (https://phabricator.wikimedia.org/T143260) (owner: 10EBernhardson) [22:08:40] (03CR) 10Krinkle: "I'm for moving it to the body. However two things to consider:" [core] - 10https://gerrit.wikimedia.org/r/306560 (owner: 10Mooeypoo) [22:08:59] 10MediaWiki-Vagrant: Integrate a modern php REPL shell with MediaWiki - https://phabricator.wikimedia.org/T117661#2580728 (10Tgr) https://gerrit.wikimedia.org/r/#/c/306487 is about integrating psysh with vagrant. [22:09:08] (03CR) 10Cindy-the-browser-test-bot: "Phan static analysis done without errors!" [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306551 (https://phabricator.wikimedia.org/T143260) (owner: 10EBernhardson) [22:09:58] 06ArchCom, 10ArchCom-RfC, 07RfC: [RFC] STUB: Creating a spec for legacy wikitext / future wiki content processing model - https://phabricator.wikimedia.org/T142803#2580729 (10ssastry) As promised, I put together a first draft at https://www.mediawiki.org/wiki/Parsing/Notes/Wikitext_2.0 which can be developed... [22:10:26] 10ProofreadPage, 13Patch-For-Review, 05WMF-deploy-2016-08-23_(1.28.0-wmf.16), 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wm... - https://phabricator.wikimedia.org/T143817#2580730 [22:10:47] 06Discovery, 06Discovery-Analysis (Current work): Wikipedia.org Portal Dashboard: add "other" to pageviews page - https://phabricator.wikimedia.org/T143605#2580731 (10mpopov) In our meeting, @chelsyx suggested to dynamically figure out the threshold based on bottom 99.99% and top 0.01%. (Or even 99.9% vs 0.1%!... [22:10:49] 10ProofreadPage, 13Patch-For-Review, 05WMF-deploy-2016-08-23_(1.28.0-wmf.16), 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wm... - https://phabricator.wikimedia.org/T143817#2580059 [22:11:04] (03CR) 10jenkins-bot: [V: 04-1] [wip] Add 'mark all read' button to the notification overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306288 (https://phabricator.wikimedia.org/T141404) (owner: 10Mooeypoo) [22:11:23] (03PS1) 10Awight: [WIP] DonationInterface settings for running under the CRM [vagrant] - 10https://gerrit.wikimedia.org/r/306564 (https://phabricator.wikimedia.org/T141487) [22:11:45] (03PS3) 10Dereckson: Fix ProofreadPage::updatePrIndex signature [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306545 [22:11:57] legoktm: here is my attack on DataUpdate: https://gerrit.wikimedia.org/r/#/c/305953 [22:12:32] 10MediaWiki-Internationalization, 13Patch-For-Review, 07User-notice: Localisation fallback between Slovak and Czech - https://phabricator.wikimedia.org/T143519#2570430 (10Johan) Side note: > (no technical news notice for example) I don't think we should rely on Tech News for something like this. Tech News... [22:13:49] (03PS1) 10Bmansurov: Beta: make the page background gray [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306565 (https://phabricator.wikimedia.org/T141002) [22:14:15] (03CR) 10jenkins-bot: [V: 04-1] [WIP] DonationInterface settings for running under the CRM [vagrant] - 10https://gerrit.wikimedia.org/r/306564 (https://phabricator.wikimedia.org/T141487) (owner: 10Awight) [22:14:44] (03CR) 10Bmansurov: "Here is a follow-up patch. I'd like to know if you see any problems with it. If not, do we want to squash the patch or merge separately?" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306472 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [22:14:47] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service: ORES extension should Assume good faith page creator's revisions - https://phabricator.wikimedia.org/T137846#2580747 (10Iniquity) >>! In T137846#2571165, @Ladsgroup wrote: > @Iniquity. Hey, Can you explain more? Hi, in Russia Wikipedia we have 'ed... [22:15:05] (03CR) 10Mooeypoo: "Regarding the first point - in vector, it works as intended, but in monobook, the overlay is on top of the personal toolbar now. I can cha" [core] - 10https://gerrit.wikimedia.org/r/306560 (owner: 10Mooeypoo) [22:19:35] 10MediaWiki-General-or-Unknown, 10Notifications, 03Collab-Team-Q1-July-Sep-2016: Append mw.notify div to the body instead of content so it is always on top - https://phabricator.wikimedia.org/T143837#2580751 (10Mooeypoo) [22:19:50] (03CR) 10Legoktm: [C: 032] "A good follow-up would be to have the extension pass WikiPage objects." [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306545 (owner: 10Dereckson) [22:20:05] (03PS2) 10Mooeypoo: Append mw.notify div to the body so it appears on top of everything [core] - 10https://gerrit.wikimedia.org/r/306560 (https://phabricator.wikimedia.org/T143837) [22:20:29] 06Discovery, 06Discovery-Analysis (Current work), 13Patch-For-Review: Wikipedia.org Portal Dashboard: add clicks by language - https://phabricator.wikimedia.org/T140816#2580766 (10mpopov) @debt: is this latest version OK to deploy to production? [22:21:21] 10MediaWiki-General-or-Unknown, 10Notifications, 03Collab-Team-Q1-July-Sep-2016, 13Patch-For-Review: Append mw.notify div to the body instead of content so it is always on top - https://phabricator.wikimedia.org/T143837#2580768 (10Mooeypoo) Posting my answer to the discussion in the commit: > Regarding th... [22:21:23] (03Merged) 10jenkins-bot: Fix ProofreadPage::updatePrIndex signature [extensions/ProofreadPage] - 10https://gerrit.wikimedia.org/r/306545 (owner: 10Dereckson) [22:22:28] 10MediaWiki-General-or-Unknown, 10Notifications, 03Collab-Team-Q1-July-Sep-2016, 13Patch-For-Review: Append mw.notify div to the body instead of content so it is always on top - https://phabricator.wikimedia.org/T143837#2580751 (10Mooeypoo) a:03Mooeypoo [22:24:06] 10ProofreadPage, 07Technical-Debt: Replace Article instances by WikiPage - https://phabricator.wikimedia.org/T143838#2580771 (10Dereckson) [22:24:56] (03PS1) 10Dereckson: Fix ProofreadPage::updatePrIndex signature [extensions/ProofreadPage] (wmf/1.28.0-wmf.16) - 10https://gerrit.wikimedia.org/r/306571 [22:24:57] 06Discovery, 06Discovery-Analysis (Current work): Wikipedia.org Portal Dashboard: add "other" to pageviews page - https://phabricator.wikimedia.org/T143605#2580786 (10BBlack) For my $0.02, I tend to think 99.99% sounds better as a threshold than 99.9%. There could be actual human outliers near that 99.9% mark... [22:25:13] 07Design, 10UI-Standardization, 06Collaboration-Team-Triage, 10Flow, 10VisualEditor: Come up with a better re-usable UI concept for a button to switch editor-mode; the current one is confusing, and hard to discover - https://phabricator.wikimedia.org/T116417#2580788 (10Volker_E) @Quiddity Yes! That's T13... [22:27:17] 10ProofreadPage, 07Easy, 07Technical-Debt: Pass WikiPage instances to updatePrIndex() in ProofreadPage - https://phabricator.wikimedia.org/T143838#2580790 (10Legoktm) [22:27:31] 10ProofreadPage, 07Easy, 07Technical-Debt: Pass WikiPage instances to updatePrIndex() in ProofreadPage - https://phabricator.wikimedia.org/T143838#2580771 (10Legoktm) WikiPage::factory( $nt ) should work. [22:27:39] (03CR) 10Jdlrobson: [C: 04-1] "You need to have the specific animations rule. Footer is white for me." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306565 (https://phabricator.wikimedia.org/T141002) (owner: 10Bmansurov) [22:27:42] (03CR) 10Jhobs: [C: 031] "+1 was originally removed because I couldn't understand why jenkins was still failing after the recheck. Now that it's confirmed it was CI" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306473 (https://phabricator.wikimedia.org/T141002) (owner: 10Jdlrobson) [22:28:13] (03CR) 10Jdlrobson: "Happy to merge this separately. We can change commit message to" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306565 (https://phabricator.wikimedia.org/T141002) (owner: 10Bmansurov) [22:29:12] 06Discovery, 06Discovery-Analysis (Current work), 13Patch-For-Review: Wikipedia.org Portal Dashboard: add clicks by language - https://phabricator.wikimedia.org/T140816#2580798 (10debt) Yes - but can you make the 'combine languages' (for clicks) be the default screen for the bottom 50, please? [22:29:12] (03CR) 10Bmansurov: "My footer is gray. Can you clear your cache?" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306565 (https://phabricator.wikimedia.org/T141002) (owner: 10Bmansurov) [22:33:46] 06Discovery, 06Discovery-Analysis (Current work), 13Patch-For-Review: Dashboard: Mobile App Events needs fixing - https://phabricator.wikimedia.org/T143447#2580802 (10mpopov) Backfilled http://discovery.wmflabs.org/metrics/#app_events [22:33:48] (03CR) 10Krinkle: [C: 04-1] Various fixes to DeferrableUpdates related classes (032 comments) [core] - 10https://gerrit.wikimedia.org/r/305953 (owner: 10Aaron Schulz) [22:33:58] (03PS2) 10Aaron Schulz: Cleanups to SqlBagOStuff [core] - 10https://gerrit.wikimedia.org/r/306553 [22:34:55] (03CR) 10Krinkle: "Tested with https://gist.github.com/Krinkle/26a58dfaef465ae1193cd8480e92b8f3" [core] - 10https://gerrit.wikimedia.org/r/305953 (owner: 10Aaron Schulz) [22:40:26] 10Mobile-Content-Service: Mobile content service doesn't invalidate on edit - https://phabricator.wikimedia.org/T143839#2580809 (10Jdlrobson) [22:40:30] 06Discovery, 06Discovery-Analysis: Add comment to search-related eventlogging in iOS and Android apps to inform Discovery of changes - https://phabricator.wikimedia.org/T143753#2580822 (10mpopov) [22:41:25] (03CR) 10Krinkle: [C: 04-1] "If we do this we should disallow clickjacking so that the JS will not run if the url is iframed (and thus cause a write action cross-domai" [core] - 10https://gerrit.wikimedia.org/r/306325 (https://phabricator.wikimedia.org/T143531) (owner: 10Legoktm) [22:41:44] (03CR) 10Krinkle: Cleanups to SqlBagOStuff (031 comment) [core] - 10https://gerrit.wikimedia.org/r/306553 (owner: 10Aaron Schulz) [22:42:12] (03CR) 10Aaron Schulz: Various fixes to DeferrableUpdates related classes (031 comment) [core] - 10https://gerrit.wikimedia.org/r/305953 (owner: 10Aaron Schulz) [22:42:19] 10MediaWiki-User-login-and-signup, 06Community-Tech: Unable to reset temporary password on mediawiki - https://phabricator.wikimedia.org/T143840#2580823 (10Jitrixis) [22:43:31] (03PS2) 10Awight: [WIP] DonationInterface settings for running under the CRM [vagrant] - 10https://gerrit.wikimedia.org/r/306564 (https://phabricator.wikimedia.org/T141487) [22:46:26] (03CR) 10Aaron Schulz: [C: 04-1] "Accessor calls missing" [core] - 10https://gerrit.wikimedia.org/r/306553 (owner: 10Aaron Schulz) [22:50:26] 10MediaWiki-General-or-Unknown, 10Notifications, 03Collab-Team-Q1-July-Sep-2016, 13Patch-For-Review: Append mw.notify div to the body instead of content so it is always on top - https://phabricator.wikimedia.org/T143837#2580845 (10Mooeypoo) @Esanders pointed out an idea that would fix both issues: mw.noti... [22:53:10] (03CR) 10Krinkle: Move Article checkLastModified() up to MediaWiki::performRequest (031 comment) [core] - 10https://gerrit.wikimedia.org/r/306334 (owner: 10Aaron Schulz) [22:55:10] 07Design, 10MediaWiki-History-or-Diffs, 06TCB-Team, 06WMDE-Design, 03TCB-Team-Sprint-2016-08-11: Indicate when text chunks have been moved, but not changed otherwise - https://phabricator.wikimedia.org/T143422#2580849 (10Alsee) ''Would it make sense to you to keep the highlighting like it is now so a mov... [22:58:53] (03CR) 10Aaron Schulz: Move Article checkLastModified() up to MediaWiki::performRequest (031 comment) [core] - 10https://gerrit.wikimedia.org/r/306334 (owner: 10Aaron Schulz) [22:59:05] 06Wikipedia-Android-App-Backlog, 10Android-app-feature-Navigation (Android-app-feature-Navigation-MVP), 03Mobile-App-Android-Sprint-89-Actinium, 13Patch-For-Review: Move the article title from the lead image - https://phabricator.wikimedia.org/T135294#2580857 (10Nirzar) Looking good :) one last thing abo... [23:00:06] Krinkle: I'm leaning towards just logging all DataUpdate exceptions [23:00:14] (03CR) 10Dereckson: [C: 032] "SWAT" [extensions/ProofreadPage] (wmf/1.28.0-wmf.16) - 10https://gerrit.wikimedia.org/r/306571 (owner: 10Dereckson) [23:00:36] * AaronSchulz thinks of use cases [23:00:54] (03PS1) 10EBernhardson: fromId was renamed to fromPageId [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306579 [23:01:46] Krinkle: so the idea would be to present some "Indexing updates failed, your item might not be searchable for a while" message in a PRESEND update? [23:01:53] (03Merged) 10jenkins-bot: Fix ProofreadPage::updatePrIndex signature [extensions/ProofreadPage] (wmf/1.28.0-wmf.16) - 10https://gerrit.wikimedia.org/r/306571 (owner: 10Dereckson) [23:01:53] or some such [23:03:01] (03PS1) 10Florianschmidtwelzow: Hide empty li elements according to tidy rules (like in mediawiki/core) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306580 (https://phabricator.wikimedia.org/T129375) [23:03:45] 06Research-and-Data, 10Research-consulting: Query on anonymous editing in the User: namespace on en.wiki - https://phabricator.wikimedia.org/T141829#2580862 (10Halfak) I've gathered a random sample of recent edits within the "User" namespace on English Wikipedia that were not to sub-pages and whose IP address... [23:04:12] (03PS1) 10Bearloga: Give option to 'combine languages' when multiple langs are selected && check it by default whenever 'Bottom 50' sorting option is chosen [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306581 (https://phabricator.wikimedia.org/T140816) [23:04:12] hmm [23:04:32] (03CR) 10Bearloga: [C: 032 V: 032] Give option to 'combine languages' when multiple langs are selected && check it by default whenever 'Bottom 50' sorting option is chosen [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306581 (https://phabricator.wikimedia.org/T140816) (owner: 10Bearloga) [23:05:10] 10MediaWiki-General-or-Unknown, 10Notifications, 03Collab-Team-Q1-July-Sep-2016, 13Patch-For-Review: Append mw.notify div to the body instead of content so it is always on top - https://phabricator.wikimedia.org/T143837#2580869 (10Esanders) You should give the global overlay an additional class so it can b... [23:05:31] AaronSchulz: No, but if a critical (rather than secondary) thing fails, but is deferreed to presend for some reason, if it throws a errorPageError it will take over the output page entirely, just like it does outside presend. [23:05:46] I don't think we'd throw ErrorPageError for secondary update failures. [23:05:54] But if something wants to, it can I suppose. [23:06:25] I think there may be cases already were something indirectly throws such error and used to be visible but got hidden when various things were moved into presend. [23:07:04] I think I'll just show them, that keeps dataupdates closer to other deferredupdates [23:08:00] (03CR) 10Krinkle: "It looks like "hacks" is mainly for Wikimedia specific things cherry-picked from production site's Common.css etc." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306580 (https://phabricator.wikimedia.org/T129375) (owner: 10Florianschmidtwelzow) [23:08:33] (03CR) 10Bmansurov: [C: 04-1] "Also, forgot about stable." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306565 (https://phabricator.wikimedia.org/T141002) (owner: 10Bmansurov) [23:10:48] (03CR) 10jenkins-bot: [V: 04-1] Hide empty li elements according to tidy rules (like in mediawiki/core) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306580 (https://phabricator.wikimedia.org/T129375) (owner: 10Florianschmidtwelzow) [23:12:49] (03PS1) 10Legoktm: debug: Don't show git branch if on a detached HEAD [core] - 10https://gerrit.wikimedia.org/r/306583 [23:12:53] (03PS1) 10Legoktm: debug: Merge mediawiki.debug.init module into mediawiki.debug [core] - 10https://gerrit.wikimedia.org/r/306584 [23:12:56] (03PS1) 10Legoktm: debug: Interpolate log messages [core] - 10https://gerrit.wikimedia.org/r/306585 [23:17:41] 10MediaWiki-Internationalization, 10MediaWiki-Page-editing, 06Community-Liaisons, 06Editing-Department, and 7 others: Re-label the "Save" button to be "Publish", to better indicate to users the outcomes of their action - https://phabricator.wikimedia.org/T131132#2232671 (10Dereckson) @Base We discussed thi... [23:17:41] (03CR) 10Krinkle: Move Article checkLastModified() up to MediaWiki::performRequest (032 comments) [core] - 10https://gerrit.wikimedia.org/r/306334 (owner: 10Aaron Schulz) [23:18:04] (03CR) 10jenkins-bot: [V: 04-1] debug: Merge mediawiki.debug.init module into mediawiki.debug [core] - 10https://gerrit.wikimedia.org/r/306584 (owner: 10Legoktm) [23:18:50] 10Wiki-Loves-Monuments (2016): On-board Turkey - https://phabricator.wikimedia.org/T143765#2578134 (10WikiBronze) Got the official list and working on it to create unicodes and transfering data to Wikipedia [23:18:53] (03PS6) 10Mooeypoo: [wip] Add 'mark all read' button to the notification overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306288 (https://phabricator.wikimedia.org/T141404) [23:20:26] 10MediaWiki-General-or-Unknown, 10Notifications, 03Collab-Team-Q1-July-Sep-2016, 13Patch-For-Review: Append mw.notify div to the body instead of content so it is always on top - https://phabricator.wikimedia.org/T143837#2580884 (10Mooeypoo) So this sounds like a good idea, and it should work - but the way... [23:20:44] (03CR) 10Krinkle: "LGTM (minor space issue). This was originally kept separate so that the module would be re-usable and testable without the init side effec" [core] - 10https://gerrit.wikimedia.org/r/306584 (owner: 10Legoktm) [23:20:56] (03CR) 10Cindy-the-browser-test-bot: [C: 031] "Cindy says good job \o/. Keep it up." [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306579 (owner: 10EBernhardson) [23:21:17] (03CR) 10Catrope: [C: 032] Followup on I75d0098e: Use getTopicTitleObj properly [extensions/Flow] - 10https://gerrit.wikimedia.org/r/306483 (https://phabricator.wikimedia.org/T143757) (owner: 10Mooeypoo) [23:21:32] (03CR) 10Cindy-the-browser-test-bot: "Phan static analysis done without errors!" [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/306579 (owner: 10EBernhardson) [23:22:26] jdlrobson: https://gerrit.wikimedia.org/r/#/c/280161/3 [23:23:37] (03PS2) 10Legoktm: debug: Merge mediawiki.debug.init module into mediawiki.debug [core] - 10https://gerrit.wikimedia.org/r/306584 [23:23:40] (03PS2) 10Legoktm: debug: Interpolate log messages [core] - 10https://gerrit.wikimedia.org/r/306585 [23:23:53] (03CR) 10jenkins-bot: [V: 04-1] [wip] Add 'mark all read' button to the notification overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306288 (https://phabricator.wikimedia.org/T141404) (owner: 10Mooeypoo) [23:23:57] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 13Patch-For-Review: Entity Suggester shows double information at certain moments - https://phabricator.wikimedia.org/T143645#2580886 (10hoo) a:03hoo Patch: https://github.com/Wikidata-lib/PropertySuggester/pull/175 The cause here is that we do the se... [23:27:47] (03Merged) 10jenkins-bot: Followup on I75d0098e: Use getTopicTitleObj properly [extensions/Flow] - 10https://gerrit.wikimedia.org/r/306483 (https://phabricator.wikimedia.org/T143757) (owner: 10Mooeypoo) [23:27:54] (03CR) 10Legoktm: [C: 032] API: Log when too many values are passed for a multi-valued parameter [core] - 10https://gerrit.wikimedia.org/r/306457 (https://phabricator.wikimedia.org/T41936) (owner: 10Anomie) [23:34:05] (03Merged) 10jenkins-bot: API: Log when too many values are passed for a multi-valued parameter [core] - 10https://gerrit.wikimedia.org/r/306457 (https://phabricator.wikimedia.org/T41936) (owner: 10Anomie) [23:34:36] 10MediaWiki-File-management, 10Beta-Cluster-Infrastructure, 06Commons, 06Multimedia, 07Tracking: Thumbnail generation should happen via the same setup in the beta cluster and in production (tracking) - https://phabricator.wikimedia.org/T84950#2580904 (10Krenair) [23:34:49] (03CR) 10Esanders: "Ping" [core] - 10https://gerrit.wikimedia.org/r/297690 (owner: 10Esanders) [23:35:32] (03CR) 10Aaron Schulz: Move Article checkLastModified() up to MediaWiki::performRequest (032 comments) [core] - 10https://gerrit.wikimedia.org/r/306334 (owner: 10Aaron Schulz) [23:36:24] 10MediaWiki-extensions-ORES, 10ORES, 06Revision-Scoring-As-A-Service, 07Epic, 03Research-and-Data-2017-Q1: [Epic] Deploy ORES review tool - https://phabricator.wikimedia.org/T140002#2580909 (10Legoktm) [23:36:26] 10MediaWiki-extensions-ORES, 06Revision-Scoring-As-A-Service, 10Wikimedia-Site-requests, 07Beta-Feature, and 2 others: Deploy ORES review tool in English Wikipedia - https://phabricator.wikimedia.org/T140003#2580908 (10Legoktm) 05Open>03Resolved [23:37:27] 10MediaWiki-extensions-WikibaseClient, 10ArticlePlaceholder, 10Wikidata: Implement a hook for linking wikibase-item values that don't have local sitelink - https://phabricator.wikimedia.org/T143841#2580916 (10hoo) [23:40:37] (03CR) 10Aaron Schulz: Move Article checkLastModified() up to MediaWiki::performRequest (031 comment) [core] - 10https://gerrit.wikimedia.org/r/306334 (owner: 10Aaron Schulz) [23:41:24] 07Design, 10MediaWiki-extensions-WikibaseClient, 10ArticlePlaceholder, 06WMDE-Design, 10Wikidata: Choose a color for linking article placeholders in place of redlinks - https://phabricator.wikimedia.org/T143842#2580932 (10hoo) [23:42:09] Krinkle: why not ViewAction::show? [23:42:38] (03PS3) 10Mooeypoo: Add an option for global overlay in mw.notify div [core] - 10https://gerrit.wikimedia.org/r/306560 [23:44:10] (03PS4) 10Mooeypoo: Add an option for global overlay in mw.notify div [core] - 10https://gerrit.wikimedia.org/r/306560 [23:45:01] 10MediaWiki-General-or-Unknown, 07JavaScript, 07Technical-Debt: Remove use of jquery.tipsy from Wikimedia-maintained code - https://phabricator.wikimedia.org/T117720#2580948 (10Legoktm) [23:46:17] 06Discovery-Analysis (Current work): Dashboards: update for Product Owner change - https://phabricator.wikimedia.org/T142436#2535042 (10mpopov) Moving into in-progress as Chelsy is currently working through the deploying-to-production steps :D [23:48:03] 06Reading-Web-Backlog, 03Reading-Web-Sprint-80-V, 07Spike: [SPIKE: 4hrs] Lazy loaded images data analysis and blog post - https://phabricator.wikimedia.org/T143627#2580953 (10Jdlrobson) a:03Jdlrobson [23:50:01] 10MediaWiki-extensions-DonationInterface, 10FR-Smashpig, 10Fundraising-Backlog, 07Spike: Spike: Reconcile SmashPig and DonationInterface configuration - https://phabricator.wikimedia.org/T143831#2580956 (10awight) [23:50:03] 10MediaWiki-extensions-DonationInterface, 10FR-ActiveMQ, 10FR-Smashpig, 10Fundraising-Backlog, and 5 others: Run the orphan rectifier job from CRM Jenkins - https://phabricator.wikimedia.org/T141487#2580955 (10awight) [23:50:43] AaronSchulz: That seems fine. [23:52:42] 10ProofreadPage, 13Patch-For-Review, 05WMF-deploy-2016-08-23_(1.28.0-wmf.16), 07Wikimedia-log-errors: Catchable fatal error: Argument 1 passed to ProofreadPage::onArticlePurge() must be an instance of Article, WikiPage given in /srv/mediawiki/php-1.28.0-wm... - https://phabricator.wikimedia.org/T143817#2580973 [23:52:45] 10Wikimedia-Site-requests: "Accounts creator" permission in arwiki - https://phabricator.wikimedia.org/T143844#2580976 (10Ibrahim.ID) [23:53:32] (03PS1) 10Mattflaschen: Dynamic actions: Reuse label/description for those fields [extensions/Echo] - 10https://gerrit.wikimedia.org/r/306589 [23:55:00] (03PS1) 10Mattflaschen: Update for new secondary link format [extensions/Flow] - 10https://gerrit.wikimedia.org/r/306590 [23:55:35] (03CR) 10Jdlrobson: "I have no cache issues. As I said in last review, the footer is white due to the css rule .animations #mw-mf-page-center" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306565 (https://phabricator.wikimedia.org/T141002) (owner: 10Bmansurov) [23:56:26] 10Wikimedia-Site-requests: "Accounts creator" permission in arwiki - https://phabricator.wikimedia.org/T143844#2580994 (10Dereckson) a:03Dereckson @Ibrahim.ID Did you discuss the issue locally? If so, could you edit the discussion and add a link to the discussion? If not, please start one at the most appropria... [23:56:53] (03CR) 10Jdlrobson: [C: 04-1] "Yup as @krinkle suggests this is not what hacks should be used for." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306580 (https://phabricator.wikimedia.org/T129375) (owner: 10Florianschmidtwelzow) [23:57:06] (03CR) 10Liuxinyu970226: [C: 031] Add extension registeration file 'extension.json'. [extensions/QuickGV] - 10https://gerrit.wikimedia.org/r/303773 (owner: 10小璋丸) [23:57:54] (03PS1) 10Chelsyx: Deploying updates for all dashboards [wikimedia/discovery/dashboard] - 10https://gerrit.wikimedia.org/r/306591 [23:58:54] (03CR) 10Jdlrobson: [C: 04-1] Beta: vertically center-align the spinner of the search results (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306561 (https://phabricator.wikimedia.org/T137068) (owner: 10Bmansurov) [23:59:34] 10Wikimedia-Extension-setup, 07Community-consensus-needed: Enable Extension:Babel's category on Czech Wikiversity - https://phabricator.wikimedia.org/T67211#2581014 (10Dereckson) 05Open>03stalled @Juandev So please start a discussion or indicate to us you don't need this anymore. [23:59:46] 07Design, 10MediaWiki-History-or-Diffs, 06TCB-Team, 06WMDE-Design, 03TCB-Team-Sprint-2016-08-11: Indicate when text chunks have been moved, but not changed otherwise - https://phabricator.wikimedia.org/T143422#2581016 (10Alsee) {F4398174} I just made a mock-up with =/= and using black. I think it works.