[01:06:57] 06Collaboration-Team-Triage, 10Notifications: Create Echo notiification to encourage users with short passwords to improve them - https://phabricator.wikimedia.org/T150627#2791485 (10Reedy) [02:02:09] 06Collaboration-Team-Triage, 10MediaWiki-extensions-PageCuration, 06Community-Tech: Update PageTriage maintenance tag list to match Twinkle's - https://phabricator.wikimedia.org/T147224#2791501 (10kaldari) @Kudpung: The maintenance tags in PageTriage should match Twinkle (which is the subject of this bug). T... [02:05:32] 06Collaboration-Team-Triage, 10MediaWiki-extensions-PageCuration: Update PageTriage speedy deletion tag list to match Twinkle's - https://phabricator.wikimedia.org/T150628#2791502 (10kaldari) [02:05:50] 06Collaboration-Team-Triage, 10MediaWiki-extensions-PageCuration, 06Community-Tech: Update PageTriage maintenance tag list to match Twinkle's - https://phabricator.wikimedia.org/T147224#2791515 (10kaldari) New task created at T150628. [02:06:20] 06Collaboration-Team-Triage, 10MediaWiki-extensions-PageCuration: Update PageTriage speedy deletion tag list to match Twinkle's - https://phabricator.wikimedia.org/T150628#2791502 (10kaldari) [03:19:50] 06Collaboration-Team-Triage, 10Notifications, 10Mention-Notification, 06TCB-Team: Notifications should be able to ignore - https://phabricator.wikimedia.org/T150630#2791549 (10Gryllida) [03:19:54] 06Collaboration-Team-Triage, 10Notifications, 10Mention-Notification, 06TCB-Team: Does not have mass actions - https://phabricator.wikimedia.org/T150631#2791562 (10Gryllida) [03:23:42] 06Collaboration-Team-Triage, 10Notifications, 10Mention-Notification, 06TCB-Team: Notifications does not have an ignore button - https://phabricator.wikimedia.org/T150630#2791575 (10Gryllida) [03:23:58] 06Collaboration-Team-Triage, 10Notifications, 10Mention-Notification, 06TCB-Team: Notification does not have mass actions for marking things as read - https://phabricator.wikimedia.org/T150631#2791576 (10Gryllida) [03:28:50] 06Collaboration-Team-Triage, 10Notifications, 10Mention-Notification, 06TCB-Team: Notification does not have mass actions for marking things as read - https://phabricator.wikimedia.org/T150631#2791593 (10Josve05a) * Mark all thanks as read as well perhaps? [03:53:29] 06Collaboration-Team-Triage, 10Notifications, 10Mention-Notification, 06TCB-Team: Notifications does not have an ignore button - https://phabricator.wikimedia.org/T150630#2791625 (10SamanthaNguyen) [03:53:31] 06Collaboration-Team-Triage, 10Notifications, 06Support-and-Safety, 13Patch-For-Review: Allow users to restrict who can send them notifications - https://phabricator.wikimedia.org/T150419#2791628 (10SamanthaNguyen) [06:13:14] 06Collaboration-Team-Triage, 10MediaWiki-extensions-PageCuration: Update PageTriage speedy deletion tag list to match Twinkle's - https://phabricator.wikimedia.org/T150628#2791502 (10TTO) Some of these are not relevant for new page patrollers: G6 move, G6 XfD, G6 history merge, G8 (both) and G13. As someone wh... [07:16:05] 06Collaboration-Team-Triage, 10Notifications: Create Echo notification to encourage users with short passwords to improve them - https://phabricator.wikimedia.org/T150627#2791723 (10Peachey88) [12:03:18] (03CR) 10Sbisson: [C: 032] Fix warnings and bugs in generateForwardedCookieForCli() [extensions/Flow] - 10https://gerrit.wikimedia.org/r/320639 (https://phabricator.wikimedia.org/T149579) (owner: 10Catrope) [12:04:32] (03CR) 10Sbisson: "Should those icons live in the Thank extension?" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/320947 (https://phabricator.wikimedia.org/T149352) (owner: 10Catrope) [12:26:18] (03Merged) 10jenkins-bot: Fix warnings and bugs in generateForwardedCookieForCli() [extensions/Flow] - 10https://gerrit.wikimedia.org/r/320639 (https://phabricator.wikimedia.org/T149579) (owner: 10Catrope) [15:17:13] 06Collaboration-Team-Triage, 10Notifications, 07Tracking: Create new types of notifications - https://phabricator.wikimedia.org/T125653#2792544 (10matej_suchanek) [15:17:15] 06Collaboration-Team-Triage, 10Notifications: Create Echo notification to encourage users with short passwords to improve them - https://phabricator.wikimedia.org/T150627#2792543 (10matej_suchanek) [15:35:35] 06Collaboration-Team-Triage (Collab-Team-Q2-Oct-Dec-2016), 13Patch-For-Review, 15User-zeljkofilipin: Create rspec tests for Echo - https://phabricator.wikimedia.org/T146916#2792559 (10zeljkofilipin) [15:50:00] 06Collaboration-Team-Triage (Collab-Team-Q2-Oct-Dec-2016), 10Notifications, 10The-Wikipedia-Library, 07User-notice, 07User-notice-collaboration: Notify editors that they are now eligible for the Wikipedia Library program - https://phabricator.wikimedia.org/T132084#2792571 (10Trizek-WMF) >>! In T132084#27... [16:43:00] 06Collaboration-Team-Triage, 10Notifications: Wrong link in an automatic mail about user rights on MediaWiki - https://phabricator.wikimedia.org/T150663#2792728 (10Lea_Lacroix_WMDE) [16:44:04] 06Collaboration-Team-Triage, 10Notifications: Wrong link in an automatic mail about user rights on MediaWiki - https://phabricator.wikimedia.org/T150663#2792742 (10Trizek-WMF) [16:44:25] 06Collaboration-Team-Triage, 10Notifications: Wrong link in an automatic mail about user rights on MediaWiki - https://phabricator.wikimedia.org/T150663#2792728 (10Trizek-WMF) [16:52:03] 06Collaboration-Team-Triage (Collab-Team-Q2-Oct-Dec-2016), 10Flow, 07User-notice-collaboration: Fix user talk pages already in inconsistent state due to to T138310 - https://phabricator.wikimedia.org/T148057#2792780 (10Trizek-WMF) What's new about that task? [18:15:52] legoktm, it's referenced in the Makefile. Apparently to check for duplicated CSS rules. [19:08:02] (03CR) 10jenkins-bot: [V: 04-1] Update for API error i18n [extensions/Thanks] - 10https://gerrit.wikimedia.org/r/321455 (owner: 10Anomie) [19:11:55] (03PS2) 10Anomie: Update for API error i18n [extensions/Thanks] - 10https://gerrit.wikimedia.org/r/321455 [19:30:41] (03CR) 10jenkins-bot: [V: 04-1] Update for API error i18n [extensions/Flow] - 10https://gerrit.wikimedia.org/r/321426 (owner: 10Anomie) [19:30:56] (03PS3) 10Anomie: Update for API error i18n [extensions/Thanks] - 10https://gerrit.wikimedia.org/r/321455 [19:38:18] (03CR) 10jenkins-bot: [V: 04-1] Update for API error i18n [extensions/Thanks] - 10https://gerrit.wikimedia.org/r/321455 (owner: 10Anomie) [20:17:08] matt_flaschen: two questions 1) fluorine logs are not reachable or I am missing smth again? 2) I see many "AbuseFilter::filterAction: cache miss for 'Page..." on logstash.wikimedia.org - not our concern? [21:29:18] Re Elena's question, see https://phabricator.wikimedia.org/T146723 . It is now at deployment-fluorine02.eqiad.wmflabs , but also, since the move, the service keeps stopping causing the logs to be lost. [21:29:47] etonkovidova, re the second question, it's just an INFO-level log. Apparently it's being tracked as an optimization. [21:30:10] ^ etonkovidova, didn't see when you re-joined. Did you see both answers? [21:31:16] matt_flaschen:yup - thx! [22:31:31] stephanebisson, matt_flaschen, question... I am writing a "find" method for the filter input to find items that have labels matching whatever you type. So, if I start typing "new" it will return anything that has "newbie" or "new user" or "renew" etc in the *label*. [22:31:52] So... I am thinking the most straightforward way is go something like item.getLabel().indexOf( searchString ) > -1 [22:32:22] That's better than going with regex search and all that, right? regexp in this case seems to be a complete overkill to me, but I'm hoping I'm not missing anything [22:37:26] mooeypoo, that should work fine. You might get a performance benefit from pre-compiling one regex or-ing everything together /(Newbie|New user|Renewable edits)/i (you can create a regex dynamically rather than hard-coding it). [22:37:37] mooeypoo, but pre-mature optimization is the root of all evil, so don't know if that's necessary. [22:37:52] matt_flaschen, I can't, though, bcause it's language dependent :\ [22:37:57] The labels are mw.msg() [22:38:12] and the typing needs to search for the result of those.. :\ [22:38:37] I think considering we have relatively limited filters, it might work well for starters. If we start doing things like looking up on API (like for namespaces, etc) then we might have to rethink this? [22:38:37] mooeypoo, you can, you just have to compile it dynamically with the RegExp constructor: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp [22:39:16] * matt_flaschen googles [22:41:52] ooh [22:43:46] mooeypoo: not at all urgent - can you please take a quick look at https://phabricator.wikimedia.org/T150572 ? [22:44:08] mooeypoo: oopsie - https://phabricator.wikimedia.org/T150615 [22:44:14] I was going to ask... :D [22:44:25] mooeypoo, quick Googling couldn't find it. You could benchmark with https://jsperf.com/ if you want. [22:44:53] matt_flaschen, I think I'll start with the indexOf() and we'll see if we need to optimize later... but good point to consider. [22:47:28] Actually, you wouldn't need the parentheses.