[00:03:56] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Adjust the mark as read "X" icon size - https://phabricator.wikimedia.org/T124025#2053613 (10Etonkovidova) Checked in betalabs .mw-echo-ui-notificationItemWidget-markAsReadButton .oo-ui-iconElement-i... [00:05:53] RoanKattouw: you can explode that query by wiki but remember that means you've got a ton of data. So only do so if you really want ALL the data. Otherwise just add the ones that you need. And you can self-merge on stuff like that [00:10:54] OK thanks [00:10:56] What's the deploy process? [00:36:05] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-02-09_(1.27.0-wmf.13), 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): If user only has foreign messages, the messages badge should not be suppressed - https://phabricator.wikimedia.org/T124372#2053701 (10Etonkovidova) Re-checked... [00:41:14] 3Collaboration-Team-Current, 10Notifications: Show descriptions for the actions under the "..." menu on notifications - https://phabricator.wikimedia.org/T126738#2053715 (10jmatazzoni) @Mooeypoo asks: > Do we have a list of the descriptions we want to have, or should we create that list? @Pginer-WMF, do we... [00:49:39] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2053719 (10jmatazzoni) Sorry @SBisson, but I'm still not s... [01:10:42] 3Collaboration-Team-Current, 10Flow, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Create a new topic with mentions in title and post sends 2 mention notifications - https://phabricator.wikimedia.org/T126345#2011717 (10jmatazzoni) The behavior, as you both note, is accordin... [01:10:53] 3Collaboration-Team-Current, 10Flow, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Create a new topic with mentions in title and post sends 2 mention notifications - https://phabricator.wikimedia.org/T126345#2053775 (10jmatazzoni) 5Open>3Resolved [01:12:44] 3Collaboration-Team-Current, 10Flow, 10MediaWiki-extensions-Nuke, 13Patch-For-Review, 7user-notice: Add Flow support to Nuke - https://phabricator.wikimedia.org/T115695#2053777 (10jmatazzoni) 5Open>3Resolved [01:22:25] 6Collaboration-Team-Backlog, 10Notifications, 7Design: x "Mark as read" in Flow messages in Echo flyout UX (too small, lacks tooltip, etc.) - https://phabricator.wikimedia.org/T71915#2053791 (10jmatazzoni) I'm pretty sure this has been fixed by later work in T124025. Closing this. [01:22:33] 6Collaboration-Team-Backlog, 10Notifications, 7Design: x "Mark as read" in Flow messages in Echo flyout UX (too small, lacks tooltip, etc.) - https://phabricator.wikimedia.org/T71915#2053792 (10jmatazzoni) 5Open>3Resolved [01:22:47] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Adjust the mark as read "X" icon size - https://phabricator.wikimedia.org/T124025#1943484 (10jmatazzoni) @Pginer-WMF, does this look OK to you? If so, please close the bug. Thanks. [01:41:30] RoanKattouw, turns out Flow page moves are completely broken. Thought it was my patch at first, but it's also on Beta. [01:41:39] Ugh [01:41:51] http://en.wikipedia.beta.wmflabs.org/wiki/Flow_test_talk:Foo_bar => http://en.wikipedia.beta.wmflabs.org/wiki/Flow_test_talk:Foo_baz [01:42:06] Do you think it's due to a change in Flow or a change in MW core? [01:42:14] RoanKattouw, probably core. [01:42:27] * RoanKattouw nods [01:42:32] RoanKattouw, I'll start looking since I need to figure it out to unblock my own patch (can't put it up if it doesn't work). [01:42:34] I didn't see anything suspicious come by in Flow at least [01:42:39] Right [01:42:42] Yeah, it sounds like bisection time [01:42:51] Good suggestion [01:43:38] It worked on enwiki right? So the bisection window is probably not too big, only since wmf13 [01:43:56] Although I guess that's two weeks' worth, at MW core velocity :/ but still, probably manageable [01:44:15] Yeah, I'll start with wmf13. [01:45:57] (03PS1) 10Mooeypoo: Transform the notification action bar to css table layout [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272651 (https://phabricator.wikimedia.org/T125909) [01:47:11] 6Collaboration-Team-Backlog, 10Flow, 7Regression: Board moves in Flow not working - https://phabricator.wikimedia.org/T127785#2053878 (10Mattflaschen) [01:47:26] 3Collaboration-Team-Current, 10Flow, 7Regression: Board moves in Flow not working - https://phabricator.wikimedia.org/T127785#2053894 (10Mattflaschen) [01:48:50] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: [minor] Notification panel layout issues for multiline messages - https://phabricator.wikimedia.org/T125909#2000378 (10Mooeypoo) The commit above fixes #2 and **should** also fix #1 (I increased the side padding) @Pginer-WMF can you clarify #... [02:00:05] RoanKattouw and matt_flaschen the crosswiki notification is so valuable to me, already used it about 10 times. so much joy ! :) [02:02:23] matanya, great to hear. Glad it works well for you. [02:03:23] Yay, nice to hear [02:03:47] (03PS3) 10Jforrester: VisualEditorWidget: Use selectLastContentOffset() from VE directly [extensions/Flow] - 10https://gerrit.wikimedia.org/r/234347 (owner: 10Catrope) [02:27:39] 3Collaboration-Team-Current, 10Flow, 7Regression: Board moves in Flow not working - https://phabricator.wikimedia.org/T127785#2053967 (10Mattflaschen) A warning on mismatched transactions was changed to an error: {ef44f4308edbcd97d6e53d211716fa2dfa143c26} We might be able to use startAtomic. [03:00:50] imo what elasticsearch is really missing in terms of full text search is all the tools to make search better. To measure relevancy, evaluate changes you make to scoring (boosts/rescore/etc), etc. etc. You have to do all that yourself [03:00:56] wrong room [04:00:04] 3Collaboration-Team-Current, 10Flow, 7Regression: Board moves in Flow not working - https://phabricator.wikimedia.org/T127785#2054076 (10Mattflaschen) I have a WIP to change it to startAtomic/endAtomic (making sure to use the same fname). However, it's not quite there yet. Most places in production will us... [04:01:32] (03PS1) 10Mattflaschen: DO NOT MERGE: Show not-yet-working board move change [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272665 (https://phabricator.wikimedia.org/T127785) [04:05:06] See you all tomorrow. [04:21:29] Would enabling Flow on wikitech be difficult or problematic? [04:22:11] I think that using Flow on the talk pages of the new Tool namespace would make managing requests and feedback to tool owners easier [05:09:07] 3Collaboration-Team-Current, 6Editing-Department, 10Notifications, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15), 7user-notice: Automatically congratulate users for reaching certain edit count milestones - https://phabricator.wikimedia.org/T124003#2054137 (10Ricordisamoa) @Jdforrester-WMF Am I right in expectin... [05:31:06] bd808: it's been awhile, but i don't think it would be technically problematic. Just socialization [05:31:44] *nod* I'll open a phab task and then we can have the battle of yes, no, whatever discussion :) [05:33:27] 6Collaboration-Team-Backlog, 10Flow, 6Labs, 10wikitech.wikimedia.org: Enable Flow for Tool namespace on wikitech - https://phabricator.wikimedia.org/T127792#2054159 (10bd808) [05:33:44] 6Collaboration-Team-Backlog, 10Flow, 6Labs, 10wikitech.wikimedia.org: Enable Flow for Tool namespace on wikitech - https://phabricator.wikimedia.org/T127792#2054173 (10bd808) [06:11:12] 6Collaboration-Team-Backlog, 10Notifications, 7Easy, 13Patch-For-Review: Echo congratulations thresholds shouldn't be hard-coded - https://phabricator.wikimedia.org/T127311#2054197 (10MZMcBride) >>! In T127311#2053337, @Jdforrester-WMF wrote: > The array (which shouldn't have been added without code review... [06:16:25] 6Collaboration-Team-Backlog, 10Flow, 6Labs, 10Tool-Labs, 10wikitech.wikimedia.org: Enable Flow for Tool namespace on wikitech - https://phabricator.wikimedia.org/T127792#2054198 (10bd808) [07:30:40] 3Collaboration-Team-Current, 6Editing-Department, 10Notifications, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15), 7user-notice: Automatically congratulate users for reaching certain edit count milestones - https://phabricator.wikimedia.org/T124003#2054324 (10TheDJ) @Ricordisamoa you are not right, such a tag is... [07:43:11] 3Collaboration-Team-Current, 10Flow, 13Patch-For-Review, 5WMF-deploy-2016-02-09_(1.27.0-wmf.13): Update and retest the convertToText script - https://phabricator.wikimedia.org/T90075#2054329 (10matthiasmullie) >>! In T90075#2053005, @Nemo_bis wrote: >>>! In T90075#1989219, @matthiasmullie wrote: >> Yes, I... [08:35:53] 3Collaboration-Team-Current, 10Notifications, 7Performance: Echo makes too many round-trips to memcached - https://phabricator.wikimedia.org/T127687#2054437 (10Legoktm) [08:35:55] 3Collaboration-Team-Current, 10Notifications: Don't hide messages badge when user has never had messages - https://phabricator.wikimedia.org/T127731#2054438 (10Legoktm) [08:41:42] 6Collaboration-Team-Backlog, 10MediaWiki-Email, 10Notifications: Broken icon in message email notification - https://phabricator.wikimedia.org/T127794#2054451 (10Krenair) [08:56:30] 3Collaboration-Team-Current, 6Editing-Department, 10Notifications, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15), 7user-notice: Automatically congratulate users for reaching certain edit count milestones - https://phabricator.wikimedia.org/T124003#2054461 (10TheDJ) Actually.. without {T127311}, there is no way... [09:20:50] (03PS1) 10Legoktm: Always show the messages flyout icon [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272686 (https://phabricator.wikimedia.org/T127731) [09:21:37] (03PS2) 10Legoktm: Always show the messages flyout icon [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272686 (https://phabricator.wikimedia.org/T127731) [10:52:53] 6Collaboration-Team-Backlog, 10Notifications, 7Crosswiki: Cros-wiki notification opens on local wiki - https://phabricator.wikimedia.org/T127812#2054751 (10Josve05a) [10:56:17] 6Collaboration-Team-Backlog, 10Notifications: Provide items inside a bundle with a more concise text copy - https://phabricator.wikimedia.org/T127813#2054765 (10Pginer-WMF) [11:27:23] 6Collaboration-Team-Backlog, 10Flow, 6Community-Liaisons: Track Flow activation - https://phabricator.wikimedia.org/T115110#2054874 (10Trizek-WMF) p:5Triage>3Normal [11:37:04] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Adjust the mark as read "X" icon size - https://phabricator.wikimedia.org/T124025#2054915 (10Pginer-WMF) It looks good. Thanks! The result seems to fit quite well the reference dimensions (F3363922): {... [11:37:14] 6Collaboration-Team-Backlog, 10Notifications: Styling adjustments for the Notification Panel - https://phabricator.wikimedia.org/T119374#2054917 (10Pginer-WMF) [11:56:42] (03PS1) 10Sbisson: Notification: use speechBubbles icon for all talk pages [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272700 (https://phabricator.wikimedia.org/T56433) [11:57:14] 3Collaboration-Team-Current, 10Notifications: Show descriptions for the actions under the "..." menu on notifications - https://phabricator.wikimedia.org/T126738#2054979 (10Pginer-WMF) Currently we have: - Secondary actions: Those are defined by the extensions. They may appear inside the "..." menu on certain... [11:57:26] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 7Voice & Tone, 7WorkType-NewFunctionality: Notification messages for mentions assume article namespace, do not reflect other namespaces - https://phabricator.wikimedia.org/T56433#2054980 (10SBisson) >>! In T56433#2053453, @Etonkovidova wro... [11:58:40] (03PS3) 10Haritha28: Echo congragulations threshold should'nt be hardcoded [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272308 (https://phabricator.wikimedia.org/T127311) [12:02:45] 3Collaboration-Team-Current, 10Notifications: DomainException on Special:Notifications on mediawiki.org - https://phabricator.wikimedia.org/T127728#2051854 (10SBisson) The code that produced this exception doesn't exist anymore since https://gerrit.wikimedia.org/r/#/c/271297 I never thought the agent could be... [12:38:50] 6Collaboration-Team-Backlog, 10Notifications, 7Crosswiki: Cross-wiki notification opens on local wiki - https://phabricator.wikimedia.org/T127812#2055099 (10Aklapper) [12:41:44] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: [minor] Notification panel layout issues for multiline messages - https://phabricator.wikimedia.org/T125909#2055103 (10Pginer-WMF) >>! In T125909#2053899, @Mooeypoo wrote: > The commit above fixes #2 and **should** also fix #1 (I increased the... [13:21:42] 3Collaboration-Team-Current, 10MediaWiki-extensions-EducationProgram, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update EducationProgram notifications to new language and format - https://phabricator.wikimedia.org/T125692#2055196 (10SBisson) >>! In T125692#2053237, @jmat... [13:47:27] 6Collaboration-Team-Backlog, 10Flow, 10Collaboration-Community-Engagement, 3Liaisons-February-2016, and 2 others: Work on a satisfaction survey concerning Flow - https://phabricator.wikimedia.org/T125632#2055244 (10Trizek-WMF) [13:49:03] 6Collaboration-Team-Backlog, 10Flow, 10Collaboration-Community-Engagement, 6Community-Liaisons, and 2 others: [Tracking] Create a general public documentation for Flow - https://phabricator.wikimedia.org/T111367#2055252 (10Trizek-WMF) [13:49:05] 6Collaboration-Team-Backlog, 10Flow, 7Documentation: Create a page about (more) advanced Flow features - https://phabricator.wikimedia.org/T111529#2055248 (10Trizek-WMF) 5Open>3stalled Stalled until {T125632} is done. [13:49:10] 6Collaboration-Team-Backlog, 10Flow, 10Collaboration-Community-Engagement, 6Community-Liaisons, and 2 others: [Tracking] Create a general public documentation for Flow - https://phabricator.wikimedia.org/T111367#1602419 (10Trizek-WMF) 5Open>3stalled Stalled until {T125632} is done. [14:04:30] 6Collaboration-Team-Backlog, 10Notifications: Provide items inside a bundle with a more concise text copy - https://phabricator.wikimedia.org/T127813#2055303 (10Trizek-WMF) Have you considered to shorten the sentences: - 4 new replies to “How the Moai moved?” -- Ludmilla: "I don't think so..." -- Fama: "Moais... [14:10:53] 6Collaboration-Team-Backlog, 10Notifications: Provide items inside a bundle with a more concise text copy - https://phabricator.wikimedia.org/T127813#2055312 (10Pginer-WMF) >>! In T127813#2055303, @Trizek-WMF wrote: > Have you considered to shorten the sentences: The main point was to highlight that we need... [14:22:04] (03PS4) 10Sbisson: [WIP] Include cross-wiki notifications in unread count [extensions/Echo] - 10https://gerrit.wikimedia.org/r/271690 (https://phabricator.wikimedia.org/T124109) [14:44:45] (03PS2) 10Zfilipin: Disable scenarios that fail at en.wikipedia.beta.wmflabs.org from running daily [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272471 (https://phabricator.wikimedia.org/T94153) [14:46:08] (03CR) 10Zfilipin: "Patch set 2 is just a manual rebase." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272471 (https://phabricator.wikimedia.org/T94153) (owner: 10Zfilipin) [15:08:45] (03PS1) 10Matthias Mullie: Add secondary links for edited topic/post on user talk pages [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272730 (https://phabricator.wikimedia.org/T121663) [15:23:47] (03PS1) 10Matthias Mullie: Delete rows in echo_unread_wikis if there are no unread notifs [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272732 (https://phabricator.wikimedia.org/T127731) [15:23:55] (03CR) 10Matthias Mullie: [C: 032] Always show the messages flyout icon [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272686 (https://phabricator.wikimedia.org/T127731) (owner: 10Legoktm) [15:24:21] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Don't hide messages badge when user has never had messages - https://phabricator.wikimedia.org/T127731#2055417 (10matthiasmullie) a:3matthiasmullie [15:34:30] (03Merged) 10jenkins-bot: Always show the messages flyout icon [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272686 (https://phabricator.wikimedia.org/T127731) (owner: 10Legoktm) [15:45:02] (03CR) 10Sbisson: "I think both 'view changes' links are conceptually the same. Having only one message would be less translation work and more consistency, " [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272730 (https://phabricator.wikimedia.org/T121663) (owner: 10Matthias Mullie) [16:19:13] (03CR) 10Catrope: [C: 04-1] "As Sbisson said, the messages are duplicative. There's also a "view changes" message in Echo already." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272730 (https://phabricator.wikimedia.org/T121663) (owner: 10Matthias Mullie) [16:21:20] RoanKattouw: how can I find on which pages a template is being used on en.wp? [16:21:50] stephanebisson: Special:Whatlinkshere [16:22:26] ok, it shows almost nothing so I thought I was at the wrong place.. thanks [16:23:11] Out of curiosity, which template are you looking at? [16:26:37] AfD-notice-NPF [16:27:03] trying to find real data to answer https://phabricator.wikimedia.org/T125693#2053719 [16:27:15] feel free to chime in [16:29:23] ha [16:30:17] I wonder if it's subst-ed instead [16:32:59] it actually is [16:33:22] {{subst:prod|reviewer notes here}} [16:33:38] that's a different template but same concept [16:39:49] (03CR) 10Esanders: [C: 032] "It seems like this should be consolidated with ve.ui.TargetWidget." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/234347 (owner: 10Catrope) [16:41:58] stephanebisson: Yeah, subst-ing isn't trackable unfortunately [16:42:18] Because {{subst:}} triggers a pre-save transformation that really just replaces it with the expansion of the template, leaving no trace behind that it was originally a subst [16:42:19] RoanKattouw: it is with search ;) [16:42:48] Oh, haha yes [16:43:46] not super reliable but good enough here [16:45:34] 6Collaboration-Team-Backlog, 10Notifications, 7Easy, 13Patch-For-Review: Echo congratulations thresholds shouldn't be hard-coded - https://phabricator.wikimedia.org/T127311#2055708 (10Jdforrester-WMF) >>! In T127311#2054197, @MZMcBride wrote: >>>! In T127311#2053337, @Jdforrester-WMF wrote: >> The array (w... [16:45:38] 6Collaboration-Team-Backlog, 10Notifications: A link from a cross-wiki Thanks notification leads to the current wiki instead of the correct target remote wiki - https://phabricator.wikimedia.org/T127843#2055709 (10Amire80) [16:46:45] 6Collaboration-Team-Backlog, 10Notifications: A link from a cross-wiki Thanks notification leads to the current wiki instead of the correct target remote wiki - https://phabricator.wikimedia.org/T127843#2055725 (10Catrope) [16:46:47] 3Collaboration-Team-Current, 10Thanks, 10Notifications: Cross-wiki notifications: Links for thanks notifications point to the local wiki - https://phabricator.wikimedia.org/T125738#2055726 (10Catrope) [16:48:16] at some point I was wondering about 'getLocalURL' vs 'getFullURL'... not anymore [16:48:45] haha [16:48:59] Oh is that why, that makes sense actually [16:49:26] I mostly chose getFullURL, that's why we have this bug with Thanks and not everywhere [16:49:36] (03Merged) 10jenkins-bot: VisualEditorWidget: Use selectLastContentOffset() from VE directly [extensions/Flow] - 10https://gerrit.wikimedia.org/r/234347 (owner: 10Catrope) [16:50:16] 3Collaboration-Team-Current, 10Thanks, 10Notifications: Cross-wiki notifications: Links for thanks notifications point to the local wiki - https://phabricator.wikimedia.org/T125738#2055740 (10Mattflaschen) a:5Mattflaschen>3None [16:50:51] stephanebisson, before I got bogged down on board moves, I was thinking of fixing it in Echo with wfExpandUrl. [16:50:52] (03PS2) 10Matthias Mullie: Add secondary links for edited topic/post on user talk pages [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272730 (https://phabricator.wikimedia.org/T121663) [16:51:12] It's already hit two notification types (thanks and revert), so by a fool-proof fix in Echo, we prevent it from affecting other types in the future as they're added. [16:51:23] matt_flaschen: sure [16:51:55] matt_flaschen: are you saying you're pretty busy but it's important so you would ike me to fix it? [16:52:51] stephanebisson, yeah, you can. I unclaimed it, since I was planning to start it yesterday but never got to it since the zhwiki board move thing expanded in scope. [16:53:38] matt_flaschen: I'll do it and we can swap it out. It's been reported 4 times already. [16:54:15] stephanebisson, sounds good, you can list me for the SWAT if you want. I'll be here. [16:54:18] (03CR) 10Matthias Mullie: [C: 04-1] Add secondary links for edited topic/post on user talk pages [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272730 (https://phabricator.wikimedia.org/T121663) (owner: 10Matthias Mullie) [16:54:51] 3Collaboration-Team-Current, 10Notifications: Notification panel: Clearer use of notification badges - https://phabricator.wikimedia.org/T115845#2055786 (10Pginer-WMF) > Mooeypoo assigned this task to Pginer-WMF. As @jmatazzoni mentioned, the discussion about reorganising the notifications has now its own tic... [16:55:00] 3Collaboration-Team-Current, 10Notifications: Notification panel: Clearer use of notification badges - https://phabricator.wikimedia.org/T115845#2055788 (10Pginer-WMF) a:5Pginer-WMF>3None [16:55:51] Oh yes, getFullURL doesn't return a full URL right? [16:56:14] RoanKattouw_away, no I think it does. [16:56:31] (03PS1) 10Matthias Mullie: Add GENDER support to 'View changes' [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272760 [16:56:32] RoanKattouw_away, but I was saying to call wfExpandUrl in Echo in case some notification type forgot to call getFullURL. [16:56:45] (03PS3) 10Matthias Mullie: Add secondary links for edited topic/post on user talk pages [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272730 (https://phabricator.wikimedia.org/T121663) [16:57:27] mlitn, could you take a look at https://phabricator.wikimedia.org/T127785 ? I think I figured out what's going on. I have one proposed solution, but it requires a core change, and I could use a second set of eyes. [16:58:09] sure [17:00:16] 6Collaboration-Team-Backlog, 10Flow, 6Labs, 10Tool-Labs, 10wikitech.wikimedia.org: Enable Flow for Tool namespace on wikitech - https://phabricator.wikimedia.org/T127792#2054159 (10Mattflaschen) Are we able to use extension1 here (that's where we normally have flowdb, except private wikis)? I know wikit... [17:00:54] 6Collaboration-Team-Backlog, 10Flow, 6Labs, 10Tool-Labs, 10wikitech.wikimedia.org: Enable Flow for Tool namespace on wikitech - https://phabricator.wikimedia.org/T127792#2054159 (10Krenair) No, wikitech cannot depend on extension1. [17:01:51] matt_flaschen: I’m not too familiar with those bits of code, so I’ll take a closer look [17:02:05] but at first glance, what you’re saying seems to make perfect sense [17:04:07] 6Collaboration-Team-Backlog, 10Flow, 6Labs, 10Tool-Labs, 10wikitech.wikimedia.org: Enable Flow for Tool namespace on wikitech - https://phabricator.wikimedia.org/T127792#2055826 (10jcrespo) Flow can be enabled, AFAIK, but it will have to live on its own database (like officewiki or some other one?). [17:04:38] 6Collaboration-Team-Backlog, 10Flow, 6Labs, 10Tool-Labs, 10wikitech.wikimedia.org: Enable Flow for Tool namespace on wikitech - https://phabricator.wikimedia.org/T127792#2055842 (10Mattflaschen) [17:06:49] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Making a null edit triggers a new message notification although null edit is not recorded in page history - https://phabricator.wikimedia.org/T123843#1939228 (10Etonkovidova) @matthiasmullie - there is... [17:10:56] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Making a null edit triggers a new message notification although null edit is not recorded in page history - https://phabricator.wikimedia.org/T123843#2055893 (10matthiasmullie) Yeah, I only fixed up Fl... [17:16:24] legoktm and/or RoanKattouw_away, could you review https://gerrit.wikimedia.org/r/#/c/250460/ (computed DB list for Flow)? It's been sitting around, and it would be good to SWAT that out. We're already planning a SWAT today for an Echo fix. [17:18:28] 6Collaboration-Team-Backlog, 10Flow, 6Labs, 10Tool-Labs, 10wikitech.wikimedia.org: Enable Flow on wikitech (labswiki and labtestwiki), then turn on for Tool talk namespace - https://phabricator.wikimedia.org/T127792#2055916 (10Mattflaschen) [17:49:12] matt_flaschen: LGTM [17:49:32] (03PS1) 10Sbisson: Echo API returns absolute URLs in notification links [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272775 (https://phabricator.wikimedia.org/T125738) [17:50:50] 3Collaboration-Team-Current, 10Thanks, 10Notifications, 13Patch-For-Review: Cross-wiki notifications: Links for thanks notifications point to the local wiki - https://phabricator.wikimedia.org/T125738#2056045 (10SBisson) a:3SBisson [17:51:54] * RoanKattouw looks up what the default is for the second param to wfExpandUrl [17:51:59] I feel like I should know because I added it [17:53:06] I guess in an HTTPS-only world it doesn't really matter [17:53:34] The default is PROTO_CURRENT, that's probably fine. I seem to recall we were trying to use PROTO_CANONICAL in API output, maybe for cacheability reasons [17:54:03] ...but this is all user-specific (since notifs are private) so that shouldn't be a concern [17:54:28] (03PS2) 10Catrope: Echo API returns absolute URLs in notification links [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272775 (https://phabricator.wikimedia.org/T125738) (owner: 10Sbisson) [17:54:46] (03CR) 10Catrope: [C: 032] Echo API returns absolute URLs in notification links [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272775 (https://phabricator.wikimedia.org/T125738) (owner: 10Sbisson) [17:55:20] Hmm, shall we backport and SWAT that change, since it's pretty high-impact for XWN users? Thoughts? [17:55:28] RoanKattouw: yes [18:02:42] (03Merged) 10jenkins-bot: Echo API returns absolute URLs in notification links [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272775 (https://phabricator.wikimedia.org/T125738) (owner: 10Sbisson) [18:04:34] (03PS1) 10Catrope: Echo API returns absolute URLs in notification links [extensions/Echo] (wmf/1.27.0-wmf.13) - 10https://gerrit.wikimedia.org/r/272780 (https://phabricator.wikimedia.org/T125738) [18:04:51] (03PS1) 10Catrope: Echo API returns absolute URLs in notification links [extensions/Echo] (wmf/1.27.0-wmf.14) - 10https://gerrit.wikimedia.org/r/272781 (https://phabricator.wikimedia.org/T125738) [18:17:24] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2056272 (10SBisson) >>! In T125693#2053719, @jmatazzoni wro... [18:19:23] RoanKattouw, should I put up the dblist one for SWAT? [18:22:04] matt_flaschen: I already did [18:22:12] Along with the absolute URLs one [18:26:15] Okay, thanks. [18:35:23] legoktm: Meeting? [18:56:34] 6Collaboration-Team-Backlog, 10Flow: Log entries missing from Flow history pages - https://phabricator.wikimedia.org/T127770#2053432 (10Mattflaschen) Is this actually a regression? I said at T90071: "I think this is fixed except {T96455}. E.g. topic deletions now go in history." If it's not a regression, not... [19:03:04] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2056443 (10jmatazzoni) @SBisson wrote: > ...it would requi... [19:04:22] 6Collaboration-Team-Backlog, 10Flow: Log entries missing from Flow history pages - https://phabricator.wikimedia.org/T127770#2056445 (10Mattflaschen) >>! In T127770#2056396, @Mattflaschen wrote: > If it's not a regression, not sure it's actually urgent if Special:Log is right. I checked, it is: https://en.wik... [19:05:15] 6Collaboration-Team-Backlog, 10Flow: Log entries missing from Flow history pages - https://phabricator.wikimedia.org/T127770#2056447 (10Mattflaschen) >>! In T127770#2056396, @Mattflaschen wrote: > Is this actually a regression? I said at T90071: > > "I think this is fixed except {T96455}. E.g. topic deletions... [19:05:35] 6Collaboration-Team-Backlog, 10Flow: Log entries missing from Flow history pages - https://phabricator.wikimedia.org/T127770#2056448 (10Mattflaschen) [19:10:08] Getting lunch [19:10:51] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2056462 (10jmatazzoni) I want to close this ticket, but hav... [19:15:41] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2056470 (10SBisson) >>! In T125693#2056462, @jmatazzoni wro... [19:17:10] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2056474 (10SBisson) >>! In T125693#2056443, @jmatazzoni wro... [19:23:41] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2056491 (10jmatazzoni) @SBisson wrote; > It is not curre... [19:25:56] 6Collaboration-Team-Backlog, 10Notifications, 10OOjs-UI, 6UI-Standardization-Kanban: OOUI PopupWidgets don't have drop shadows in the MediaWiki theme - https://phabricator.wikimedia.org/T111889#2056499 (10ggellerman) [19:27:51] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2056536 (10SBisson) >>! In T125693#2056491, @jmatazzoni wro... [19:32:07] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2056587 (10jmatazzoni) We are done here. Closing this. Than... [19:32:16] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2056589 (10jmatazzoni) 5Open>3Resolved [19:37:20] 6Collaboration-Team-Backlog, 10Flow, 10Collaboration-Community-Engagement, 3Liaisons-February-2016, and 2 others: Work on a satisfaction survey concerning Flow - https://phabricator.wikimedia.org/T125632#2056649 (10egalvezwmf) hi! just a friendly reminder to post this survey on the engagement calendars: h... [20:05:14] 6Collaboration-Team-Backlog, 10MediaWiki-extensions-UniversalLanguageSelector, 10Notifications, 7Design, and 2 others: Use consistent preferences icon - https://phabricator.wikimedia.org/T52843#2056799 (10Volker_E) [20:05:18] 6Collaboration-Team-Backlog, 10Notifications, 10UI-Standardization, 7Design, 13Patch-For-Review: Use consistent preferences icon in Echo - https://phabricator.wikimedia.org/T76513#2056794 (10Volker_E) 5Open>3stalled p:5Normal>3Lowest [20:07:00] 6Collaboration-Team-Backlog, 10Notifications, 10UI-Standardization, 7Design, 13Patch-For-Review: Use consistent preferences icon in Echo - https://phabricator.wikimedia.org/T76513#802264 (10Volker_E) Citing @Mooeypoo from PS 7 > Code-Review-1 > I am not sure this is the right thing to do anymore. We ar... [20:25:45] 6Collaboration-Team-Backlog, 10Notifications, 7Design, 7Easy: Notification icons are not ready for high-density displays (use SVG instead of PNG where supported) - https://phabricator.wikimedia.org/T55875#565249 (10Volker_E) @Pginer-WMF How much true is this still? [20:26:44] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 7user-notice: Overflow on Notification panel with very long page names - https://phabricator.wikimedia.org/T123657#1935132 (10jmatazzoni) Closing this bug. Pretty sure it is being handled by T121822. Let me know if you disagree. [20:27:04] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 7user-notice: Overflow on Notification panel with very long page names - https://phabricator.wikimedia.org/T123657#2056920 (10jmatazzoni) 5Open>3Resolved [20:32:29] (03CR) 10Catrope: [C: 032] Add GENDER support to 'View changes' [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272760 (owner: 10Matthias Mullie) [20:45:38] (03Merged) 10jenkins-bot: Add GENDER support to 'View changes' [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272760 (owner: 10Matthias Mullie) [20:51:07] 3Collaboration-Team-Current, 10Flow, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Flow\Exception\FlowException from line 159 of Flow/includes/Formatter/ChangesListQuery.php: Revision not found in revisionCache - https://phabricator.wikimedia.org/T122329#2057076 (10Southparkfan) I use the lat... [20:58:59] 6Collaboration-Team-Backlog, 10Notifications, 7Crosswiki: Cross-wiki notification opens on local wiki - https://phabricator.wikimedia.org/T127812#2057118 (10Quiddity) [20:59:01] 3Collaboration-Team-Current, 10Thanks, 10Notifications, 13Patch-For-Review: Cross-wiki notifications: Links for thanks notifications point to the local wiki - https://phabricator.wikimedia.org/T125738#2057119 (10Quiddity) [21:10:45] (03CR) 10Legoktm: [C: 032] Delete rows in echo_unread_wikis if there are no unread notifs [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272732 (https://phabricator.wikimedia.org/T127731) (owner: 10Matthias Mullie) [21:11:00] 3Collaboration-Team-Current, 10Notifications, 7Performance: Echo makes too many round-trips to memcached - https://phabricator.wikimedia.org/T127687#2057190 (10Legoktm) [21:11:05] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Don't hide messages badge when user has never had messages - https://phabricator.wikimedia.org/T127731#2057189 (10Legoktm) 5Open>3Resolved [21:15:33] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-02-09_(1.27.0-wmf.13), 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): If user only has foreign messages, the messages badge should not be suppressed - https://phabricator.wikimedia.org/T124372#2057247 (10Legoktm) 5Open>3Resolv... [21:18:14] (03Merged) 10jenkins-bot: Delete rows in echo_unread_wikis if there are no unread notifs [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272732 (https://phabricator.wikimedia.org/T127731) (owner: 10Matthias Mullie) [21:27:31] 3Collaboration-Team-Current, 10Notifications: Email's footer is partially displayed in Email Notifications for short email messages - https://phabricator.wikimedia.org/T127098#2057287 (10SBisson) a:3SBisson [21:27:44] (03PS1) 10Sbisson: Remove footer from email notification excerpt [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272836 (https://phabricator.wikimedia.org/T127098) [21:31:16] 3Collaboration-Team-Current, 10Notifications: User-right notifications - provide log access - https://phabricator.wikimedia.org/T127249#2057309 (10SBisson) a:3SBisson [21:31:43] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Email's footer is partially displayed in Email Notifications for short email messages - https://phabricator.wikimedia.org/T127098#2057315 (10SBisson) p:5Triage>3Normal [21:32:14] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Cross-wiki notifications: Links for reverts notifications point to the local wiki - https://phabricator.wikimedia.org/T127697#2050858 (10SBisson) a:3SBisson [21:45:06] 3Collaboration-Team-Current, 10Flow, 13Patch-For-Review: Convert LQT->Flow on fiwikimedia - https://phabricator.wikimedia.org/T127576#2057409 (10Catrope) 5Open>3Resolved [21:45:08] 3Collaboration-Team-Current, 10Flow: Convert LQT to Flow on remaining wikis - https://phabricator.wikimedia.org/T127575#2057410 (10Catrope) [21:52:30] 6Collaboration-Team-Backlog, 10Notifications: Show descriptions for the actions under the "..." menu on notifications - https://phabricator.wikimedia.org/T126738#2057451 (10jmatazzoni) [23:07:34] 3Collaboration-Team-Current, 10Flow, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Mentions in board description and topic summary don't generate flow-mention notifications - https://phabricator.wikimedia.org/T125120#1978916 (10Etonkovidova) Checked in betalabs - for details on testing, scree... [23:10:36] 6Collaboration-Team-Backlog, 10Notifications, 7WorkType-NewFunctionality: Clarify read/unread status for notifications - https://phabricator.wikimedia.org/T126214#2057845 (10jmatazzoni) > A checkmark represents a more "final" state I think this is a subtle but interesting point. Either of these models co... [23:20:33] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Email's footer is partially displayed in Email Notifications for short email messages - https://phabricator.wikimedia.org/T127098#2057973 (10jmatazzoni) @Etonkovidova, What is the purpose of showing part of the footer? Especially if, as in th... [23:35:07] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Email's footer is partially displayed in Email Notifications for short email messages - https://phabricator.wikimedia.org/T127098#2032361 (10Quiddity) >>! In T127098#2057973, @jmatazzoni wrote: > @Etonkovidova, What is the purpose of showing... [23:35:52] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Email's footer is partially displayed in Email Notifications for short email messages - https://phabricator.wikimedia.org/T127098#2058131 (10Etonkovidova) @jmatazzoni - the footer was shown because the email body is too short. The text excerp...