nray (Nick Ray)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Tuesday

  • Clear sailing ahead.

User Details

User Since
Jul 3 2018, 4:57 PM (23 w, 4 d)
Availability
Available
LDAP User
Nray
MediaWiki User
NRay (WMF) [ Global Accounts ]

Recent Activity

Fri, Dec 14

nray moved T205989: Popups: To support testing it should be possible to generate all possible combinations of page previews from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Fri, Dec 14, 8:54 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Audiences-QA, Design, Page-Previews
nray added a comment to T209314: Popups: Upgrade Popups dependencies .

A run of npm outdated tells me:
Package Current Wanted Latest Location
wdio-mocha-framework 0.6.3 0.6.3 0.6.4

Should we bump wdio-mocha-framework the extra step?

Per npm audit, there are 4 "high" vulnerabilities (with 44 "moderate" and 11 "low").

Thanks for that @Shreyasminocha! Annoyingly in production we are using node 6 so we can't benefit from these audits (have to do them manually every now and again). You are right though that these are dev dependencies.

I'm seeing just one issue now that could be resolved with

Run  npm install --save-dev grunt-contrib-watch@1.1.0  to resolve 2 vulnerabilities
Fri, Dec 14, 5:08 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Previews

Thu, Dec 13

nray placed T209314: Popups: Upgrade Popups dependencies up for grabs.
Thu, Dec 13, 10:24 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Previews
nray moved T209314: Popups: Upgrade Popups dependencies from Doing to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Thu, Dec 13, 10:24 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Previews

Wed, Dec 12

nray placed T197883: Popups: Update babel-preset-env and friends up for grabs.
Wed, Dec 12, 10:02 PM · Google-Code-in-2018, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Patch-For-Review, MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Technical-Debt, Page-Previews
nray moved T197883: Popups: Update babel-preset-env and friends from Needs QA to Ready for Signoff on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Wed, Dec 12, 10:02 PM · Google-Code-in-2018, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Patch-For-Review, MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Technical-Debt, Page-Previews
nray edited projects for T197883: Popups: Update babel-preset-env and friends, added: MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018; removed Readers-Web-Backlog.
Wed, Dec 12, 10:01 PM · Google-Code-in-2018, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Patch-For-Review, MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Technical-Debt, Page-Previews
nray edited projects for T197883: Popups: Update babel-preset-env and friends, added: Readers-Web-Backlog; removed MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018.

Tested anon / logged in users, previews with images / no images using https://en.wikipedia.beta.wmflabs.org/wiki/VisualEditor. All appears to be working as intended.

Wed, Dec 12, 9:53 PM · Google-Code-in-2018, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Patch-For-Review, MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Technical-Debt, Page-Previews
nray added a comment to T211724: [Spike] Discuss OO.EventEmitter usage for Views.

@Jdlrobson I really like the more granular pattern you've suggested which, as you said, is similar to how react does it and think it could work really well. We should talk about it at the next super-happy-dev-time. The thing that I didn't like about my approach is that it made each component responsible for knowing about an event bus and how to use it. Your suggestion would resolve that by moving that layer out of components (which I think is a good thing). By doing this, it seems like it would make our components more like presentational components and less like container + presentational components.

Wed, Dec 12, 9:08 PM · Technical-Debt (RW-Tech-Debt), MobileFrontend (MobileFrontend and MinervaNeue architecture), Readers-Web-Backlog

Tue, Dec 11

nray placed T209007: MFA: View: Make isBorderBox and className passable options to the View class to reduce reliance on inheritance up for grabs.
Tue, Dec 11, 10:30 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray moved T209007: MFA: View: Make isBorderBox and className passable options to the View class to reduce reliance on inheritance from Needs Code Review to Needs QA on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Tue, Dec 11, 10:30 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray created T211724: [Spike] Discuss OO.EventEmitter usage for Views.
Tue, Dec 11, 9:28 PM · Technical-Debt (RW-Tech-Debt), MobileFrontend (MobileFrontend and MinervaNeue architecture), Readers-Web-Backlog
nray updated the task description for T156186: MFA: MobileFrontend should not have a global event emitter.
Tue, Dec 11, 7:07 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray claimed T197883: Popups: Update babel-preset-env and friends.
Tue, Dec 11, 6:27 PM · Google-Code-in-2018, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Patch-For-Review, MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Technical-Debt, Page-Previews
nray moved T156186: MFA: MobileFrontend should not have a global event emitter from Needs Code Review to Needs QA on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Tue, Dec 11, 6:20 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)

Mon, Dec 10

nray placed T156186: MFA: MobileFrontend should not have a global event emitter up for grabs.
Mon, Dec 10, 7:10 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray moved T156186: MFA: MobileFrontend should not have a global event emitter from Doing to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Mon, Dec 10, 7:10 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray claimed T156186: MFA: MobileFrontend should not have a global event emitter.
Mon, Dec 10, 6:30 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray moved T156186: MFA: MobileFrontend should not have a global event emitter from Needs More Work to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Mon, Dec 10, 6:29 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray claimed T209314: Popups: Upgrade Popups dependencies .
Mon, Dec 10, 4:29 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Previews
nray moved T209314: Popups: Upgrade Popups dependencies from To Do to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Mon, Dec 10, 4:29 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Previews

Fri, Dec 7

nray moved T156186: MFA: MobileFrontend should not have a global event emitter from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Fri, Dec 7, 8:57 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray moved T208920: MFA: add tests for Drawer and CtaDrawer from Needs Code Review to Needs QA on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Fri, Dec 7, 8:47 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray placed T156186: MFA: MobileFrontend should not have a global event emitter up for grabs.
Fri, Dec 7, 2:47 AM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)

Thu, Dec 6

nray claimed T156186: MFA: MobileFrontend should not have a global event emitter.
Thu, Dec 6, 11:54 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray placed T208920: MFA: add tests for Drawer and CtaDrawer up for grabs.
Thu, Dec 6, 11:53 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray claimed T208920: MFA: add tests for Drawer and CtaDrawer.
Thu, Dec 6, 8:34 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)

Wed, Dec 5

nray placed T156186: MFA: MobileFrontend should not have a global event emitter up for grabs.
Wed, Dec 5, 10:58 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray moved T156186: MFA: MobileFrontend should not have a global event emitter from Doing to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Wed, Dec 5, 10:58 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray claimed T156186: MFA: MobileFrontend should not have a global event emitter.
Wed, Dec 5, 6:52 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray moved T156186: MFA: MobileFrontend should not have a global event emitter from Needs More Work to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Wed, Dec 5, 6:52 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray moved T156186: MFA: MobileFrontend should not have a global event emitter from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Wed, Dec 5, 4:47 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray placed T208920: MFA: add tests for Drawer and CtaDrawer up for grabs.
Wed, Dec 5, 2:18 AM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray moved T208920: MFA: add tests for Drawer and CtaDrawer from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Wed, Dec 5, 2:18 AM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)

Tue, Dec 4

nray added a comment to T156186: MFA: MobileFrontend should not have a global event emitter.

The following patches need review:

Tue, Dec 4, 11:11 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray placed T156186: MFA: MobileFrontend should not have a global event emitter up for grabs.
Tue, Dec 4, 10:49 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray moved T156186: MFA: MobileFrontend should not have a global event emitter from Doing to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Tue, Dec 4, 10:49 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray claimed T208920: MFA: add tests for Drawer and CtaDrawer.
Tue, Dec 4, 6:42 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)

Mon, Dec 3

nray added a project to T210873: A link popup is (partially) outside the browser view: Readers-Web-Backlog.
Mon, Dec 3, 8:26 PM · Readers-Web-Backlog, Page-Previews
nray added a project to T211006: Search results on Mobile web don't include a red link to create a new article: Readers-Web-Backlog.
Mon, Dec 3, 8:25 PM · Advanced Mobile Contributions, Readers-Web-Backlog (Design), Discovery, Editing-team, MobileFrontend

Fri, Nov 30

nray closed T86210: MobileFrontend throws away warnings and other data from action=parse as Resolved.
Fri, Nov 30, 10:17 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, MobileFrontend
nray closed T86210: MobileFrontend throws away warnings and other data from action=parse, a subtask of T109461: [EPIC]: Improvements to mobileview API, as Resolved.
Fri, Nov 30, 10:17 PM · Readers-Web-Backlog, Category, Epic, MobileFrontend
nray closed T86210: MobileFrontend throws away warnings and other data from action=parse, a subtask of T164010: [EPIC] Strengthen the APIs we provide in reading web maintained extensions, as Resolved.
Fri, Nov 30, 10:17 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Epic
nray added a comment to T86210: MobileFrontend throws away warnings and other data from action=parse.

Am now seeing warnings in the output (as intended)

Fri, Nov 30, 10:17 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, MobileFrontend
nray updated the task description for T86210: MobileFrontend throws away warnings and other data from action=parse.
Fri, Nov 30, 10:16 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, MobileFrontend

Thu, Nov 29

nray claimed T86210: MobileFrontend throws away warnings and other data from action=parse.
Thu, Nov 29, 6:05 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, MobileFrontend

Wed, Nov 28

nray closed T206647: Prepare selenium browser test for page issues A/B test as Resolved.
Wed, Nov 28, 9:15 PM · User-Ryasmeen, MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Issue-Warnings
nray added a comment to T206647: Prepare selenium browser test for page issues A/B test.

The page issues tests pass for me locally as shown below. Unfortunately, the tests run on the beta cluster are still flakey (and probably will be for the foreseeable future).

Wed, Nov 28, 9:14 PM · User-Ryasmeen, MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Issue-Warnings
nray awarded T209126: MFA: [Spike, 2hr] Agree on a test approach for Views a Like token.
Wed, Nov 28, 6:35 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Spike
nray added a comment to T209126: MFA: [Spike, 2hr] Agree on a test approach for Views.

The doc looks good to me so far. I'm sure there will be revisions down the road as we continue to port and learn new things

Wed, Nov 28, 6:35 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Spike

Tue, Nov 27

nray added a comment to T206647: Prepare selenium browser test for page issues A/B test.

Good news: I haven't noticed any test failures from pages issues in the last several builds.
Bad news: Other tests are failing and the beta cluster read only errors are making this hard sign off on. I *think* it's working, but there is a lot of noise right now.

Tue, Nov 27, 10:09 PM · User-Ryasmeen, MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Issue-Warnings
nray closed T205008: AMC: wgMFSchemaMainMenuClickTrackingSampleRate should be set in production not MobileFrontend as Resolved.
Tue, Nov 27, 8:27 PM · MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MinervaNeue, goodfirstbug, Technical-Debt, Wikimedia-Site-requests
nray added a comment to T205008: AMC: wgMFSchemaMainMenuClickTrackingSampleRate should be set in production not MobileFrontend.

Tested on https://en.m.wikipedia.org and we are still sending events at what seems like 50% of page views

Tue, Nov 27, 8:27 PM · MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MinervaNeue, goodfirstbug, Technical-Debt, Wikimedia-Site-requests
nray claimed T206647: Prepare selenium browser test for page issues A/B test.
Tue, Nov 27, 6:22 PM · User-Ryasmeen, MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Issue-Warnings
nray claimed T205008: AMC: wgMFSchemaMainMenuClickTrackingSampleRate should be set in production not MobileFrontend.
Tue, Nov 27, 6:22 PM · MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MinervaNeue, goodfirstbug, Technical-Debt, Wikimedia-Site-requests
nray claimed T156186: MFA: MobileFrontend should not have a global event emitter.
Tue, Nov 27, 4:17 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)
nray moved T156186: MFA: MobileFrontend should not have a global event emitter from To Do to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Tue, Nov 27, 4:17 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), MobileFrontend (MobileFrontend and MinervaNeue architecture), Technical-Debt (RW-Tech-Debt)

Mon, Nov 26

nray added a project to T210440: Ignored lang-switch for multilingual SVG Images: Readers-Web-Backlog.
Mon, Nov 26, 11:40 PM · Readers-Web-Backlog, MobileFrontend, Android-app-Bugs, Wikipedia-Android-App-Backlog

Sat, Nov 17

nray moved T209185: Add test coverage for rlModuleLoader.js from Needs More Work to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Sat, Nov 17, 1:31 AM · MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)

Nov 16 2018

nray placed T206887: [Research 4hrs] [Bug] Page issues link overlapping in multiple languages for larger screens up for grabs.
Nov 16 2018, 12:04 AM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Audiences-QA, Patch-For-Review, Spike, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Issue-Warnings
nray moved T206887: [Research 4hrs] [Bug] Page issues link overlapping in multiple languages for larger screens from Needs Code Review to Needs QA on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 16 2018, 12:04 AM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Audiences-QA, Patch-For-Review, Spike, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Page-Issue-Warnings

Nov 15 2018

nray moved T202557: [Bug] Special:MobileOptions heading padding should be consistent from Needs Code Review to Needs QA on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 15 2018, 11:57 PM · MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, goodfirstbug, Mobile, MobileFrontend
nray moved T209185: Add test coverage for rlModuleLoader.js from Doing to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 15 2018, 10:56 PM · MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray claimed T209185: Add test coverage for rlModuleLoader.js.
Nov 15 2018, 4:32 PM · MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray moved T209185: Add test coverage for rlModuleLoader.js from To Do to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 15 2018, 4:31 PM · MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray added a comment to T207805: MFA: Port the references feature to webpack: Merge mobile.references and mobile.references.gateway.

1 big patch merged! Still more AC to do so I'm moving to Needs More Work column

Nov 15 2018, 1:22 AM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray moved T207805: MFA: Port the references feature to webpack: Merge mobile.references and mobile.references.gateway from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 15 2018, 1:21 AM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray moved T207805: MFA: Port the references feature to webpack: Merge mobile.references and mobile.references.gateway from Needs More Work to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 15 2018, 12:06 AM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray moved T202557: [Bug] Special:MobileOptions heading padding should be consistent from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 15 2018, 12:01 AM · MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, goodfirstbug, Mobile, MobileFrontend

Nov 14 2018

nray moved T207805: MFA: Port the references feature to webpack: Merge mobile.references and mobile.references.gateway from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 14 2018, 10:32 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray added a comment to T209007: MFA: View: Make isBorderBox and className passable options to the View class to reduce reliance on inheritance.

Moving this to Needs More work. One patch has been reviewed and merge relating to class name but there is still AC to be met including passing isBorderBox

Nov 14 2018, 10:32 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)
nray moved T209007: MFA: View: Make isBorderBox and className passable options to the View class to reduce reliance on inheritance from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 14 2018, 10:31 PM · MW-1.33-notes (1.33.0-wmf.9; 2018-12-18), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), MobileFrontend (MobileFrontend and MinervaNeue architecture)

Nov 13 2018

nray closed T206698: Increase test coverage for non-View files with 0% coverage as Resolved.
Nov 13 2018, 9:06 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Technical-Debt
nray closed T206698: Increase test coverage for non-View files with 0% coverage, a subtask of T195478: [EPIC] Speed up unit test execution and increase code coverage, as Resolved.
Nov 13 2018, 9:06 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Readers-Web-Kanbanana-Board-Old, User-Jdlrobson, Epic, MobileFrontend, MinervaNeue
nray added a comment to T206698: Increase test coverage for non-View files with 0% coverage.

100% test coverage for icons.js and context.js!

Nov 13 2018, 9:05 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Technical-Debt
nray claimed T206698: Increase test coverage for non-View files with 0% coverage.
Nov 13 2018, 6:34 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Technical-Debt

Nov 9 2018

nray placed T206699: Add test coverage for Button, Panel and Section up for grabs.
Nov 9 2018, 4:32 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2)

Nov 8 2018

nray placed T205359: Download icon is causing invalid HTML up for grabs.
Nov 8 2018, 8:21 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Audiences-QA, Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, goodfirstbug, Accessibility, MinervaNeue
nray moved T205359: Download icon is causing invalid HTML from Needs QA to Ready for Signoff on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 8 2018, 8:21 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Audiences-QA, Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, goodfirstbug, Accessibility, MinervaNeue
nray added a comment to T205359: Download icon is causing invalid HTML.

Lgtm! Tested on chrome desktop and mobile (android), was able to see the spinner, download button worked, and html is valid.

Nov 8 2018, 8:17 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Audiences-QA, Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, goodfirstbug, Accessibility, MinervaNeue
nray updated the task description for T205359: Download icon is causing invalid HTML.
Nov 8 2018, 8:14 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Audiences-QA, Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Google-Code-in-2018, Need-volunteer, goodfirstbug, Accessibility, MinervaNeue

Nov 5 2018

nray updated the task description for T208808: [Spike, 2hrs] Identify the problem with the toast notification tests on the beta cluster.
Nov 5 2018, 11:06 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Patch-For-Review, Spike, MinervaNeue
nray added a project to T208698: What links here on mobileview: Readers-Web-Backlog.
Nov 5 2018, 10:55 PM · Advanced Mobile Contributions, Readers-Web-Backlog (Design), MobileFrontend
nray added a project to T208642: Brave browser: Unreadable text (line breaks at every character) for text next to an info box: Readers-Web-Backlog.
Nov 5 2018, 10:54 PM · Readers-Web-Backlog (Tracking), Mobile, Reading-Web-Local-Wiki-Issues
nray added a project to T208758: Extend Popups-Extension to render popups for annotated <math>: Readers-Web-Backlog.
Nov 5 2018, 10:52 PM · Readers-Web-Backlog (Design), Page-Previews
nray renamed T208808: [Spike, 2hrs] Identify the problem with the toast notification tests on the beta cluster from Fix "Page actions menu when anonymous.I cannot edit a protected page when anonymous" test to Fix toast notification tests.
Nov 5 2018, 10:41 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Patch-For-Review, Spike, MinervaNeue
nray updated the task description for T208808: [Spike, 2hrs] Identify the problem with the toast notification tests on the beta cluster.
Nov 5 2018, 10:16 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Patch-For-Review, Spike, MinervaNeue
nray created T208808: [Spike, 2hrs] Identify the problem with the toast notification tests on the beta cluster.
Nov 5 2018, 10:13 PM · MW-1.33-notes (1.33.0-wmf.8; 2018-12-11), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Patch-For-Review, Spike, MinervaNeue
nray placed T200793: Disable page issues A/B test up for grabs.
Nov 5 2018, 8:59 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2)
nray moved T200793: Disable page issues A/B test from Needs Code Review to Ready for Signoff on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 5 2018, 8:59 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2)
nray added a comment to T200793: Disable page issues A/B test.

Moving this to ready for sign off as it was a simple config change

Nov 5 2018, 8:59 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2)
nray added a comment to T200793: Disable page issues A/B test.

Deployed and am noticing a drop in page issues events

Nov 5 2018, 8:19 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2)

Nov 2 2018

nray added a comment to T208605: Muhogan approach in RelatedArticles and QuickSurveys is error prone.

Mmm interesting.. I wonder if https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MobileFrontend/+/469153/ will take care of this when it gets deployed...

Nov 2 2018, 9:02 PM · Technical-Debt, QuickSurveys, RelatedArticles, MinervaNeue, Readers-Web-Backlog

Nov 1 2018

nray moved T186062: Unused MobileFrontend/Minerva variables loaded on desktop/Vector page views from Needs More Work to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 1 2018, 6:34 PM · Readers-Web-Backlog, MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), MinervaNeue, Performance-Team (Radar), MobileFrontend
nray moved T186062: Unused MobileFrontend/Minerva variables loaded on desktop/Vector page views from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 1 2018, 6:27 PM · Readers-Web-Backlog, MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), MinervaNeue, Performance-Team (Radar), MobileFrontend
nray claimed T206699: Add test coverage for Button, Panel and Section.
Nov 1 2018, 4:56 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2)
nray moved T206699: Add test coverage for Button, Panel and Section from To Do to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Nov 1 2018, 4:56 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2)
nray closed T206178: Feature branch: Remove pageIssues A-B test code as Resolved.
Nov 1 2018, 4:45 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Technical-Debt, Page-Issue-Warnings, Design
nray closed T206178: Feature branch: Remove pageIssues A-B test code, a subtask of T201634: [EPIC] Post A-B test technical debt cleanup, as Resolved.
Nov 1 2018, 4:45 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Epic, Technical-Debt, Page-Issue-Warnings, Design
nray added a comment to T206178: Feature branch: Remove pageIssues A-B test code.

Per sign off instructions, created new task T208514 for merging feature branch to master

Nov 1 2018, 4:39 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2), Technical-Debt, Page-Issue-Warnings, Design