[00:01:05] 03Collab-Team-Q1-July-Sep-2016, 10MediaWiki-extensions-CentralAuth, 10Wikimedia-General-or-Unknown, 05MW-1.28-release-notes, and 3 others: Increase "remember me" login cookie expiry from 30 days to 1 year on Wikimedia wikis - https://phabricator.wikimedia.org/T68699#2538800 (10Mattflaschen-WMF) Done prep w... [00:28:35] RoanKattouw, turns out I mis-skimmed it before, and I do have feedback from Aaron on "Update "Better handle already-used rev_id when restoring" (https://gerrit.wikimedia.org/r/#/c/290293/)". [00:28:40] So I've been working on that. [00:29:07] Oh, right, I see [00:29:11] That comment is on the same patchset [01:00:08] 03Collab-Team-Q1-July-Sep-2016, 10Flow, 05MW-1.28-release-notes, 13Patch-For-Review: Improve Flow deletion/undeletion resilience - https://phabricator.wikimedia.org/T122262#2538882 (10jmatazzoni) 05Open>03Resolved [03:11:00] (03Abandoned) 10Legoktm: Use new UserResetAllOptions hook for new users [extensions/Echo] - 10https://gerrit.wikimedia.org/r/145473 (https://bugzilla.wikimedia.org/47895) (owner: 10Legoktm) [03:44:12] 06Collaboration-Team-Triage, 10WikiLove, 10MediaWiki-extensions-SocialProfile, 10Social-Tools, 07Tracking: Improve social tools' usability on mobile devices (tracking) - https://phabricator.wikimedia.org/T106268#2538968 (10SamanthaNguyen) In the process of converting this to a parent task and separating... [04:09:59] 06Collaboration-Team-Triage, 10Flow: Investigate echo_notification identical timestamps (notification_timestamp and notification_read_timestamp) - https://phabricator.wikimedia.org/T118220#2538978 (10Legoktm) a:05Legoktm>03None [04:13:12] 06Collaboration-Team-Triage, 10Flow, 07WorkType-NewFunctionality: Flow should make sure all non-logged, non-caught exceptions are logged - https://phabricator.wikimedia.org/T103394#2538988 (10Legoktm) a:05Legoktm>03None [08:32:10] (03PS2) 10Legoktm: Remove $wgEchoNewMsgAlert and 'echo-show-alert' preference [extensions/Echo] - 10https://gerrit.wikimedia.org/r/223213 (https://phabricator.wikimedia.org/T58845) [08:32:17] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Flow, 10DBA, 13Patch-For-Review: Cleanup ptwikibooks conversion - https://phabricator.wikimedia.org/T119509#2539190 (10matthiasmullie) @jcrespo: you're `jynus`, right? [08:32:46] (03CR) 10Legoktm: "PS2: Rebased." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/223213 (https://phabricator.wikimedia.org/T58845) (owner: 10Legoktm) [08:39:57] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Flow, 10DBA, 13Patch-For-Review: Cleanup ptwikibooks conversion - https://phabricator.wikimedia.org/T119509#2539197 (10jcrespo) Yes, it is on my [[ https://phabricator.wikimedia.org/p/jcrespo/ | profile ]]. [08:52:23] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Flow, 10DBA, 13Patch-For-Review: Cleanup ptwikibooks conversion - https://phabricator.wikimedia.org/T119509#2539224 (10matthiasmullie) I had no idea this was on Phabricator's user pages as well. Just updated mine too, thanks! :) [09:02:35] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Flow, 10DBA, 13Patch-For-Review: Cleanup ptwikibooks conversion - https://phabricator.wikimedia.org/T119509#2539245 (10jcrespo) Backups done, on standby if my help is needed. [09:15:37] 06Collaboration-Team-Triage, 10Flow, 10VisualEditor, 10VisualEditor-MediaWiki-Links, and 3 others: [Regression wmf23] On creating a wikilink/an inter-wiki link, the nice regex strips section parts of the link - https://phabricator.wikimedia.org/T112898#2539267 (10QuimGil) Was this fix deployed yesterday in... [09:45:50] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Flow, 10DBA, 13Patch-For-Review: Cleanup ptwikibooks conversion - https://phabricator.wikimedia.org/T119509#2539315 (10matthiasmullie) I ran a very small test on **tin**: `mwscript extensions/Flow/maintenance/FlowRemoveOldTopics.ph... [11:43:07] 06Collaboration-Team-Triage, 10Flow, 10VisualEditor, 10VisualEditor-MediaWiki-Links, and 3 others: [Regression wmf23] On creating a wikilink/an inter-wiki link, the nice regex strips section parts of the link - https://phabricator.wikimedia.org/T112898#1649374 (10Johan) Isn't that because it'll be deployed... [11:43:28] 06Collaboration-Team-Triage, 10Flow, 10VisualEditor, 10VisualEditor-MediaWiki-Links, and 3 others: [Regression wmf23] On creating a wikilink/an inter-wiki link, the nice regex strips section parts of the link - https://phabricator.wikimedia.org/T112898#2539511 (10Johan) Working fine on the test wiki for me. [11:54:57] 06Collaboration-Team-Triage, 10Flow, 10MediaWiki-extensions-UserMerge: Exception when using UserMerge and Flow together - https://phabricator.wikimedia.org/T142483#2539536 (10Aklapper) Please provide specific version information that might allow reproducing your setup - which exact Git revision? Which exact... [12:03:52] 06Collaboration-Team-Triage, 10Notifications, 10Mention-Notification, 06TCB-Team, and 2 others: Update mention sending information - https://phabricator.wikimedia.org/T138918#2539592 (10WMDE-leszek) Some progress has been made - Failed mention and Succesful mention notifications have been described in http... [12:04:08] 06Collaboration-Team-Triage, 10Notifications, 10Mention-Notification, 06TCB-Team, and 2 others: Update mention sending information - https://phabricator.wikimedia.org/T138918#2539593 (10WMDE-leszek) p:05Triage>03Normal [12:08:49] 03Collab-Team-Q1-July-Sep-2016, 10MediaWiki-extensions-GuidedTour, 05MW-1.27-release-notes, 13Patch-For-Review, 07Technical-Debt: GuidedTour: Remove use of sync XHR - use mw.Api and async for parsing - https://phabricator.wikimedia.org/T46925#2539601 (10phuedx) 05Open>03Resolved Per T46925#2539471. [12:09:32] 03Collab-Team-Q1-July-Sep-2016, 10MediaWiki-extensions-GuidedTour, 05MW-1.27-release-notes, 13Patch-For-Review, and 2 others: GuidedTour: Remove use of sync XHR - use mw.Api and async for parsing - https://phabricator.wikimedia.org/T46925#2539617 (10phuedx) [12:16:25] 06Collaboration-Team-Triage, 10Notifications, 10Collaboration-Community-Engagement, 06Community-Liaisons (Jul-Sep-2016): Review if "notices" are not translated by "messages" for Notifications fly-outs on TranslateWiki - https://phabricator.wikimedia.org/T140645#2539635 (10Trizek-WMF) [12:17:39] 03Collab-Team-Q1-July-Sep-2016, 10Notifications, 10Collaboration-Community-Engagement, 13Patch-For-Review, and 2 others: Change 'messages' to 'notices' internally (method names and such) - https://phabricator.wikimedia.org/T139682#2539644 (10Trizek-WMF) [12:17:41] 03Collab-Team-Q1-July-Sep-2016, 10Notifications, 13Patch-For-Review: Change 'messages' to 'notices' everywhere (where 'messages' referred to a section name) - https://phabricator.wikimedia.org/T139520#2539645 (10Trizek-WMF) [12:17:43] 06Collaboration-Team-Triage, 10Notifications, 10Collaboration-Community-Engagement, 06Community-Liaisons (Jul-Sep-2016): Review if "notices" are not translated by "messages" for Notifications fly-outs on TranslateWiki - https://phabricator.wikimedia.org/T140645#2471134 (10Trizek-WMF) 05Open>03Resolved... [12:23:28] (03PS8) 10Jakob: Send notification for mentions on changes [extensions/Echo] - 10https://gerrit.wikimedia.org/r/300528 (https://phabricator.wikimedia.org/T138938) [13:18:17] 03Collab-Team-Q1-July-Sep-2016, 10Notifications, 07Regression, 07User-notice: [betalabs-Regression] cross-wiki bundle displays only wiki names - no notifications - https://phabricator.wikimedia.org/T140836#2477647 (10Johan) This has been marked for inclusion in Tech News, so asking to be on the safe side:... [13:19:08] 03Collab-Team-Q1-July-Sep-2016, 10Notifications, 07Regression, 07User-notice: [betalabs-Regression] Incorrect count when only cross-wiki notifications are present - https://phabricator.wikimedia.org/T140327#2460353 (10Johan) This has been marked for inclusion in Tech News, so asking to be on the safe side:... [15:03:10] 06Collaboration-Team-Triage, 10Flow, 10MediaWiki-extensions-UserMerge: Exception when using UserMerge and Flow together - https://phabricator.wikimedia.org/T142483#2539963 (10Tuxxic) OK - sorry. Here are the files I downloaded : UserMerge-REL1_27-31ea86d.tar.gz Flow-REL1_27-af307cc.tar.gz MediaWiki 1.27.0... [15:05:49] 06Collaboration-Team-Triage, 10Flow, 10Analytics, 06Community-Tech, and 6 others: statistics about edit conflicts according to page type - https://phabricator.wikimedia.org/T139019#2539972 (10Lea_WMDE) fyi: @addshore just updated https://grafana.wikimedia.org/dashboard/db/mediawiki-edit-conflicts to show e... [15:30:59] 06Collaboration-Team-Triage, 10Notifications, 10Mention-Notification, 06TCB-Team, and 7 others: Bundle mention notifications per save - https://phabricator.wikimedia.org/T140224#2540015 (10Charlie_WMDE) Visual update. Please correct me if there's a mistake {F4350986} {F4350987} {F4350991} {F4350988} {... [16:14:57] 03Collab-Team-Q1-July-Sep-2016, 10Notifications, 07Regression, 07User-notice: [betalabs-Regression] cross-wiki bundle displays only wiki names - no notifications - https://phabricator.wikimedia.org/T140836#2477647 (10Catrope) >>! In T140836#2539739, @Johan wrote: > This has been marked for inclusion in Tec... [16:16:13] 03Collab-Team-Q1-July-Sep-2016, 10Notifications, 07Regression, 07User-notice: [betalabs-Regression] Incorrect count when only cross-wiki notifications are present - https://phabricator.wikimedia.org/T140327#2460353 (10Catrope) >>! In T140327#2539743, @Johan wrote: > This has been marked for inclusion in Te... [16:23:17] matt_flaschen: update: the script has just stopped running [16:23:28] all bad topics should’ve been removed now [16:23:39] I’ll do some checking tonight & tomorrow [16:25:11] 03Collab-Team-Q1-July-Sep-2016, 10Notifications, 07Regression, 07User-notice: [betalabs-Regression] cross-wiki bundle displays only wiki names - no notifications - https://phabricator.wikimedia.org/T140836#2477647 (10SBisson) I don't think this is completely fixed. https://gerrit.wikimedia.org/r/301382 sti... [16:26:32] Thanks, mlitn. [16:33:45] (03Abandoned) 10Paladox: Update JS closure style [extensions/Echo] - 10https://gerrit.wikimedia.org/r/253899 (owner: 10Paladox) [16:34:45] (03Abandoned) 10Paladox: Update grunt-jscs to 2.6.0 [extensions/Echo] - 10https://gerrit.wikimedia.org/r/243503 (owner: 10Paladox) [16:42:50] stephanebisson: I guess I need some help with the following: I want to know how many deleted (purged) events user ET27 has. So I do [16:42:55] stephanebisson: select event_type, event_deleted from echo_event join echo_notification on event_id=notification_event where event_id in (select notification_event from echo_notification where notification_user=7060); [16:44:17] stephanebisson: all event_deleted=0 and the count of events is correct. Hmm... I cannot join on event_id=notification_event? [16:45:04] etonkovidova: select count(*) from echo_event join echo_notification on event_id=notification_event where event_deleted=1 and notification_user=7060; [16:46:10] stephanebisson: ok - trying [16:47:21] etonkovidova: also interesting: select notification_user,count(*) from echo_event join echo_notification on event_id=notification_event where event_deleted=1 group by notification_user; [16:47:48] stephanebisson: kk [16:49:06] stephanebisson: select count(*) from echo_event join echo_notification on event_id=notification_event where event_deleted=1 and notification_user=7060; [16:49:42] stephanebisson: gives 0 ?!!! why -I clearly see in UI that the notifcations are not displayed [16:50:43] etonkovidova: there is a distinction between a notification that cannot be rendered, and a notification that is explicitely deleted. You probably have tons of the first and none the second. [16:51:24] etonkovidova: https://gerrit.wikimedia.org/r/#/c/301382/ will turn the first into the second and fix pagination, amoung other things [16:51:32] but it's still waiting for review... [16:51:44] ping RoanKattouw ^ [16:51:50] ;) [16:52:37] stephanebisson: I am thinking.... [17:03:59] stephanebisson: Yes, your moderation changes have been the victim of my not reviewing changes the past few day s:/ [17:15:53] (03CR) 10Mooeypoo: [wip] Allow watching and unwatching of boards from Flow notifications (031 comment) [extensions/Flow] - 10https://gerrit.wikimedia.org/r/303315 (https://phabricator.wikimedia.org/T132975) (owner: 10Mooeypoo) [17:16:02] (03CR) 10Mooeypoo: [wip] Add dynamic secondary actions to items (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/303314 (https://phabricator.wikimedia.org/T132975) (owner: 10Mooeypoo) [17:32:13] Trizek, what is a good time to discuss the communication for the one-year login patch? [18:46:54] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Flow, 10DBA, 13Patch-For-Review: Cleanup ptwikibooks conversion - https://phabricator.wikimedia.org/T119509#2541108 (10matthiasmullie) @jcrespo: these are 2 of the 4 entries that were deleted in DB (as far as I can tell) but are st... [18:55:33] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Flow, 10DBA, 13Patch-For-Review: Cleanup ptwikibooks conversion - https://phabricator.wikimedia.org/T119509#2541151 (10matthiasmullie) The script to remove all untouched imported topics has completed running. The only Flow topics t... [18:59:23] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Flow, 10DBA, 13Patch-For-Review: Cleanup ptwikibooks conversion - https://phabricator.wikimedia.org/T119509#2541154 (10jcrespo) Definitely; I will check the rows mentioned tomorrow, too. [19:00:55] 10Collab-Notifications-Page, 03Collab-Team-Q1-July-Sep-2016: Special:Notifications - no-js page briefly displayed when page is loading - https://phabricator.wikimedia.org/T142451#2541160 (10Etonkovidova) [19:10:11] array_filter( array, callback ) but array_map( callback, array ). Thanks PHP [20:58:50] 06Collaboration-Team-Triage, 10Thanks, 10Revision-Slider, 06TCB-Team: RevisionSlider fails to re-trigger the Thanks JS code when the diff changes - https://phabricator.wikimedia.org/T142636#2541631 (10Jdforrester-WMF) [21:09:57] 03Collab-Team-Q1-July-Sep-2016, 10Notifications: Cannot see or clear unread notifications at Beta Cluster - https://phabricator.wikimedia.org/T141816#2513204 (10Etonkovidova) Re-check after the fix for {T93673} is merged. Curious that re-running the above queries, I do not see the event_page_id 13955, 116853... [21:12:57] 03Collab-Team-2016-Apr-Jun-Q4, 06Collaboration-Team-Triage, 10Flow, 05Security, 07Vuln-Infoleak: Topics that are deleted then suppressed expose topic title in public deletion log - https://phabricator.wikimedia.org/T137593#2541712 (10demon) [21:13:15] 06Collaboration-Team-Triage, 10Flow, 05Security, 07Security-General: Applying deletelogentry restrictions to flow deletion log entries does not work - https://phabricator.wikimedia.org/T137288#2541716 (10demon) [21:14:14] 03Collab-Team-2016-Apr-Jun-Q4, 06Collaboration-Team-Triage, 10Flow, 05Security, 07Vuln-Infoleak: Topics that are deleted then suppressed expose topic title in public deletion log - https://phabricator.wikimedia.org/T137593#2541721 (10demon) [21:14:17] 06Collaboration-Team-Triage, 10Flow, 05Security, 07Security-General: Applying deletelogentry restrictions to flow deletion log entries does not work - https://phabricator.wikimedia.org/T137288#2541722 (10demon) [21:27:18] (03PS4) 10Mooeypoo: Log Special:Notifications page visits [extensions/Echo] - 10https://gerrit.wikimedia.org/r/303079 (https://phabricator.wikimedia.org/T121939) [21:27:49] 06Collaboration-Team-Triage, 10Thanks, 10Revision-Slider, 06TCB-Team: RevisionSlider fails to re-trigger the Thanks JS code when the diff changes - https://phabricator.wikimedia.org/T142636#2541631 (10Legoktm) We probably need an mw.hook thing that thanks can listen to? [21:29:08] 06Collaboration-Team-Triage, 10Thanks, 10Revision-Slider, 06TCB-Team: RevisionSlider fails to re-trigger the Thanks JS code when the diff changes - https://phabricator.wikimedia.org/T142636#2541765 (10Jdforrester-WMF) >>! In T142636#2541757, @Legoktm wrote: > We probably need an mw.hook thing that thanks c... [21:30:27] 06Collaboration-Team-Triage, 10Thanks, 10Revision-Slider, 06TCB-Team: RevisionSlider fails to re-trigger the Thanks JS code when the diff changes - https://phabricator.wikimedia.org/T142636#2541780 (10Legoktm) There is `mw.hook( 'wikipage.diff' ).fire( $wikiDiff.find( 'table.diff' ) );`, but that runs on p... [21:32:50] 06Collaboration-Team-Triage, 10Thanks: "MediaWiki:Thanks-confirmation2" should support wiki syntax - https://phabricator.wikimedia.org/T142428#2541784 (10Guycn2) [[ https://translatewiki.net/wiki/MediaWiki:Thanks-confirmation-special/en | MediaWiki:Thanks-confirmation-special ]] should support wiki syntax as w... [21:33:40] RoanKattouw: https://phabricator.wikimedia.org/T141816 - Nick filed; event_page_id=13955 does not exist anywhere [21:48:51] 06Collaboration-Team-Triage, 10Thanks, 10Revision-Slider, 06TCB-Team: RevisionSlider fails to re-trigger the Thanks JS code when the diff changes - https://phabricator.wikimedia.org/T142636#2541922 (10Jdforrester-WMF) Helpful. [22:09:59] (03CR) 10Catrope: [C: 04-1] Moderate notifications (033 comments) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/301382 (https://phabricator.wikimedia.org/T140327) (owner: 10Sbisson) [22:11:42] (03CR) 10Catrope: [C: 031] Moderate events that don't have target-page entries [extensions/Flow] - 10https://gerrit.wikimedia.org/r/301618 (https://phabricator.wikimedia.org/T93673) (owner: 10Sbisson) [22:15:15] (03PS5) 10Mooeypoo: [wip] Add dynamic secondary actions to items [extensions/Echo] - 10https://gerrit.wikimedia.org/r/303314 (https://phabricator.wikimedia.org/T132975) [22:15:26] (03PS4) 10Mooeypoo: [wip] Allow watching and unwatching of boards from Flow notifications [extensions/Flow] - 10https://gerrit.wikimedia.org/r/303315 (https://phabricator.wikimedia.org/T132975) [22:42:37] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Notifications, 05MW-1.28-release-notes, 05WMF-deploy-2016-08-09_(1.28.0-wmf.14): Multiple "You made your edit!" notifications - https://phabricator.wikimedia.org/T128249#2068619 (10Etonkovidova) Checked in betalabs - wo... [22:47:25] 03Collab-Team-Q1-July-Sep-2016, 10Collaboration-Team-Archive-2015-2016, 10Notifications, 05MW-1.28-release-notes, 05WMF-deploy-2016-08-09_(1.28.0-wmf.14): Multiple "You made your edit!" notifications - https://phabricator.wikimedia.org/T128249#2542163 (10Catrope) In production, all `wmf.13` e... [23:28:25] 10Collab-Notifications-Page, 03Collab-Team-Q1-July-Sep-2016, 13Patch-For-Review, 05WMF-deploy-2016-08-09_(1.28.0-wmf.14): In mobile view, bundled notifications extend outside the screen limit - https://phabricator.wikimedia.org/T140523#2542325 (10Etonkovidova) Checked in betlabs (iPHone 6s iOS 9.2.3) {F435... [23:41:56] (03CR) 10jenkins-bot: [V: 04-1] Log Special:Notifications page visits [extensions/Echo] - 10https://gerrit.wikimedia.org/r/303079 (https://phabricator.wikimedia.org/T121939) (owner: 10Mooeypoo) [23:43:03] 03Collab-Team-Q1-July-Sep-2016, 10Notifications: [mobile] Different size boxes for bundled notifications - https://phabricator.wikimedia.org/T142662#2542355 (10Etonkovidova) [23:55:06] 03Collab-Team-Q1-July-Sep-2016, 10Notifications, 07Mobile: Different size boxes for bundled notifications - https://phabricator.wikimedia.org/T142662#2542452 (10Danny_B)