[00:17:15] legoktm: http://52.23.170.224/index.php/Special:Version [00:19:52] 1.26.2 [00:36:19] cariaso: you'll need to use an older version of the OAuth extension then [00:38:09] if I should be getting a certain branch of OAuth, I wasn't seeing it at https://phabricator.wikimedia.org/diffusion/EOAU/ but someone in #mediawiki just pointed me to https://phabricator.wikimedia.org/diffusion/EOAU/browse/REL1_26/ so I'll pick it up from there [04:05:24] 6Collaboration-Team-Backlog, 10Thanks, 10pywikibot-core: Pywikibot support for Thanks - https://phabricator.wikimedia.org/T129049#2093372 (10jayvdb) [04:07:00] 6Collaboration-Team-Backlog, 10Thanks, 10Possible-Tech-Projects, 10pywikibot-core: Pywikibot support for Thanks - https://phabricator.wikimedia.org/T129049#2093372 (10jayvdb) [04:07:21] 6Collaboration-Team-Backlog, 10Thanks, 10Possible-Tech-Projects, 10pywikibot-core: Pywikibot support for Thanks - https://phabricator.wikimedia.org/T129049#2093372 (10jayvdb) p:5Triage>3Low [04:08:20] 6Collaboration-Team-Backlog, 10ArchCom-RfC, 10Librarization, 10Notifications: RfC: Notifications in core - https://phabricator.wikimedia.org/T128351#2093392 (10RobLa-WMF) We plan to discuss this in {E147} [05:18:01] 6Collaboration-Team-Backlog, 10Thanks, 10Possible-Tech-Projects, 10pywikibot-core: Pywikibot support for Thanks - https://phabricator.wikimedia.org/T129049#2093457 (10jayvdb) [05:46:36] 6Collaboration-Team-Backlog, 10Thanks, 10Possible-Tech-Projects, 10pywikibot-core: Pywikibot support for Thanks - https://phabricator.wikimedia.org/T129049#2093498 (10jayvdb) [07:10:19] 3Collaboration-Team-Current, 10Collaboration-Team-Sprint-D-2015-06-30, 10Collaboration-Team-Sprint-E-Everywhere-2015-07-14, 10Flow, and 2 others: E10. Have Flow use ExternalStore on MediaWiki-Vagrant - https://phabricator.wikimedia.org/T95870#2093556 (10Mattflaschen) [07:15:50] 3Collaboration-Team-Current, 10Flow, 10Notifications: Double escaping in Flow notifications - https://phabricator.wikimedia.org/T125130#2093561 (10Mattflaschen) Fixed by {339a78e6059a3107c0ad0d5208ac34980b38112f}. [08:19:10] 3Collaboration-Team-Current, 10MediaWiki-extensions-GuidedTour: ext.guidedTour gives malformed uri sequence at nowiki - https://phabricator.wikimedia.org/T125793#2093628 (10jeblad) The bug has been gone for several days now,.. Sort of miss it... [09:18:33] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15), 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Truncate Page and Topic Names and Msg Excerpts in Notifications Fly-Outs and Page - https://phabricator.wikimedia.org/T121822#2093669 (10Pginer-WMF) If I under... [09:24:01] 6Collaboration-Team-Backlog, 10Notifications: User preferences for cross-wiki notifications - https://phabricator.wikimedia.org/T114917#2093675 (10Pginer-WMF) For the initial steps, the beta feature settings can be used to control where to have cross-wiki notifications visible. Based on how people use this, w... [09:24:21] (03PS5) 10Matthias Mullie: Don't delete unread messages before all read messages have been deleted [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274690 (https://phabricator.wikimedia.org/T124687) [09:24:51] (03CR) 10Matthias Mullie: "I think the problem where it deletes things it's not supposed to should also be fixed now." (034 comments) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274690 (https://phabricator.wikimedia.org/T124687) (owner: 10Matthias Mullie) [10:37:40] (03PS1) 10Matthias Mullie: Use same link for flow & flowusertalk summary notification [extensions/Flow] - 10https://gerrit.wikimedia.org/r/275385 (https://phabricator.wikimedia.org/T125119) [10:50:08] (03CR) 10Siebrand: [C: 031] "i18n/L10n reviewed." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/264070 (https://phabricator.wikimedia.org/T100974) (owner: 10Haritha28) [10:55:50] (03CR) 10jenkins-bot: [V: 04-1] Notification when account is blocked [extensions/Echo] - 10https://gerrit.wikimedia.org/r/264070 (https://phabricator.wikimedia.org/T100974) (owner: 10Haritha28) [11:39:52] (03PS1) 10Siebrand: Remove use of "successful" in strings [extensions/Flow] - 10https://gerrit.wikimedia.org/r/275410 [11:46:02] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15), 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Truncate Page and Topic Names and Msg Excerpts in Notifications Fly-Outs and Page - https://phabricator.wikimedia.org/T121822#2093980 (10SBisson) Exactly, the... [11:57:33] (03PS4) 10Sbisson: build: Enable jscs rule 'requireVarDeclFirst' and make pass [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274010 (owner: 10Ricordisamoa) [12:05:45] (03CR) 10Sbisson: [C: 04-1] Use same link for flow & flowusertalk summary notification (031 comment) [extensions/Flow] - 10https://gerrit.wikimedia.org/r/275385 (https://phabricator.wikimedia.org/T125119) (owner: 10Matthias Mullie) [12:06:37] (03CR) 10Sbisson: [C: 032] build: Enable jscs rule 'requireVarDeclFirst' and make pass [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274010 (owner: 10Ricordisamoa) [12:06:51] (03PS2) 10Matthias Mullie: Use same link for flow & flowusertalk summary notification [extensions/Flow] - 10https://gerrit.wikimedia.org/r/275385 (https://phabricator.wikimedia.org/T125119) [12:08:01] (03CR) 10Matthias Mullie: Use same link for flow & flowusertalk summary notification (031 comment) [extensions/Flow] - 10https://gerrit.wikimedia.org/r/275385 (https://phabricator.wikimedia.org/T125119) (owner: 10Matthias Mullie) [12:09:51] (03CR) 10Alex Monk: "You don't seem to understand - the old Hooks::run call resulted in Flow's function being run, this does not. Please go and read that exten" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275358 (https://phabricator.wikimedia.org/T128769) (owner: 10Tahteche) [12:10:41] (03CR) 10Sbisson: [C: 04-1] "Like Alex said, other extensions rely on the hook being called, so we still have to call it." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275358 (https://phabricator.wikimedia.org/T128769) (owner: 10Tahteche) [12:13:50] (03CR) 10Sbisson: [C: 032] Use same link for flow & flowusertalk summary notification [extensions/Flow] - 10https://gerrit.wikimedia.org/r/275385 (https://phabricator.wikimedia.org/T125119) (owner: 10Matthias Mullie) [12:17:10] (03CR) 10jenkins-bot: [V: 04-1] build: Enable jscs rule 'requireVarDeclFirst' and make pass [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274010 (owner: 10Ricordisamoa) [12:24:15] (03Merged) 10jenkins-bot: Use same link for flow & flowusertalk summary notification [extensions/Flow] - 10https://gerrit.wikimedia.org/r/275385 (https://phabricator.wikimedia.org/T125119) (owner: 10Matthias Mullie) [12:53:25] 3Collaboration-Team-Current, 10Notifications: Adjust cross-wiki bundle when it is the only unread item - https://phabricator.wikimedia.org/T128564#2094158 (10Pginer-WMF) Since users already have other read messages in the local wiki, the current message is not completely disorienting. I think we can keep using... [12:59:52] (03CR) 10Sbisson: "recheck" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274010 (owner: 10Ricordisamoa) [13:03:44] (03CR) 10Paladox: "check experimental" [extensions/Thanks] - 10https://gerrit.wikimedia.org/r/275342 (owner: 10L10n-bot) [13:06:18] (03CR) 10Sbisson: build: Enable jscs rule 'requireVarDeclFirst' and make pass [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274010 (owner: 10Ricordisamoa) [13:06:24] (03CR) 10Sbisson: [C: 032] build: Enable jscs rule 'requireVarDeclFirst' and make pass [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274010 (owner: 10Ricordisamoa) [13:16:51] (03Merged) 10jenkins-bot: build: Enable jscs rule 'requireVarDeclFirst' and make pass [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274010 (owner: 10Ricordisamoa) [13:29:28] (03PS1) 10Ricordisamoa: build: Enable jscs jsDoc rule 'requireNewlineAfterDescription' and make pass [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275450 [14:01:52] (03CR) 10Sbisson: "It looks like the 'prioritized' flag is not as useful as we thought it would be. The container gets to decide how many links to show where" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275142 (https://phabricator.wikimedia.org/T126617) (owner: 10Mooeypoo) [14:33:07] 3Collaboration-Team-Current, 10Flow, 13Patch-For-Review: Flow board move requiring allowCreation fails on zh.wp - https://phabricator.wikimedia.org/T127693#2094292 (10Trizek-WMF) I've ask permission to Nbfreeh to move his page. [14:45:27] 6Collaboration-Team-Backlog, 10Collaboration-Community-Engagement, 3Liaisons-March-2016: Prepare and review announcements concerning Cross-wiki Notifications as a Beta feature on all wikis - https://phabricator.wikimedia.org/T128865#2094304 (10Trizek-WMF) [15:04:03] 6Collaboration-Team-Backlog, 10Notifications: Give a way for users to opt-out Cross-wiki Notifications - https://phabricator.wikimedia.org/T129077#2094340 (10Trizek-WMF) [15:15:26] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 7user-notice: Roll out cross-wiki notifications beta feature - https://phabricator.wikimedia.org/T124234#2094484 (10Trizek-WMF) [15:31:06] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15), 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Truncate Page and Topic Names and Msg Excerpts in Notifications Fly-Outs and Page - https://phabricator.wikimedia.org/T121822#2094540 (10Trizek-WMF) Have we co... [15:43:32] (03CR) 10Sbisson: [C: 032] Remove deprecated API functions [extensions/Thanks] - 10https://gerrit.wikimedia.org/r/275040 (owner: 10Legoktm) [15:44:16] (03CR) 10Tahteche: "@Alex Thanks. I understand now." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275358 (https://phabricator.wikimedia.org/T128769) (owner: 10Tahteche) [15:52:13] (03Merged) 10jenkins-bot: Remove deprecated API functions [extensions/Thanks] - 10https://gerrit.wikimedia.org/r/275040 (owner: 10Legoktm) [15:53:40] 6Collaboration-Team-Backlog, 10Notifications, 7Easy, 13Patch-For-Review, 7Technical-Debt: Echo should not hook into itself with EchoAbortEmailNotification - https://phabricator.wikimedia.org/T128769#2085271 (10tahteche) [[ https://gerrit.wikimedia.org/r/275358 | Provided a patch ]] to this but I was clar... [16:11:10] (03PS1) 10Matthias Mullie: Turn dumpBackup into proper Maintenance script [extensions/Flow] - 10https://gerrit.wikimedia.org/r/275519 [16:14:50] 3Collaboration-Team-Current, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2094617 (10Pginer-WMF) Some thoughts (and a quick mockup): - The general approach proposed makes sense. - We may want to use a specific icon to avoid a... [17:02:28] 6Collaboration-Team-Backlog, 10Notifications: email notification issue after email change - https://phabricator.wikimedia.org/T129094#2094893 (10Nnvu) [17:05:12] 6Collaboration-Team-Backlog, 10Notifications: email notification issue after email change - https://phabricator.wikimedia.org/T129094#2094933 (10Elitre) [17:07:24] 6Collaboration-Team-Backlog, 10Notifications: email notification issue after email change - https://phabricator.wikimedia.org/T129094#2094893 (10Elitre) For context, the user has switched back to his former address, and is still unable to receive said notifications. [17:28:53] mlitn: hey, are you around? [17:50:06] 6Collaboration-Team-Backlog, 10Collaboration-Community-Engagement, 10Notifications: Have a Community Notification Board - https://phabricator.wikimedia.org/T129103#2095172 (10Trizek-WMF) [17:52:37] 3Collaboration-Team-Current, 10Flow, 10MediaWiki-General-or-Unknown, 7JavaScript, 13Patch-For-Review: Support single API for pre-filling new topics for both old-style talk pages and Flow - https://phabricator.wikimedia.org/T107637#1500033 (10Quiddity) I believe the first option was implemented, with the... [17:58:54] stephanebisson: hey, I am now [17:59:25] mlitn: any idea what `echo_event.event_variant` is supposed to contain? [18:01:29] I have no idea [18:01:35] I remember looking that up at some point too [18:01:48] but doesn’t really seem to be used at all, iirc [18:04:26] 6Collaboration-Team-Backlog, 10Notifications: email notification issue after email change - https://phabricator.wikimedia.org/T129094#2095268 (10Aklapper) Does the "Email this user" functionality work? When it comes to projects, I assume this is unrelated to Echo/#Notifications but #Wikimedia-General-or-Unknow... [18:11:07] stephanebisson: [18:11:12] mysql:wikiadmin@10.64.16.20 [enwiki]> SELECT * FROM echo_event WHERE event_variant IS NOT NULL; [18:11:12] Empty set (2 min 56.07 sec) [18:11:20] not used at all [18:11:27] mlitn: thanks [18:38:32] 6Collaboration-Team-Backlog, 10Collaboration-Community-Engagement, 10Notifications: Have a Community Notification Board - https://phabricator.wikimedia.org/T129103#2095172 (10Legoktm) Duplicate of {T58362}? [18:55:48] 6Collaboration-Team-Backlog, 10Collaboration-Community-Engagement, 10Notifications: Have a Community Notification Board - https://phabricator.wikimedia.org/T129103#2095552 (10Trizek-WMF) [18:55:50] 6Collaboration-Team-Backlog, 10Flow, 10Notifications: Allow users to create custom notifications onwiki - https://phabricator.wikimedia.org/T58362#2095554 (10Trizek-WMF) [18:56:50] 6Collaboration-Team-Backlog, 10Flow, 10Notifications: Allow users to create custom notifications onwiki - https://phabricator.wikimedia.org/T58362#608735 (10Trizek-WMF) Duplicate's description: (Based on an interesting internal conversation about how to warn people about a new feature or a survey.) The id... [19:43:32] mooeypoo: I'll join for the first day. Masada looks grandiose [19:43:46] there better be hummus [19:48:49] (03PS3) 10Legoktm: Defer onPersonalUrls() DB writes to post-send [extensions/Echo] - 10https://gerrit.wikimedia.org/r/250761 (https://phabricator.wikimedia.org/T117531) (owner: 10Aaron Schulz) [19:48:52] (03PS1) 10Legoktm: Fix EchoAttributeManager::getNotificationSection() php doc [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275603 [19:50:29] (03CR) 10Legoktm: "I haven't tested this yet, but it should work..." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/250761 (https://phabricator.wikimedia.org/T117531) (owner: 10Aaron Schulz) [20:01:14] 6Collaboration-Team-Backlog, 10Flow, 7Mobile: Flow 'Add topic' button is unexpectedly grayed out on mobile (Android Chrome) - https://phabricator.wikimedia.org/T129126#2095966 (10brion) [20:29:08] (03CR) 10Catrope: [C: 032] Fix EchoAttributeManager::getNotificationSection() php doc [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275603 (owner: 10Legoktm) [20:29:56] (03PS4) 10Catrope: Defer onPersonalUrls() DB writes to post-send [extensions/Echo] - 10https://gerrit.wikimedia.org/r/250761 (https://phabricator.wikimedia.org/T117531) (owner: 10Aaron Schulz) [20:45:35] 3Collaboration-Team-Current, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2096260 (10Capt_Swing) @Pginer-WMF I agree we want to avoid showing this across wikis (although if we only have the survey link up for a day or two, wh... [20:46:46] (03CR) 10Catrope: [C: 031] Defer onPersonalUrls() DB writes to post-send (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/250761 (https://phabricator.wikimedia.org/T117531) (owner: 10Aaron Schulz) [20:46:56] (03PS2) 10Catrope: Fix EchoAttributeManager::getNotificationSection() php doc [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275603 (owner: 10Legoktm) [20:53:12] (03CR) 10jenkins-bot: [V: 04-1] Fix EchoAttributeManager::getNotificationSection() php doc [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275603 (owner: 10Legoktm) [20:56:02] 3Collaboration-Team-Current, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2096303 (10egalvezwmf) Hi @Capt_Swing, has legal already reviewed your survey? They typically will offer guidance when they review your survey (which i... [20:58:17] matt_flaschen: any idea what is the purpose of `echo_event.event_variant`? [20:58:48] stephanebisson, no. [21:00:44] If posts were subpages of topics (Topic:/), it would be much easier to locate events... [21:04:19] stephanebisson, do you want to discuss that, or did you figure out an approach? [21:04:56] matt_flaschen: I'm happy to hear your ideas [21:05:54] stephanebisson, are you trying to locate all events about a post ID? [21:06:56] matt_flaschen: Yes. When a post is moderated, I'm looking for the 'post-edited', 'post-reply' connected with that post [21:07:29] that information is hidden in the 'extra' json [21:08:17] I can narrow it down using event_type and echo_target_page with the topic page but I don't know if it's good enough [21:08:35] that can still leaves a large number of rows to search [21:10:45] stephanebisson, yeah, I don't know a non-hacky way to query them in the DB. [21:10:59] or does it? how many events are really generated for one topic [21:11:49] matt_flaschen: how big can a topic be, in term of number of post? [21:11:57] Probably not a insane amount. A lot of flow-post-reply, a few flow-post-edited, some number of Flow mentions. Topics renames and summary changes should be relatively rare. [21:12:32] We could get metrics, but I don't know an alternative approach other than a schema change, or LIKE-ing into event_extra (super-hacky, not recommended). [21:13:21] stephanebisson, one small thing, I don't think you need to use echo_target_page. You can use event_page_id, which at least saves a join. [21:13:52] matt_flaschen: unfortunately, event_page_id generally refers to the board [21:14:35] Wow, that kinda sucks that it's the board even on topic-specific actions. [21:15:29] matt_flaschen: well, the board and topic pages both good candidates about 'where the event took place', that's supposed to be the meaning of event_page_id [21:15:57] In my opinion, topic would clearly be better for flow-post-reply, but it is what it is. [21:16:49] So here's a hack I suggested last week [21:16:57] makes sense, we could say that 'new-topic' or 'desc-edited' happens on the board while post-* happens on the topic page [21:17:04] Write two echo_target_page rows, one for the topic and one for the post [21:17:25] The one for the post would be ns=2600, title=uuid and as such wouldn't be a valid topic page, but I don't believe there's any code that cares [21:18:24] RoanKattouw, wouldn't that look really similar to a valid topic page, considering UUIDs all the look the same? [21:18:39] RoanKattouw: echo_target_page only has pageId (int), it doesn't have NS + title [21:19:26] stephanebisson, this happens at moderation-time, not at view-time, right? [21:19:42] matt_flaschen: right, ideally [21:20:16] stephanebisson, so there's a little more leeway. We still don't want it to be slow, but it's not happening all the time. [21:20:48] RoanKattouw, stephanebisson I don't really like the idea of making our DB values/backend less intuitive. [21:21:05] And it doesn't look like that will work anyway per stephanebisson [21:21:22] RoanKattouw: I don't think a distinct target_page row for the post would be hacky at all, if only we could address a post with a pageId [21:21:57] hence the idea of a post being a subpage of a topic [21:22:06] but that's probably too big of a change [21:22:35] Yeah, that would be perfectly reasonable if posts were pages. Someday I'll find out why they're not (I've only heard vague things about not wanting to parse a bunch of pages at view-time, like LQT, but not super-convincing). [21:24:51] Oh, it's by page ID [21:24:53] Yeah, never mind then [21:25:11] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15), 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Truncate Page and Topic Names and Msg Excerpts in Notifications Fly-Outs and Page - https://phabricator.wikimedia.org/T121822#2096647 (10jmatazzoni) > Exactly,... [21:25:13] in the meantime, I'll try to filter with event-type and topic-page-id and then work in-memory to find the right post [21:25:29] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15), 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Truncate Page and Topic Names and Msg Excerpts in Notifications Fly-Outs and Page - https://phabricator.wikimedia.org/T121822#2096661 (10jmatazzoni) 5Open>3... [21:26:14] RoanKattouw, There are unused (apparently) event_page_namespace and event_page_title, but that would still be very hacky to use them to point to posts. [21:26:31] Yes, those are there for b/c AFAICT [21:26:51] The table was migrated from using ns/title pairs to page IDs, and there's b/c code to parse old rows that use ns/title [21:27:43] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15), 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Truncate Page and Topic Names and Msg Excerpts in Notifications Fly-Outs and Page - https://phabricator.wikimedia.org/T121822#2096689 (10SBisson) >>! In T12182... [21:29:27] I'll try to find out stats about posts per topic (avg, min, max, mode, ...) Do we have that in a dashboard by any chance? [21:30:01] 3Collaboration-Team-Current, 10Flow: Problem while converting from LiquidThreads to Flow - https://phabricator.wikimedia.org/T128949#2096724 (10Mattflaschen) [21:36:57] (03CR) 10Mattflaschen: "> It looks like the 'prioritized' flag is not as useful as we thought it would be. The container gets to decide how many links to show whe" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275142 (https://phabricator.wikimedia.org/T126617) (owner: 10Mooeypoo) [21:44:15] 3Collaboration-Team-Current, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2096840 (10jmatazzoni) @Pginer-WMF, the mockup shown above is what people looked at in the Community meeting htis morning. In light of today's discussi... [21:47:29] (03CR) 10Sbisson: "> It is still being used." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275142 (https://phabricator.wikimedia.org/T126617) (owner: 10Mooeypoo) [21:52:56] (03CR) 10Mattflaschen: [C: 04-1] "Sorry, some of this is pre-existing (but now is a good time to deal with)." (035 comments) [extensions/Flow] - 10https://gerrit.wikimedia.org/r/275519 (owner: 10Matthias Mullie) [21:55:13] 3Collaboration-Team-Current, 10Flow, 10MediaWiki-General-or-Unknown, 7JavaScript, 13Patch-For-Review: Support single API for pre-filling new topics for both old-style talk pages and Flow - https://phabricator.wikimedia.org/T107637#2096919 (10Mattflaschen) >>! In T107637#2095201, @Quiddity wrote: > I beli... [21:57:53] 3Collaboration-Team-Current, 10MediaWiki-Vagrant: echo role may not work unless centralauth role is explicitly enabled - https://phabricator.wikimedia.org/T124707#2096926 (10Mattflaschen) Moving this out of current, unless/until it is reported again. [21:58:01] 6Collaboration-Team-Backlog, 10MediaWiki-Vagrant: echo role may not work unless centralauth role is explicitly enabled - https://phabricator.wikimedia.org/T124707#2096927 (10Mattflaschen) [21:58:11] 6Collaboration-Team-Backlog, 10MediaWiki-Vagrant: echo role may not work unless centralauth role is explicitly enabled - https://phabricator.wikimedia.org/T124707#1963394 (10Mattflaschen) a:5Mattflaschen>3None [21:58:52] 3Collaboration-Team-Current, 10Notifications, 7I18n, 13Patch-For-Review: Notification-header-flow-topic-renamed-user-talk and Notification-bundle-header-flow-post-edited-user-talk seem to have incorrect parameter documentation - https://phabricator.wikimedia.org/T125253#2096929 (10jmatazzoni) 5Open>3Res... [21:58:56] 6Collaboration-Team-Backlog, 10Notifications: Styling adjustments for the Notification Panel - https://phabricator.wikimedia.org/T119374#2096946 (10Catrope) [21:58:58] 3Collaboration-Team-Current, 10Notifications: Adjust cross-wiki bundle when it is the only unread item - https://phabricator.wikimedia.org/T128564#2096944 (10Catrope) 5Open>3declined >>! In T128564#2094158, @Pginer-WMF wrote: > Since users already have other read messages in the local wiki, the current mes... [21:59:44] 3Collaboration-Team-Current, 10Flow, 10MediaWiki-Vagrant: Use custom $wgFlowCluster and $wgFlowDefaultWikiDb in MediaWiki-Vagrant - https://phabricator.wikimedia.org/T121827#1889516 (10Mattflaschen) This did cause {T115695} (well, the different testing environment meant it was not caught), but it's also goin... [22:00:23] (03PS2) 10Siebrand: Remove double emphasis in strings [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274657 (https://phabricator.wikimedia.org/T128711) [22:00:48] 3Collaboration-Team-Current, 10Flow: Give the user the number of history items they asked for - https://phabricator.wikimedia.org/T108291#2096952 (10Mattflaschen) a:5Mattflaschen>3None [22:01:18] (03CR) 10Mooeypoo: "That's why I thought that the **widget** should have that imposed restriction (more than 2 go into the dotdotdot menu) and not the back en" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275142 (https://phabricator.wikimedia.org/T126617) (owner: 10Mooeypoo) [22:04:58] (03CR) 10Mooeypoo: "Also, just to answer the earlier comment about the 'prioritized' flag not being as useful - I actually semi agree (and semi disagree ;)" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275142 (https://phabricator.wikimedia.org/T126617) (owner: 10Mooeypoo) [22:06:13] 3Collaboration-Team-Current, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2096995 (10Capt_Swing) Update: I've reached out to @APalmer_WMF, and provided her with a copy of the survey questions for review. [22:06:43] 3Collaboration-Team-Current, 6Design-Research-Backlog, 10Notifications: Invite Users to Take Notifications Survey (Using Notifications Panel) - https://phabricator.wikimedia.org/T128937#2096997 (10Capt_Swing) [22:08:01] (03CR) 10jenkins-bot: [V: 04-1] Remove double emphasis in strings [extensions/Echo] - 10https://gerrit.wikimedia.org/r/274657 (https://phabricator.wikimedia.org/T128711) (owner: 10Siebrand) [22:09:04] 3Collaboration-Team-Current, 10Flow, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Generate flow-mention for create/edit of board desc, topic, post, summary - https://phabricator.wikimedia.org/T125427#2097006 (10jmatazzoni) 5Open>3Resolved [22:10:22] (03CR) 10Mooeypoo: Add MobileNotificationsWrapper for Mobile display (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/251561 (https://phabricator.wikimedia.org/T124188) (owner: 10Mooeypoo) [22:10:34] (03PS10) 10Mooeypoo: Add MobileNotificationsWrapper for Mobile display [extensions/Echo] - 10https://gerrit.wikimedia.org/r/251561 (https://phabricator.wikimedia.org/T124188) [22:11:13] 3Collaboration-Team-Current, 10Notifications, 7I18n, 13Patch-For-Review: notification-header-user-rights-* should have GENDER support for the current user - https://phabricator.wikimedia.org/T125251#2097013 (10jmatazzoni) 5Open>3Resolved [22:11:53] 3Collaboration-Team-Current, 10Beta-Cluster-Infrastructure, 10DBA, 13Patch-For-Review, 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Use External Store on Beta Cluster - https://phabricator.wikimedia.org/T95871#2097031 (10Mattflaschen) >>! In T95871#2087771, @hashar wrote: > The `wikiadmin_pass` was provisione... [22:11:55] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): dotdotdot menu layout breaks on scroll / window resize - https://phabricator.wikimedia.org/T128436#2097032 (10jmatazzoni) 5Open>3Resolved [22:18:45] 3Collaboration-Team-Current, 10Notifications: Bundle messages with imprecise amount of "other" not properly localizable - https://phabricator.wikimedia.org/T127288#2097067 (10Mattflaschen) a:3Mattflaschen [22:23:36] 3Collaboration-Team-Current, 10Notifications: Make pagination on Special:Notifications clearer - https://phabricator.wikimedia.org/T129169#2097078 (10Catrope) [22:35:34] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Action and timestamp position needs adjustment - https://phabricator.wikimedia.org/T125969#2097137 (10Etonkovidova) After adjustments in betalabs. @Pginer-WMF - the screenshots show positioning of timestamp & labels for secondary links for u... [22:36:07] 3Collaboration-Team-Current, 10Notifications: Clean up HTML output and CSS for Special:Notifications - https://phabricator.wikimedia.org/T129172#2097139 (10Catrope) [22:37:21] 3Collaboration-Team-Current, 10Notifications: Make pagination on Special:Notifications clearer - https://phabricator.wikimedia.org/T129169#2097156 (10Catrope) [22:37:23] 3Collaboration-Team-Current, 10Notifications: Clean up HTML output and CSS for Special:Notifications - https://phabricator.wikimedia.org/T129172#2097139 (10Catrope) [22:37:25] 6Collaboration-Team-Backlog, 10Notifications, 7Epic: Better organisation of the Notification Page - https://phabricator.wikimedia.org/T115316#2097154 (10Catrope) [22:42:29] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review: Action and timestamp position needs adjustment - https://phabricator.wikimedia.org/T125969#2097166 (10Etonkovidova) @Pginer-WMF - bottom margin adjustment(s) is ok? {F3553340} [22:53:56] 3Collaboration-Team-Current, 10Notifications: Minimal JS version of Special:Notifications - https://phabricator.wikimedia.org/T129176#2097219 (10Catrope) [22:58:26] 6Collaboration-Team-Backlog, 10Notifications, 7Easy, 13Patch-For-Review: Echo congratulations thresholds shouldn't be hard-coded - https://phabricator.wikimedia.org/T127311#2097271 (10Jdforrester-WMF) 5Open>3declined Per discussion. [23:07:04] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Adjust contrast and hover behaviour for the secondary actions displayed directly on notifications - https://phabricator.wikimedia.org/T128444#2074953 (10Etonkovidova) For Pginer-WMF to take a look: In... [23:25:18] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Allow marking whole cross-wiki bundles as read - https://phabricator.wikimedia.org/T128562#2097383 (10Etonkovidova) Checked in betalabs according to the following specs: > - Available whether the bun... [23:28:18] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): [cross-wiki] 'Mark all as read' should not apply to bundled cross-wiki notifications - https://phabricator.wikimedia.org/T128621#2097392 (10Etonkovidova) Checked the fix in betalabs along with {T128562}. [23:52:26] (03PS5) 10Legoktm: Defer onPersonalUrls() DB writes to post-send [extensions/Echo] - 10https://gerrit.wikimedia.org/r/250761 (https://phabricator.wikimedia.org/T117531) (owner: 10Aaron Schulz) [23:52:48] (03CR) 10Legoktm: Defer onPersonalUrls() DB writes to post-send (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/250761 (https://phabricator.wikimedia.org/T117531) (owner: 10Aaron Schulz) [23:55:27] 6Collaboration-Team-Backlog, 10Notifications: Styling adjustments for the Notification Panel - https://phabricator.wikimedia.org/T119374#2097526 (10jmatazzoni) [23:55:29] 3Collaboration-Team-Current, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-03-08_(1.27.0-wmf.16): Allow marking whole cross-wiki bundles as read - https://phabricator.wikimedia.org/T128562#2097525 (10jmatazzoni) 5Open>3Resolved [23:56:15] (03PS1) 10Legoktm: EchoTargetPage: Don't call Title::newFromId() repeatedly [extensions/Echo] - 10https://gerrit.wikimedia.org/r/275721