[07:18:00] (03PS1) 10Catrope: Fix typo: @inherits->@extends [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272443 [07:19:30] (03PS1) 10Catrope: [WIP] Expose individual events inside bundles in API response [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272444 [08:06:45] (03PS2) 10Catrope: [WIP] Expandable bundles proof of concept [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272444 [08:07:27] (03PS3) 10Catrope: [WIP] Expandable bundles proof of concept [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272444 (https://phabricator.wikimedia.org/T114356) [08:11:17] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Expandable bundles proof of concept [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272444 (https://phabricator.wikimedia.org/T114356) (owner: 10Catrope) [08:13:11] 3Collaboration-Team-Current, 10Thanks, 10Notifications: Cross-wiki notifications: Links for thanks notifications point to the local wiki - https://phabricator.wikimedia.org/T125738#2050553 (10Trizek-WMF) p:5Triage>3High Triaged as Hugh, due to the important number of feedback received. [08:28:06] 6Collaboration-Team-Backlog, 10Notifications, 7Performance: Echo makes too many round-trips to memcached - https://phabricator.wikimedia.org/T127687#2050564 (10ori) [08:56:14] (03CR) 10Legoktm: [C: 032] Fix typo: @inherits->@extends [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272443 (owner: 10Catrope) [09:04:35] (03Merged) 10jenkins-bot: Fix typo: @inherits->@extends [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272443 (owner: 10Catrope) [09:44:23] 3Collaboration-Team-Current, 10Thanks, 10Notifications: Cross-wiki notifications: Links for thanks notifications point to the local wiki - https://phabricator.wikimedia.org/T125738#2050671 (10Trizek-WMF) >>! In T125738#2048125, @Matanya wrote: > I can reproduce that between he.wiki and a thank on pl.wiki. >... [11:10:16] 6Collaboration-Team-Backlog, 10Flow: "Create Flow boards in any location" permission lost on zh.wp - https://phabricator.wikimedia.org/T127693#2050743 (10Trizek-WMF) [12:29:46] 3Collaboration-Team-Current, 10Flow, 13Patch-For-Review, 5WMF-deploy-2016-02-23_(1.27.0-wmf.15): ref_target should be NOT NULL - https://phabricator.wikimedia.org/T122111#2050844 (10matthiasmullie) I misread @Mattflaschen's earlier comment to mean that "production is fine", when in fact he was only saying... [12:31:25] 3Collaboration-Team-Current, 10Flow, 10DBA, 7Blocked-on-schema-change, and 2 others: ref_target should be NOT NULL - https://phabricator.wikimedia.org/T122111#2050845 (10matthiasmullie) [12:43:01] 3Collaboration-Team-Current, 10Notifications: Show descriptions for the actions under the "..." menu on notifications - https://phabricator.wikimedia.org/T126738#2050849 (10Pginer-WMF) > Can we mix items with descriptions and items without? Or does the design require it? My first option would be to provide a s... [12:49:32] (03PS2) 10Matthias Mullie: Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) [12:52:29] 6Collaboration-Team-Backlog, 10Notifications: Cross-wiki notifications: Links for reverts notifications point to the local wiki - https://phabricator.wikimedia.org/T127697#2050858 (10Trizek-WMF) [12:53:57] 6Collaboration-Team-Backlog, 10Notifications: Cross-wiki notifications: Links for reverts notifications point to the local wiki - https://phabricator.wikimedia.org/T127697#2050873 (10Trizek-WMF) p:5Triage>3High Triaged to high, due to the important impact of that kind of notification [13:34:25] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-02-23_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2050899 (10SBisson) >>! In T125693#2048394, @Etonkovidova w... [13:35:57] (03PS2) 10Haritha28: Echo congragulations threshold should'nt be hardcoded [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272308 (https://phabricator.wikimedia.org/T127311) [13:50:13] (03PS2) 10Zfilipin: Fix Cucumber tags for Selenium tests [extensions/Echo] - 10https://gerrit.wikimedia.org/r/271533 (https://phabricator.wikimedia.org/T94152) [13:51:18] (03PS3) 10Zfilipin: Fix Cucumber tags for Selenium tests [extensions/Echo] - 10https://gerrit.wikimedia.org/r/271533 (https://phabricator.wikimedia.org/T94152) [13:52:13] (03CR) 10Zfilipin: "I have disable flow on user talk for selenium user (not sure if that will break flow tests) and updated the cucumber tags. All tests shoul" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/271533 (https://phabricator.wikimedia.org/T94152) (owner: 10Zfilipin) [13:56:41] (03PS3) 10Matthias Mullie: Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) [14:01:06] (03CR) 10jenkins-bot: [V: 04-1] Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) (owner: 10Matthias Mullie) [14:10:00] (03CR) 10Sbisson: [C: 032] Fix Cucumber tags for Selenium tests [extensions/Echo] - 10https://gerrit.wikimedia.org/r/271533 (https://phabricator.wikimedia.org/T94152) (owner: 10Zfilipin) [14:17:00] (03Merged) 10jenkins-bot: Fix Cucumber tags for Selenium tests [extensions/Echo] - 10https://gerrit.wikimedia.org/r/271533 (https://phabricator.wikimedia.org/T94152) (owner: 10Zfilipin) [15:16:47] (03PS4) 10Matthias Mullie: Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) [15:27:39] (03PS1) 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) [15:30:17] 6Collaboration-Team-Backlog, 10Flow, 7Browser-Tests, 13Patch-For-Review: Fix or delete failing Flow browsertests Jenkins jobs - https://phabricator.wikimedia.org/T94153#2051153 (10zeljkofilipin) a:3zeljkofilipin [15:31:57] (03CR) 10jenkins-bot: [V: 04-1] 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) (owner: 10Zfilipin) [15:39:16] 6Collaboration-Team-Backlog, 10Flow, 7Browser-Tests, 13Patch-For-Review: Disable Flow scenarios that fail at en.wikipedia.beta.wmflabs.org from running daily - https://phabricator.wikimedia.org/T94153#2051214 (10zeljkofilipin) [16:08:44] (03CR) 10Zfilipin: "recheck" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/272471 (https://phabricator.wikimedia.org/T94153) (owner: 10Zfilipin) [16:20:05] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-02-23_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2051403 (10jmatazzoni) @SBisson, Is there a reason why you... [16:21:22] 6Collaboration-Team-Backlog, 10Wikimedia-Extension-setup, 13Patch-For-Review: Install WikiLove extension at Azerbaijani Wikipedia - https://phabricator.wikimedia.org/T119727#2051406 (10demon) 5Open>3Resolved [16:22:13] 6Collaboration-Team-Backlog, 10Flow, 7Browser-Support-Internet-Explorer, 7Tracking: IE 8 support - Flow issues (tracking) - https://phabricator.wikimedia.org/T87121#2051409 (10Aklapper) p:5Normal>3Lowest [16:38:38] 3Collaboration-Team-Current, 10MediaWiki-extensions-PageCuration, 10Notifications, 13Patch-For-Review, 5WMF-deploy-2016-02-23_(1.27.0-wmf.15): Update PageTriage notifications to new language and format - https://phabricator.wikimedia.org/T125693#2051548 (10SBisson) >>! In T125693#2051403, @jmatazzoni wro... [16:39:00] (03CR) 10Florianschmidtwelzow: [C: 04-1] "needs rebase." [extensions/WikiLove] - 10https://gerrit.wikimedia.org/r/272282 (https://phabricator.wikimedia.org/T127608) (owner: 10Ori.livneh) [16:44:58] 6Collaboration-Team-Backlog, 10Wikimedia-Extension-setup, 13Patch-For-Review: Install WikiLove extension at Azerbaijani Wikipedia - https://phabricator.wikimedia.org/T119727#2051610 (10Serkanland) Thank you very much, @Dereckson! It works! [16:46:26] 6Collaboration-Team-Backlog, 10Wikimedia-Extension-setup, 13Patch-For-Review: Install WikiLove extension at Azerbaijani Wikipedia - https://phabricator.wikimedia.org/T119727#2051615 (10Dereckson) You're welcome. [17:37:03] 6Collaboration-Team-Backlog, 10Notifications, 6Operations: DomainException on Special:Notifications on mediawiki.org - https://phabricator.wikimedia.org/T127728#2051854 (10Dereckson) [17:39:17] 6Collaboration-Team-Backlog, 10Notifications, 6Operations: DomainException on Special:Notifications on mediawiki.org - https://phabricator.wikimedia.org/T127728#2051878 (10hoo) [17:39:55] 6Collaboration-Team-Backlog, 10Notifications: DomainException on Special:Notifications on mediawiki.org - https://phabricator.wikimedia.org/T127728#2051854 (10hoo) [17:56:52] (03CR) 10Sbisson: [C: 032] Cleanup Cucumber tags in Selenium tests [extensions/Flow] - 10https://gerrit.wikimedia.org/r/271539 (https://phabricator.wikimedia.org/T127423) (owner: 10Zfilipin) [18:05:32] (03Merged) 10jenkins-bot: Cleanup Cucumber tags in Selenium tests [extensions/Flow] - 10https://gerrit.wikimedia.org/r/271539 (https://phabricator.wikimedia.org/T127423) (owner: 10Zfilipin) [18:10:30] 3Collaboration-Team-Current, 10Flow, 13Patch-For-Review, 5WMF-deploy-2016-02-23_(1.27.0-wmf.15), 7user-notice: Create new notification type for when a board description changes - https://phabricator.wikimedia.org/T125429#2052101 (10Trizek-WMF) [18:20:52] 6Collaboration-Team-Backlog, 10Notifications, 7WorkType-NewFunctionality: Clarify read/unread status for notifications - https://phabricator.wikimedia.org/T126214#2052155 (10Pginer-WMF) >>! In T126214#2033717, @Volker_E wrote: > @Pginer-WMF I'd argue for proposal B), as //read// isn't necessarily congruent w... [18:22:32] 3Collaboration-Team-Current, 10Notifications, 7Performance: Echo makes too many round-trips to memcached - https://phabricator.wikimedia.org/T127687#2052171 (10Catrope) [18:29:58] 3Collaboration-Team-Current, 10Notifications: Don't hide messages badge when user has never had messages - https://phabricator.wikimedia.org/T127731#2052206 (10Catrope) [18:30:49] 3Collaboration-Team-Current, 10Notifications, 7Performance: Echo makes too many round-trips to memcached - https://phabricator.wikimedia.org/T127687#2050564 (10Catrope) Re `had:messages`: I'm proposing we get rid of the code that uses that key, see {T127731}. As for the others, I agree a single key would be... [18:38:03] 3Collaboration-Team-Current, 10Notifications, 7I18n, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Display human-readable wiki names for cross-wiki notifications - https://phabricator.wikimedia.org/T121936#2052265 (10Nemo_bis) [18:42:49] 3Collaboration-Team-Current, 10Notifications: Cross-wiki notifications: Links for reverts notifications point to the local wiki - https://phabricator.wikimedia.org/T127697#2052277 (10Catrope) [18:43:06] 3Collaboration-Team-Current, 10Notifications: Cross-wiki: click on mouse wheel, to open the notification in a new tab, collapses partially the cross-wiki notification bundle - https://phabricator.wikimedia.org/T127416#2052278 (10Catrope) [18:45:30] (03CR) 10Matthias Mullie: [C: 04-1] "Need "same language = only wiki name"" [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) (owner: 10Matthias Mullie) [18:48:19] 6Collaboration-Team-Backlog, 10Notifications, 7WorkType-NewFunctionality: Clarify read/unread status for notifications - https://phabricator.wikimedia.org/T126214#2052306 (10Volker_E) @Pginer-WMF That's an interesting point and Google Inbox an interesting example. Then again, one of their main selling argume... [18:48:51] (03PS5) 10Matthias Mullie: Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) [18:49:39] (03PS6) 10Matthias Mullie: Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) [19:08:16] (03PS7) 10Matthias Mullie: Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) [19:09:59] (03PS8) 10Matthias Mullie: Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) [19:15:35] 6Collaboration-Team-Backlog, 10Flow: "Create Flow boards in any location" permission lost on zh.wp - https://phabricator.wikimedia.org/T127693#2052446 (10Mattflaschen) a:3Mattflaschen [19:48:50] 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#1995020 (10Etonkovidova) @SBisson {icon exclamat... [19:52:41] James_F: hi, are you leading the crosswiki notification stuff ? [19:53:13] or is it RoanKattouw ? [19:53:55] anyhow, wanted to speak to one of you about it, since it was enabled in he.wiki, and i'd like to share some real-world feedback [19:55:43] 3Collaboration-Team-Current, 10Thanks, 10Notifications: Cross-wiki notifications: Links for thanks notifications point to the local wiki - https://phabricator.wikimedia.org/T125738#2052563 (10Mattflaschen) a:3Mattflaschen [19:57:50] thanks for taking this ^ matt_flaschen [19:58:16] matanya, no problem. RoanKattouw is team lead and jmatazzoni__ is project manager. [19:58:42] ah, cool, thanks. [20:01:27] 6Collaboration-Team-Backlog, 6Editing-Department: Consider providing a mechanism to prompt users to congratulate other users for reaching milestones - https://phabricator.wikimedia.org/T126820#2052581 (10TheDJ) [20:02:09] matanya: Hey there :) happy to hear your feedback [20:02:19] hi RoanKattouw [20:02:30] first of all: THANK YOU ALL! [20:02:42] long requested feature well executed [20:03:36] matt_flaschen, I am a little confused about the echo api comment about the 'finishLoading' event -- we discussed on IRC on friday, but to make sure -- did we agree the event should only be emitted in success? I think that was the agreement? or am I remebering wrong? [20:03:55] It's not entirely clear from the code review message which strategy we settled on [20:04:36] RoanKattouw: some feedback : the wiki name is not localized in the message you get [20:04:57] so i have messages from commonswiki in hebrew wiki [20:05:15] so usual rtl joy .. [20:05:22] mooeypoo: ^ ;) [20:07:01] matanya: Do you mean the part where it says "enwiki" instead of "English Wikipedia", or the part where user names like "Roan Kattouw (WMF)" look messed up, or something else? [20:07:04] In addition, once you have read crosswiki notifications they disappear [20:07:30] the wiki name, and the user name, if in other than source lang [20:07:32] matanya, we're actually working on that -- if you mean the wiki names [20:07:34] Right [20:07:35] screenshots in a few [20:07:49] The user name we have a bug for, mooeypoo noticed that a few days ago [20:08:02] ok. [20:08:18] oh, now it is lost, cause i read it, so can't screenshot :/ [20:08:23] The wiki names, we have a patch that'll probably be merged today that provides for translated wiki names, but we haven't looked at bidi isolation there yet (not sure if we'll need that once wiki names are translated) [20:08:44] matanya: Does your screenshot look like https://phabricator.wikimedia.org/T127407? [20:08:48] (03CR) 10TheDJ: [C: 04-1] Echo congragulations threshold should'nt be hardcoded (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272308 (https://phabricator.wikimedia.org/T127311) (owner: 10Haritha28) [20:09:01] exatly [20:09:04] exactly [20:10:10] Can i preserve the messages after reading them ? [20:10:34] Hmm, re read messages, that one's tricky :( [20:10:49] Because of the architecture we don't have easy access to read messages on other wikis [20:11:07] How did the message become read? Did you explicitly mark it as read, or did you click it? [20:11:45] clicked [20:13:01] Hmm OK [20:13:13] Yeah that is annoying [20:13:23] In theory, I suppose, once you click the notification, you are taken to the wiki that notification came from [20:13:37] yes [20:13:41] that is what happens [20:13:43] And if you look at your notifications there, you'll still see it [20:14:33] Yeah, it's annoying though, I agree [20:15:33] to recap on this, the main problem is inconsistency [20:16:04] You mean, there's a notification that you see on wiki A but not on wiki B? [20:16:07] the list of messages in the flyout changes, and the user might not understand where some of his messgaes went to [20:16:55] All in one wiki (A in this case) [20:17:22] one more: mention in another wiki doesn't work, you probably know that [20:17:41] What do you mean? [20:17:41] Oh, I think I know [20:17:55] e.g https://commons.wikimedia.org/wiki/Commons:Sandbox [20:17:56] matanya: Something like https://phabricator.wikimedia.org/T125738 ? [20:18:11] no, more severe [20:18:32] i am going to be disconnected in a few [20:18:48] freenode is rebooting this server [20:18:55] Hmm... that should work [20:18:56] Not aware of that being broken, that's news [20:19:04] i'll reconnect shortly [20:19:38] ok, so probably should be phab'ed [20:20:09] Yes [20:22:39] (03PS19) 10Mooeypoo: Echo API layer [extensions/Echo] - 10https://gerrit.wikimedia.org/r/264766 [20:23:51] (03CR) 10Mooeypoo: Echo API layer (032 comments) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/264766 (owner: 10Mooeypoo) [20:24:07] ok, i am back [20:26:13] OK, yeah, so I'm not aware of a bug with cross-wiki mention not working, if you have time to write up a phab task for that that'd be great [20:26:44] I will [20:27:12] Hah etonkovidova is telling me she's noticed something similar, something about other namespaces [20:27:41] Hmm, hold on [20:27:42] Lemme look at your test [20:27:54] matanya: Yeah, I understand what happened to you [20:28:05] There's this long-standing annoyance about how Echo is really particular about what is and isn't a mention [20:28:27] ok, and ? [20:28:40] In particular, I think you have to sign (~~~~) in order for a mention to be picked up [20:28:43] it only works on talk: namespace ? [20:29:22] yes, i forget about this limitation, similar to "don't edit your posts, it is not a mention" [20:29:33] I'm not sure about namespace stuff [20:29:44] I would expect it to work outside of the talk namespaces because we have stuff like [[Wikipedia:Village pump]] [20:29:44] and to: "don't mention too many people" [20:29:49] Looking through the code now [20:29:59] Oh, yeah, we raised that from 20 to 50 a while ago [20:30:26] ok, tested [20:30:33] with a sig it works [20:30:43] I don't see anything about namespaces, I think it's just about signing [20:31:05] it is not related crosswiki notifcation then, it is a "feature" in echo :) [20:31:08] The other day legoktm wrote a user subpage about how he thinks we should redo mentions, because the current system kinda sucks [20:31:21] very confusing [20:31:23] It's because we don't want to accidentally pick up user page links ([[User:Matanya]]) as mentions [20:31:27] But the result is a confusing mess [20:31:29] and i don't get most of them [20:31:46] So lego said we should instead have something like {{#ping:matanya}} that's unambiguously a mention [20:31:46] I have a good example off jhand [20:32:01] we hacked something like that in he.wiki [20:32:07] ( {{ping|matanya}} is nothing magical, it's just a template that expands to @[[User:Matanya|matanya]]) [20:32:16] yes [20:32:38] And then there's {{noping}}, which I didn't even know about until last week [20:33:00] https://he.wikipedia.org/w/index.php?title=%D7%95%D7%99%D7%A7%D7%99%D7%A4%D7%93%D7%99%D7%94:%D7%93%D7%9C%D7%A4%D7%A7_%D7%99%D7%99%D7%A2%D7%95%D7%A5&diff=prev&oldid=18044106 [20:33:04] The fact that {{noping}} exists is a sign that something is wrong with the current system [20:33:08] didn't get pinged about this one [20:33:24] 3Collaboration-Team-Current, 10Notifications: Cross-wiki: click on mouse wheel, to open the notification in a new tab, collapses partially the cross-wiki notification bundle - https://phabricator.wikimedia.org/T127416#2052740 (10Mooeypoo) a:3Mooeypoo [20:33:30] matanya: Yeah I think that's because it's a multi-section edit :( [20:33:44] the main problem for me is people getting angry i "ignore" them [20:33:49] I'm sure there was a good reason for that rule, but the rules are really annoying most of the time [20:34:08] And there's no way they can verify that they pinged you, either [20:34:29] and when you say, sorry, it didn't mention, it sounds like "sorry, i was too busy to reply" or some other execuse [20:34:29] (03PS1) 10Mooeypoo: Keep group open on click [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272537 (https://phabricator.wikimedia.org/T127416) [20:34:43] 6Collaboration-Team-Backlog, 10Notifications: cross-wiki: Mention from non-main namespaces are not display - https://phabricator.wikimedia.org/T127750#2052748 (10Etonkovidova) [20:35:33] good timing ^ [20:35:43] 6Collaboration-Team-Backlog, 10Notifications: cross-wiki: Mention from non-main namespaces are not displayed - https://phabricator.wikimedia.org/T127750#2052753 (10Etonkovidova) [20:35:48] (03CR) 10Catrope: [C: 032] Keep group open on click [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272537 (https://phabricator.wikimedia.org/T127416) (owner: 10Mooeypoo) [20:37:09] RoanKattouw: one more: [20:37:55] when mentioning, if you get it in a wiki other than the wiki you where mentioned, it is not cleared from your unread notications on the target wiki [20:38:09] 3Collaboration-Team-Current, 10Notifications: cross-wiki: Mention from non-main namespaces are not displayed - https://phabricator.wikimedia.org/T127750#2052757 (10Catrope) [20:38:29] you need to actively click the flyout on target wiki to get it marked as read [20:40:47] 3Collaboration-Team-Current, 10Notifications: cross-wiki: Mention from non-main namespaces are not displayed - https://phabricator.wikimedia.org/T127750#2052762 (10Etonkovidova) 5Open>3Invalid [20:40:59] Yeah the bug that etonkovidova just closed is the reverse of that [20:41:51] Alerts are marked as read when the flyout is opened (which is nasty and we're going to get rid of that behavior) [20:42:00] But only local alerts [20:42:21] Foreign alerts are not marked as read when you see them, until you click them, or click the "X", or view them on the wiki they came from [20:42:34] This is a bit weird but we thought it'd be the least insane option [20:42:46] And we want to kill this whole mark-as-read-when-seen thing in the near future anyway [20:42:58] I hope that makes sense? [20:43:44] 3Collaboration-Team-Current, 10Notifications: cross-wiki: Mention from non-main namespaces are not displayed - https://phabricator.wikimedia.org/T127750#2052767 (10Etonkovidova) The order of vieweing Alerts caused a false alarm - if Alerts are viewed in a source wiki, then they will never appear in a cross-wik... [20:44:19] RoanKattouw: yeah, sort of [20:44:39] (03Merged) 10jenkins-bot: Keep group open on click [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272537 (https://phabricator.wikimedia.org/T127416) (owner: 10Mooeypoo) [20:44:51] 3Collaboration-Team-Current, 10Notifications: cross-wiki: Mention from non-main namespaces are not displayed - https://phabricator.wikimedia.org/T127750#2052724 (10Catrope) Recording outcome of in-person conversation: it turns out that this does work, but auto-mark-as-read makes it difficult to see it work. Be... [20:46:19] RoanKattouw: thank, i'll drop by if is see more stuff. Thank you all again for this great feature [20:47:48] Ugh why does Firefox let Ctrl+Q close the entire browser without confirmation [20:48:00] And it's right next to Ctrl+W which I use all the time [20:48:01] matanya: Thanks a lot for coming by and giving feedback! [20:52:42] (03CR) 10Catrope: [C: 032] Pick up on mentions in topic summaries [extensions/Flow] - 10https://gerrit.wikimedia.org/r/269105 (https://phabricator.wikimedia.org/T125120) (owner: 10Matthias Mullie) [20:53:00] (03CR) 10jenkins-bot: [V: 04-1] Pick up on mentions in topic summaries [extensions/Flow] - 10https://gerrit.wikimedia.org/r/269105 (https://phabricator.wikimedia.org/T125120) (owner: 10Matthias Mullie) [20:57:01] (03PS10) 10Catrope: Pick up on mentions in topic summaries [extensions/Flow] - 10https://gerrit.wikimedia.org/r/269105 (https://phabricator.wikimedia.org/T125120) (owner: 10Matthias Mullie) [20:57:21] 3Collaboration-Team-Current, 10Notifications, 7I18n, 13Patch-For-Review, 5WMF-deploy-2016-03-01_(1.27.0-wmf.15): Display human-readable wiki names for cross-wiki notifications - https://phabricator.wikimedia.org/T121936#2052838 (10Mooeypoo) I've added a section in "Add a wiki" procedure. Feel free to mak... [20:57:35] (03CR) 10Catrope: [C: 032] Pick up on mentions in topic summaries [extensions/Flow] - 10https://gerrit.wikimedia.org/r/269105 (https://phabricator.wikimedia.org/T125120) (owner: 10Matthias Mullie) [21:05:35] mooeypoo, re finishLoading, you started thinking about the bigger picture and whether we were timing the right thing at all: "If we want to measure only the UI appearing for the user, then there's no need to even look at the promise; we just put this in the toggle or something. If we want to measure with the promise (Which we are now and did before) then you're right, probably only successes count" [21:07:39] mooeypoo, if I understand right, timing.MediaWiki.echo.overlay.api should be based on the API promise (and probably use success), timing.MediaWiki.echo.overlay should probably be the overall UI. [21:12: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#2052957 (10Etonkovidova) Checked in betalabs. {icon check-... [21:13:53] (03CR) 10Catrope: [C: 032] Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) (owner: 10Matthias Mullie) [21:17:47] matt_flaschen, RoanKattouw_away made a good point that the .overlay checks how long it took for the user to get something *useful* in the overlay... that makes it dependent on the promise, I guess [21:17:59] either way, the current fix duplicates what we tested so far (success only) [21:18:11] so if we're changing it, we should probably change in another commit anyways ? [21:18:19] (03Merged) 10jenkins-bot: Pick up on mentions in topic summaries [extensions/Flow] - 10https://gerrit.wikimedia.org/r/269105 (https://phabricator.wikimedia.org/T125120) (owner: 10Matthias Mullie) [21:20:39] 6Collaboration-Team-Backlog, 10Flow, 10VisualEditor, 10VisualEditor-MediaWiki-Links: [Regression wmf23] On creating an inter-wiki link, the nice regex strips section parts of the link - https://phabricator.wikimedia.org/T112898#2052991 (10matmarex) [21:22:04] 3Collaboration-Team-Current, 10Notifications: DomainException on Special:Notifications on mediawiki.org - https://phabricator.wikimedia.org/T127728#2052993 (10Mattflaschen) p:5Triage>3High [21:22:28] (03Merged) 10jenkins-bot: Display human-readable wiki names for cross-wiki notifications [extensions/Echo] - 10https://gerrit.wikimedia.org/r/266474 (https://phabricator.wikimedia.org/T121936) (owner: 10Matthias Mullie) [21:23:27] 6Collaboration-Team-Backlog, 10Flow, 10VisualEditor, 10VisualEditor-MediaWiki-Links: [Regression wmf23] On creating an inter-wiki link, the nice regex strips section parts of the link - https://phabricator.wikimedia.org/T112898#2053004 (10Jdforrester-WMF) Eight dupes. Maybe we should fix this. :-( [21:23:30] 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#1052144 (10Nemo_bis) >>! In T90075#1989219, @matthiasmullie wrote: > Yes, I believe that was intentional: hidden topics are... [21:25:37] "It's because we don't want to accidentally pick up user page links (http://en.wikipedia.org/wiki/Special:Search?go=Go&search=User:Matanya) as mentions" This would definitely be a "Is it a feature or a bug thing?" in some cases (some people might want such casual links as mentions). But might be worth that tradeoff to simplify and get rid of noping, etc. [21:28:06] mooeypoo, can you clarify? How do timing.MediaWiki.echo.overlay and timing.MediaWiki.echo.overlay.api work in your latest version of the Echo API patch? [21:29:57] I'm running to a quick meeting, but they're *supposed* to work the same way they have before -- the .overlay was logging on the promise's ".then" which is success-only, so I moved it to success [21:30:55] mooeypoo, okay, I'll look at the patch. [21:30:56] the .api I'm not sure if we've missed it. We should probably add it to EchoAPI itself [21:31:03] I gotta run, bbiab [21:35:45] 6Collaboration-Team-Backlog, 10Flow: "Create Flow boards in any location" permission lost on zh.wp - https://phabricator.wikimedia.org/T127693#2053059 (10Mattflaschen) It looks right: ``` mattflaschen@terbium:~$ mwrepl zhwiki Welcome to HipHop Debugger! Type "help" or "?" for a complete list of commands. Pro... [21:42:47] RoanKattouw_away, the technical restrictions for Mentions, are documented at https://www.mediawiki.org/wiki/Manual:Echo#Mentions -- I believe the *main* thing that they/we are trying to avoid, is accidental pings when either: Refactoring content within a discussion, or archiving sections to a subpage. (In case that helps. :) [21:56:18] 6Collaboration-Team-Backlog, 10Flow: "Create Flow boards in any location" permission lost on zh.wp - https://phabricator.wikimedia.org/T127693#2053140 (10Mattflaschen) This is probably because the are several things that can cause allowCreation to fail, but it wrongly assumes it was user rights. Fixing the er... [22:02:35] 3Collaboration-Team-Current, 10Flow: Update move permission code - https://phabricator.wikimedia.org/T126389#2053151 (10Mattflaschen) a:3Mattflaschen [22:03:23] 3Collaboration-Team-Current, 10Flow: "Create Flow boards in any location" permission lost on zh.wp - https://phabricator.wikimedia.org/T127693#2053155 (10Mattflaschen) [22:09:51] 6Collaboration-Team-Backlog, 10Flow, 6Language-Engineering, 10Parsoid, and 3 others: Provide an API flag to suppress auto-generated - https://phabricator.wikimedia.org/T113331#2053182 (10Jdforrester-WMF) [22:14:41] matt_flaschen: Ugh, assuming that a failure had a certain reason in the context of Flow on zhwiki sounds familiar [22:14:57] Stephane fixed something like that in October and it led to me discovering the ipblockexempt thing [22:15:09] There the real error message was also suppressed and replaced with something bogus [22:15:19] (Then of course when I got to the real error message, it was in Chinese :D ) [22:16:21] 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#1995020 (10jmatazzoni) Right. Elena is right. The... [22:22:18] RoanKattouw, yeah, I recall, hence error-handling being the first step. I may also fix https://phabricator.wikimedia.org/T126389 while I'm at it. [22:27:25] 6Collaboration-Team-Backlog, 10Notifications, 7Easy, 13Patch-For-Review: Echo congratulations thresholds shouldn't be hard-coded - https://phabricator.wikimedia.org/T127311#2053337 (10Jdforrester-WMF) > These congratulations thresholds should be configurable instead of being hard-coded. Some wikis might wa... [22:27:51] (03CR) 10Jforrester: [C: 04-1] "Subject to discussion on the task; that should be resolved before this is merged." [extensions/Echo] - 10https://gerrit.wikimedia.org/r/272308 (https://phabricator.wikimedia.org/T127311) (owner: 10Haritha28) [22:41:23] 3Collaboration-Team-Current, 10Notifications: Show descriptions for the actions under the "..." menu on notifications - https://phabricator.wikimedia.org/T126738#2022242 (10Mooeypoo) This commit: https://gerrit.wikimedia.org/r/#/c/267399/ should fix / display these descriptions. We seem to be missing the descr... [22:51:48] 6Collaboration-Team-Backlog, 10Flow: Log entries missing from Flow history pages - https://phabricator.wikimedia.org/T127770#2053432 (10Quiddity) [22:54:42] 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#2053453 (10Etonkovidova) @SBisson Please correct #2 "speech... [23:57:14] 6Collaboration-Team-Backlog, 10Flow: On protected Flow pages, remove/replace action links, and/or inform users without sufficient rights to use them - https://phabricator.wikimedia.org/T127774#2053561 (10Quiddity) [23:59:21] 3Collaboration-Team-Current, 10Flow, 10Wikimedia-Site-Requests: Remove Flow from Wikipedia talk:WikiProject Breakfast on enwp - https://phabricator.wikimedia.org/T122961#2053583 (10Quiddity) >>! In T122961#2048214, @Alsee wrote: > My description of the bug was inaccurate because the bugs here themselves crea...