[00:19:04] (03CR) 10Jdlrobson: [C: 031] Add MobileNotificationsWrapper for Mobile display [extensions/Echo] - 10https://gerrit.wikimedia.org/r/251561 (https://phabricator.wikimedia.org/T124188) (owner: 10Mooeypoo) [00:19:05] 6Collaboration-Team-Backlog, 10Notifications, 7Epic: Echo: Provide the new bundling system replacing the existing hacky one - https://phabricator.wikimedia.org/T120153#2121003 (10jmatazzoni) [00:19:10] 6Collaboration-Team-Backlog, 10Notifications, 7Epic, 7WorkType-NewFunctionality, 7user-notice: Notifications Panel: Support cross-wiki notifications - https://phabricator.wikimedia.org/T114350#2121004 (10jmatazzoni) [00:19:12] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 7WorkType-NewFunctionality: Implement expandable notification groups - https://phabricator.wikimedia.org/T115423#2121002 (10jmatazzoni) 5Open>3Resolved [00:19:47] 3Collaboration-Team-Current, 10MediaWiki-Vagrant, 10Notifications: Setup cross-wiki notifications in MediaWiki-Vagrant - https://phabricator.wikimedia.org/T119999#2121005 (10Mooeypoo) The image looks a bit weird to me; there should not be tags or {{FlowMention|...}} renders in the description or text.... [00:22:07] 3Collaboration-Team-Current, 10Flow, 10Notifications: Double escaping in Flow notifications - https://phabricator.wikimedia.org/T125130#2121009 (10jmatazzoni) 5Open>3Resolved [00:31:28] (03CR) 10Mooeypoo: [C: 032] "oops. Good catch!" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277426 (owner: 10Legoktm) [00:47:22] (03Merged) 10jenkins-bot: Have mw.echo.api.APIHandler.markItemRead actually return a promise [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277426 (owner: 10Legoktm) [00:51:21] 3Collaboration-Team-Current, 10MediaWiki-Vagrant, 10Notifications: Setup cross-wiki notifications in MediaWiki-Vagrant - https://phabricator.wikimedia.org/T119999#1842363 (10jmatazzoni) As far as I can tell, this particular ticket is satisfied. @Mooeypoo , @Etonkovidova, do you need to open a separate bug... [00:55:52] 3Collaboration-Team-Current, 10Flow, 10MediaWiki-General-or-Unknown, 7I18n, and 2 others: [[MediaWiki:Notification-links-flow-description-edited-view-page/qqq]] i18n issue - https://phabricator.wikimedia.org/T129351#2121048 (10jmatazzoni) 5Open>3Resolved [00:56:13] 3Collaboration-Team-Current, 10MediaWiki-Vagrant, 10Notifications: Setup cross-wiki notifications in MediaWiki-Vagrant - https://phabricator.wikimedia.org/T119999#2121050 (10Mooeypoo) The only thing I want to make sure in this case is whether this is a general bug or a vagrant specific bug. If it's a Vagrant... [01:00:46] 6Collaboration-Team-Backlog, 10Notifications: Styling adjustments for the Notification Panel - https://phabricator.wikimedia.org/T119374#2121060 (10jmatazzoni) [01:14:32] (03CR) 10Catrope: "I'm not very happy with the COUNT(*) query, I dislike those in general. There's no way to limit how many rows such a query will need to ex" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277242 (owner: 10Matthias Mullie) [01:39:30] 6Collaboration-Team-Backlog, 10Notifications, 13Patch-For-Review: Notifications panel: Easily explore bundled notifications - https://phabricator.wikimedia.org/T114356#2121155 (10Catrope) >>! In T114356#2050550, @gerritbot wrote: > Change 272444 had a related patch set uploaded (by Catrope): > [WIP] Expandab... [02:45:53] 6Collaboration-Team-Backlog, 10Notifications, 6Wikisource, 7Crosswiki: Change "Error Access to the remote domain was denied." message? - https://phabricator.wikimedia.org/T129764#2121228 (10Mattflaschen) >>! In T129764#2118871, @AuFCL wrote: > Thanks @Mattflaschen. All good to know and makes sense now the... [02:46:07] 6Collaboration-Team-Backlog, 10Notifications, 6Wikisource, 7Crosswiki: Change "Error Access to the remote domain was denied." (echo-api-failure-cross-wiki) message and use errorObj? - https://phabricator.wikimedia.org/T129764#2121229 (10Mattflaschen) [02:58:11] 3Collaboration-Team-Current, 10MediaWiki-Vagrant, 10Notifications: Setup cross-wiki notifications in MediaWiki-Vagrant - https://phabricator.wikimedia.org/T119999#2121238 (10Mattflaschen) >>! In T119999#2121005, @Mooeypoo wrote: > The image looks a bit weird to me; there should not be tags or {{FlowMent... [02:59:29] Heading out. Have a good night. [03:02:40] 6Collaboration-Team-Backlog, 10Notifications, 6Wikisource, 7Crosswiki: Change "Error Access to the remote domain was denied." (echo-api-failure-cross-wiki) message and use errorObj? - https://phabricator.wikimedia.org/T129764#2121248 (10AuFCL) >>! In T129764#2121228, @Mattflaschen wrote: >>>! In T129764#21... [05:12:39] 6Collaboration-Team-Backlog, 10Notifications, 6Wikisource, 7Crosswiki: Change "Error Access to the remote domain was denied." (echo-api-failure-cross-wiki) message and use errorObj? - https://phabricator.wikimedia.org/T129764#2115268 (10Jay8g) I've been getting this on (highly customized) Firefox. Again, I... [07:45:22] 6Collaboration-Team-Backlog, 10Collaboration-Community-Engagement: Explore ways to avoid cross-wiki spam from welcome bots - https://phabricator.wikimedia.org/T129904#2119451 (10Elitre) You probably want to revisit the cross-wiki behavior then. Several wikis welcome users on their talk page automatically when... [08:16:49] 6Collaboration-Team-Backlog, 10Notifications, 13Patch-For-Review: Notifications panel: Easily explore bundled notifications - https://phabricator.wikimedia.org/T114356#2121603 (10Pginer-WMF) > Did we decide what happens to a sub-notification when it's marked as read? Does it stay in the bundle, or does it de... [09:36:37] 3Collaboration-Team-Current, 10Flow, 10Dumps-Generation, 13Patch-For-Review, and 2 others: Add Flow to database dumps - https://phabricator.wikimedia.org/T89398#2121749 (10ArielGlenn) [09:40:08] 6Collaboration-Team-Backlog, 10MoodBar: MoodBar registers a dependency on a key that isn't specified - https://phabricator.wikimedia.org/T129978#2121765 (10Jdforrester-WMF) [09:40:34] 6Collaboration-Team-Backlog, 10Flow: Flow registers a dependency on a key that isn't specified - https://phabricator.wikimedia.org/T129979#2121779 (10Jdforrester-WMF) [09:42:24] 6Collaboration-Team-Backlog, 10MediaWiki-extensions-PageCuration: PageTriage registers a dependency on keys that aren't specified - https://phabricator.wikimedia.org/T129981#2121810 (10Jdforrester-WMF) [09:52:15] (03PS1) 10Jforrester: Drop dependency on 'feedbackresponse-success' key [extensions/MoodBar] - 10https://gerrit.wikimedia.org/r/277481 (https://phabricator.wikimedia.org/T129978) [11:54:34] 6Collaboration-Team-Backlog, 10Flow, 10Collaboration-Community-Engagement, 3Liaisons-March-2016, 7Surveys: Work on a satisfaction survey concerning Flow - https://phabricator.wikimedia.org/T125632#2122252 (10Trizek-WMF) [12:44:37] 6Collaboration-Team-Backlog, 10Notifications, 13Patch-For-Review: Notifications panel: Easily explore bundled notifications - https://phabricator.wikimedia.org/T114356#2122331 (10SBisson) I'm facing similar challenges in {T93673} when moderating a notification that is part of a bundle. The concept of bundle... [13:04:29] (03PS2) 10Zfilipin: Followup 9eaa667: Use @skip instead so it's more obvious and greppable [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277261 (https://phabricator.wikimedia.org/T94153) (owner: 10Mattflaschen) [13:09:14] 6Collaboration-Team-Backlog, 10Flow, 7I18n: [[MediaWiki:Flow-action-not-page/en]]: Missing preposition in "Page object is the wrong class" - https://phabricator.wikimedia.org/T129920#2122358 (10Aklapper) [14:05:54] 6Collaboration-Team-Backlog, 10Notifications, 13Patch-For-Review: Notifications panel: Easily explore bundled notifications - https://phabricator.wikimedia.org/T114356#2122417 (10Pginer-WMF) I don't know about the technical details, but dealing with bundle at the display level seems to provide some flexibili... [14:11:41] 6Collaboration-Team-Backlog, 10Flow, 10Collaboration-Community-Engagement, 10Wikimedia-Site-Requests: Enable Flow as a Beta feature on Polish Wikipedia - https://phabricator.wikimedia.org/T130009#2122424 (10Trizek-WMF) [14:22:28] 6Collaboration-Team-Backlog, 10Flow, 10Collaboration-Community-Engagement, 10Wikimedia-Site-Requests: Enable Flow as a Beta feature on Polish Wikipedia - https://phabricator.wikimedia.org/T130009#2122491 (10tarlocesilion) [15:01:20] 3Collaboration-Team-Current, 6Design-Research-Backlog, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2122587 (10Pginer-WMF) I think the current approach should work. We can emphasise the call to action in bold as we do for U... [15:31:09] (03CR) 10Alex Monk: [C: 032] Drop dependency on 'feedbackresponse-success' key [extensions/MoodBar] - 10https://gerrit.wikimedia.org/r/277481 (https://phabricator.wikimedia.org/T129978) (owner: 10Jforrester) [15:33:09] (03PS3) 10Zfilipin: Followup 9eaa667: Use @skip instead so it's more obvious and greppable [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277261 (https://phabricator.wikimedia.org/T94153) (owner: 10Mattflaschen) [15:33:56] (03CR) 10Zfilipin: "Patch set 2 just adds phabricator task to the commit message." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277261 (https://phabricator.wikimedia.org/T94153) (owner: 10Mattflaschen) [15:37:53] (03CR) 10Zfilipin: "Patch set 3 is refactoring cucumber tags:" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277261 (https://phabricator.wikimedia.org/T94153) (owner: 10Mattflaschen) [15:39:31] 3Collaboration-Team-Current, 10Collaboration-Community-Engagement, 10WMF-Product-Development-Process, 7user-notice: Cross-wiki notifications - announcements planning - https://phabricator.wikimedia.org/T123570#2122783 (10Trizek-WMF) [15:41:34] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: 'Mark as read' dots menu is misaligned - https://phabricator.wikimedia.org/T126686#2122806 (10Pginer-WMF) >>! In T126686#2119734, @jmatazzoni wrote: > @Pginer-wmf, this looks about right though I'm not sure it's hitting your spec in all cases... [15:53:15] (03CR) 10Zfilipin: "The changes are tested here:" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277261 (https://phabricator.wikimedia.org/T94153) (owner: 10Mattflaschen) [15:58:43] (03Merged) 10jenkins-bot: Drop dependency on 'feedbackresponse-success' key [extensions/MoodBar] - 10https://gerrit.wikimedia.org/r/277481 (https://phabricator.wikimedia.org/T129978) (owner: 10Jforrester) [16:08:55] 3Collaboration-Team-Current, 10Flow, 10Collaboration-Community-Engagement, 10Wikimedia-Site-Requests: Enable Flow as a Beta feature on Polish Wikipedia - https://phabricator.wikimedia.org/T130009#2122873 (10Catrope) a:3Catrope [16:27:41] 3Collaboration-Team-Current, 10Flow, 10Collaboration-Community-Engagement, 10Wikimedia-Site-Requests, 13Patch-For-Review: Enable Flow as a Beta feature on Polish Wikipedia - https://phabricator.wikimedia.org/T130009#2122927 (10Catrope) Scheduled for deployment at 00:00 UTC (6.5 hours from now) [16:37:35] (03PS1) 10Matthias Mullie: Block undeleting an article over an existing Flow board [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277536 (https://phabricator.wikimedia.org/T104591) [16:38:01] 3Collaboration-Team-Current, 10Flow, 13Patch-For-Review: Deleted wikitext Talk page cannot be restored after EnableFlow creates the same page - https://phabricator.wikimedia.org/T104591#2122982 (10matthiasmullie) a:3matthiasmullie [16:39:23] 6Collaboration-Team-Backlog, 10Notifications: [betalabs] Regression: 'Mark as read' for cross wiki bundled notifications is not persistent - https://phabricator.wikimedia.org/T130026#2122996 (10Etonkovidova) [16:40:00] 3Collaboration-Team-Current, 10Flow: Improve Flow deletion/undeletion resilience - https://phabricator.wikimedia.org/T122262#2123009 (10matthiasmullie) a:3matthiasmullie [16:50:51] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: 'Mark as read' dots menu is misaligned - https://phabricator.wikimedia.org/T126686#2123080 (10jmatazzoni) > It seems the expand/collapse action is not treated the same as other actions. Adding the missing margin to it would help making things... [17:12:15] (03PS7) 10Mooeypoo: Add mark-as-read button to notifications in Special:Notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/276256 (https://phabricator.wikimedia.org/T115528) [17:18:46] 3Collaboration-Team-Current, 10MediaWiki-Vagrant, 10Notifications: Setup cross-wiki notifications in MediaWiki-Vagrant - https://phabricator.wikimedia.org/T119999#2123172 (10Mattflaschen) @Etonkovidova said these were topic titles. [17:19:46] (03CR) 10jenkins-bot: [V: 04-1] Add mark-as-read button to notifications in Special:Notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/276256 (https://phabricator.wikimedia.org/T115528) (owner: 10Mooeypoo) [17:19:49] 3Collaboration-Team-Current, 6Design-Research-Backlog, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2123174 (10Capt_Swing) @Pginer-WMF @jmatazzoni looking good! A few questions and requests: # per @egalvezwmf's comment a... [17:31:49] 6Collaboration-Team-Backlog, 10Collaboration-Community-Engagement, 10Notifications: Explore ways to avoid cross-wiki spam from welcome bots - https://phabricator.wikimedia.org/T129904#2123230 (10Quiddity) [17:50:10] 6Collaboration-Team-Backlog, 3Collaboration-Team-Current, 10Flow, 10VisualEditor, 10VisualEditor-CopyPaste: Don't strip links when pasting HTML into Flow VisualEditor - https://phabricator.wikimedia.org/T129833#2123373 (10Jdforrester-WMF) p:5Triage>3Low a:3Esanders [17:50:20] 6Collaboration-Team-Backlog, 3Collaboration-Team-Current, 10Flow, 10VisualEditor, 10VisualEditor-CopyPaste: Don't strip links when pasting HTML into Flow VisualEditor - https://phabricator.wikimedia.org/T129833#2117358 (10Jdforrester-WMF) Leaving to Collaboration to mark as resolved. [18:07:17] Getting lunch [18:18:12] 3Collaboration-Team-Current, 6Design-Research-Backlog, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2123526 (10Pginer-WMF) >>! In T128937#2123174, @Capt_Swing wrote: > @Pginer-WMF @jmatazzoni looking good! A few questions a... [18:49:53] 6Collaboration-Team-Backlog, 10MoodBar, 13Patch-For-Review: MoodBar registers a dependency on a message key that isn't specified - https://phabricator.wikimedia.org/T129978#2123641 (10Krinkle) [18:50:00] 6Collaboration-Team-Backlog, 10MoodBar, 13Patch-For-Review: MoodBar registers a dependency on a message key that isn't specified - https://phabricator.wikimedia.org/T129978#2121765 (10Krinkle) 5Open>3Resolved a:3Krinkle [18:56:46] (03PS1) 10Krinkle: Drop dependency on 'feedbackresponse-success' key [extensions/MoodBar] (wmf/1.27.0-wmf.16) - 10https://gerrit.wikimedia.org/r/277565 (https://phabricator.wikimedia.org/T129978) [18:59:28] 3Collaboration-Team-Current, 10Notifications, 7Community-Wishlist-Survey, 7Design, and 4 others: Allow marking Echo notifications as unread - https://phabricator.wikimedia.org/T73564#2123690 (10Etonkovidova) >For alerts, marking as unread keeps the notification as unread even if the panel is opened/closed... [18:59:29] (03CR) 10Krinkle: [C: 032] Drop dependency on 'feedbackresponse-success' key [extensions/MoodBar] (wmf/1.27.0-wmf.16) - 10https://gerrit.wikimedia.org/r/277565 (https://phabricator.wikimedia.org/T129978) (owner: 10Krinkle) [19:11:48] Where are the namespace restrictions for mention notifications defined, code-wise? (IIRC it varies per-wiki, with some like Metawiki allowing it in mainspace? Or I might be mixing it up with something else...) [19:12:07] 6Collaboration-Team-Backlog, 10MediaWiki-Email, 10MediaWiki-User-preferences, 10MediaWiki-extensions-PasswordReset, 10Notifications: Proposal: allow a second email address (wikimail, password reset, notifications) - https://phabricator.wikimedia.org/T129747#2123830 (10csteipp) [19:15:48] (03Merged) 10jenkins-bot: Drop dependency on 'feedbackresponse-success' key [extensions/MoodBar] (wmf/1.27.0-wmf.16) - 10https://gerrit.wikimedia.org/r/277565 (https://phabricator.wikimedia.org/T129978) (owner: 10Krinkle) [19:27:59] 3Collaboration-Team-Current, 6Design-Research-Backlog, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2124016 (10Capt_Swing) Actually @Pginer-WMF, I've received new information from @APalmer_WMF that suggests perhaps we don't... [20:00:24] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Add secondary link for the agent to some notification types - https://phabricator.wikimedia.org/T121737#2124120 (10Etonkovidova) Checked according to @SBisson comments - all specs are in place. >I think the primary action for 'flow-new-topic... [20:14:04] matt_flaschen: intersting that Notification primary links do redirection correctly if a page has been moved - it's great [20:30:08] quiddity: namespace restrictions were removed [20:30:36] ah ty! [20:40:33] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Add secondary link for the agent to some notification types - https://phabricator.wikimedia.org/T121737#2124301 (10jmatazzoni) 5Open>3Resolved [20:40:35] 3Collaboration-Team-Current, 10Notifications, 7I18n, 13Patch-For-Review: Clean up lego message in user-rights notification - https://phabricator.wikimedia.org/T121661#2124303 (10jmatazzoni) [20:40:49] fyi https://wikitech.wikimedia.org/wiki/X-Wikimedia-Debug for read only mode testing [20:45:30] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-29_(1.27.0-wmf.18), 7Wikimedia-log-errors: BadMethodCallException in EditThresholdPresentationModel.php - https://phabricator.wikimedia.org/T129641#2111502 (10Etonkovidova) Checked in betalabs - if a page was deleted, a... [20:47:56] 3Collaboration-Team-Current, 6Design-Research-Backlog, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2124366 (10jmatazzoni) Thanks for your new designs Pau. I feel this one hits a very nice balance of looking harmonious in t... [21:14:44] (03PS1) 10Sbisson: [WIP] Stop counting notifications objects on the client [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277609 (https://phabricator.wikimedia.org/T129726) [21:18:04] (03PS10) 10Sbisson: [WIP] Moderate notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275049 (https://phabricator.wikimedia.org/T93673) [21:19:09] (03PS1) 10Mattflaschen: Message tweak for flow-action-not-page [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277611 (https://phabricator.wikimedia.org/T129920) [21:19:36] 6Collaboration-Team-Backlog, 10Flow, 7I18n, 13Patch-For-Review: [[MediaWiki:Flow-action-not-page/en]]: Missing preposition in "Page object is the wrong class" - https://phabricator.wikimedia.org/T129920#2120040 (10Mattflaschen) Yes, this is a very technical error message that will hopefully never occur. U... [21:21:45] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Stop counting notifications objects on the client [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277609 (https://phabricator.wikimedia.org/T129726) (owner: 10Sbisson) [21:22:33] (03CR) 10Mattflaschen: [C: 04-1] "That is failing" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277261 (https://phabricator.wikimedia.org/T94153) (owner: 10Mattflaschen) [21:26:19] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Moderate notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275049 (https://phabricator.wikimedia.org/T93673) (owner: 10Sbisson) [21:49:14] 3Collaboration-Team-Current, 10Flow, 10Datasets-General-or-Unknown, 13Patch-For-Review, 7WorkType-Maintenance: Create XML Schema for Flow dump format(s) - https://phabricator.wikimedia.org/T112799#2124554 (10Mattflaschen) In addition to validation, XSDs can also be used for code generation. We should pr... [21:52:29] RoanKattouw, jmatazzoni__, et al, any suggestions on what to put for Scrum of Scrums. I started it at https://etherpad.wikimedia.org/p/Scrum-of-Scrums [21:54:57] 3Collaboration-Team-Current, 10Collaboration-Team-Sprint-D-2015-06-30, 10Collaboration-Team-Sprint-E-Everywhere-2015-07-14, 10Flow, and 4 others: E10. Have Flow use ExternalStore on MediaWiki-Vagrant - https://phabricator.wikimedia.org/T95870#2124608 (10Mattflaschen) Vagrant patch itself (https://gerrit.wi... [21:57:05] 3Collaboration-Team-Current, 10Collaboration-Team-Sprint-E-Everywhere-2015-07-14, 10Flow, 13Patch-For-Review, 7WorkType-Maintenance: Migrate Flow content to new separate logical External Store - https://phabricator.wikimedia.org/T106363#2124611 (10Mattflaschen) [21:57:07] 3Collaboration-Team-Current, 10Collaboration-Team-Sprint-D-2015-06-30, 10Collaboration-Team-Sprint-E-Everywhere-2015-07-14, 10Flow, and 4 others: E10. Have Flow use ExternalStore on MediaWiki-Vagrant - https://phabricator.wikimedia.org/T95870#1202103 (10Mattflaschen) [22:22:11] 3Collaboration-Team-Current, 10Flow, 10Collaboration-Community-Engagement, 10Wikimedia-Site-Requests, 13Patch-For-Review: Enable Flow as a Beta feature on Polish Wikipedia - https://phabricator.wikimedia.org/T130009#2122424 (10Wostr) There is no community consensus about activating this feature. The disc... [22:32:12] 3Collaboration-Team-Current, 6Design-Research-Backlog, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2124763 (10Capt_Swing) [22:41:44] matt_flaschen: Re SoS, what's up with Flow dumps, are we blocked on or blocking somebody with that? [22:46:15] 3Collaboration-Team-Current, 10Flow, 10Datasets-Archiving, 10Dumps-Generation, 7Blocked-on-Operations: Publish recurring Flow dumps at http://dumps.wikimedia.org/ - https://phabricator.wikimedia.org/T119511#2124834 (10Mattflaschen) It has been merged. [22:46:39] legoktm: Could I get you to code review https://gerrit.wikimedia.org/r/276687? [22:47:27] RoanKattouw, we're still working with Ariel on them. I don't know if they retested after https://gerrit.wikimedia.org/r/#/c/275519/ was merged. See https://phabricator.wikimedia.org/T119511 . [22:48:15] RoanKattouw, also, Matthias is working on an XSD schema for the dump. Ariel is following that, but I don't know if it's really a blocker: https://phabricator.wikimedia.org/T112799 [22:48:29] OK [22:48:36] I've added it back to SoS, ping might not hurt. [22:49:50] (03PS1) 10Catrope: Fix brokenness in FlowPresentationModel::isUserTalkPage() [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277682 [22:50:00] Yeah sounds good [22:50:05] matt_flaschen: Also, can I get a quick review on ---^^ ? [22:50:22] RoanKattouw, sure [22:51:59] RoanKattouw, isn't getPrefixedText User:Catrope vs. Catrope ($username)? [22:54:00] (03CR) 10Mattflaschen: [C: 04-1] Fix brokenness in FlowPresentationModel::isUserTalkPage() (031 comment) [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277682 (owner: 10Catrope) [22:55:15] 3Collaboration-Team-Current, 10Flow, 10Collaboration-Community-Engagement, 10Wikimedia-Site-Requests, 13Patch-For-Review: Enable Flow as a Beta feature on Polish Wikipedia - https://phabricator.wikimedia.org/T130009#2124865 (10Catrope) >>! In T130009#2124722, @Wostr wrote: > There is no community consens... [22:58:01] Ahm, yes, it is [22:58:04] Good point [22:58:24] (03PS1) 10Catrope: MentionPresentationModel: use Title::equals() [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277683 [22:58:33] RoanKattouw, I think you want getText() [22:58:35] (03PS2) 10Catrope: Fix brokenness in FlowPresentationModel::isUserTalkPage() [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277682 [22:58:38] Yup [22:59:04] Well what I really want is ->equals( $this->user->getTalkPage() ) but $this->user is private [22:59:06] * RoanKattouw looks at legoktm [22:59:10] RoanKattouw, jmatazzoni__, also, did you see the new inspire campaign is about content review and curation? Mentioniong this because it's relevant to things we've discussed for the roadmap. [22:59:24] matt_flaschen: Yeah I think Joe and Chris have talked about it [22:59:39] Or there's an email thread at least [23:00:50] (03CR) 10Mattflaschen: [C: 032] Fix brokenness in FlowPresentationModel::isUserTalkPage() [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277682 (owner: 10Catrope) [23:01:15] Thanks matt_flaschen [23:02:25] No problem, didn't test but it should work. [23:04:09] (03PS8) 10Mooeypoo: Add mark-as-read button to notifications in Special:Notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/276256 (https://phabricator.wikimedia.org/T115528) [23:14:10] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Usernames invisible in Echo notification text - https://phabricator.wikimedia.org/T129531#2108314 (10Etonkovidova) 1. {{ping|Username}} produces - notice the added colon `@Username:` {{FlowMention|Username}} will display `@Username` {{ {{{|... [23:15:13] (03Merged) 10jenkins-bot: Fix brokenness in FlowPresentationModel::isUserTalkPage() [extensions/Flow] - 10https://gerrit.wikimedia.org/r/277682 (owner: 10Catrope) [23:15:43] (03PS1) 10Mooeypoo: Apply buttons styling to 'expand' bundle button [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277689 (https://phabricator.wikimedia.org/T126686) [23:15:50] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: 'Mark as read' dots menu is misaligned - https://phabricator.wikimedia.org/T126686#2124927 (10Mooeypoo) Good point. I've fixed that in this commit. [23:18:45] (03CR) 10Catrope: [C: 032] Apply buttons styling to 'expand' bundle button [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277689 (https://phabricator.wikimedia.org/T126686) (owner: 10Mooeypoo) [23:19:00] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Usernames invisible in Echo notification text - https://phabricator.wikimedia.org/T129531#2124930 (10Etonkovidova) The test case to check after the fix: -on a wikitext User talk page create a new section with a title: `ὣ` - in the body... [23:28:24] (03PS1) 10Mooeypoo: Adjust bottom-alignment of action buttons in notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277690 (https://phabricator.wikimedia.org/T125969) [23:28:33] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Action and timestamp position needs adjustment - https://phabricator.wikimedia.org/T125969#2124960 (10Mooeypoo) >>! In T125969#2098313, @Pginer-WMF wrote: > It has improved with respect to the original situation, but @Etonkovidova points to tw... [23:34:52] (03Merged) 10jenkins-bot: Apply buttons styling to 'expand' bundle button [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277689 (https://phabricator.wikimedia.org/T126686) (owner: 10Mooeypoo) [23:36:01] (03PS1) 10Mooeypoo: Adjust line-height of bundle group titles [extensions/Echo] - 10https://gerrit.wikimedia.org/r/277693 (https://phabricator.wikimedia.org/T128069) [23:37:06] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Adjust label layout inside cross-wiki notification bundles - https://phabricator.wikimedia.org/T128069#2124973 (10Mooeypoo) >>! In T128069#2106557, @Pginer-WMF wrote: >>>! In T128069#2104218, @Etonkovi... [23:43:29] Heading out. See you all tomorrow. [23:44:37] see ya matt_flaschen [23:44:43] oh , why am I marked away O.o [23:45:32] 6Collaboration-Team-Backlog, 10Collaboration-Team-Sprint-U-2015-04-22, 10Flow, 13Patch-For-Review: Flow data missing on Wikimedia production wikis - https://phabricator.wikimedia.org/T95580#1194991 (10Etonkovidova) As a sanity check, in betalabs [enwiki]> select distinct(rev_mod_state) from flow_revision...