Jdlrobson (Jon Robson)
User

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Nov 6 2014, 11:05 PM (201 w, 6 d)
Availability
Available
LDAP User
Unknown
MediaWiki User
Jdlrobson [ Global Accounts ]

Recent Activity

Today

Jdlrobson moved T203817: Continue webpack + test transition of mobile.startup from Needs Code Review to Ready for Signoff on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Thu, Sep 20, 8:17 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson triaged T205008: wgMFSchemaMainMenuClickTrackingSampleRate should be set in production not MobileFrontend as Low priority.
Thu, Sep 20, 7:57 PM · Technical-Debt, Wikimedia-Site-requests, MobileFrontend
Jdlrobson created T205008: wgMFSchemaMainMenuClickTrackingSampleRate should be set in production not MobileFrontend.
Thu, Sep 20, 7:57 PM · Technical-Debt, Wikimedia-Site-requests, MobileFrontend
Jdlrobson reassigned T204746: It should be possible to opt into new page issues treatment via query string parameter from Jdrewniak to Ryasmeen.

This now is live!
Try it out:
https://reading-web-staging.wmflabs.org/w/index.php?title=Pharmacovigilance&mobileaction=toggle_view_mobile&minerva-issues=b

Thu, Sep 20, 7:52 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Audiences-QA, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MinervaNeue
Jdlrobson updated the task description for T204746: It should be possible to opt into new page issues treatment via query string parameter.
Thu, Sep 20, 7:52 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Audiences-QA, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MinervaNeue
Jdlrobson updated the task description for T204746: It should be possible to opt into new page issues treatment via query string parameter.
Thu, Sep 20, 7:50 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Audiences-QA, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MinervaNeue
Jdlrobson assigned T204144: ReadingDepth sometimes initialises before PageIssues leading to incorrect or missing ReadingDepth events to Ryasmeen.

The code is live on staging
http://reading-web-staging.wmflabs.org/wiki/Pharmacovigilance?debug=true
Config is set to

Thu, Sep 20, 7:49 PM · Audiences-QA, MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Page-Issue-Warnings
Jdlrobson updated the task description for T204144: ReadingDepth sometimes initialises before PageIssues leading to incorrect or missing ReadingDepth events.
Thu, Sep 20, 7:48 PM · Audiences-QA, MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Page-Issue-Warnings
Jdlrobson updated the task description for T204144: ReadingDepth sometimes initialises before PageIssues leading to incorrect or missing ReadingDepth events.
Thu, Sep 20, 7:47 PM · Audiences-QA, MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Page-Issue-Warnings
Jdlrobson added a comment to T204691: MobileFrontend overlay toolbar can have no top border on Android.

#eaecf0 is live on https://reading-web-staging.wmflabs.org/

Thu, Sep 20, 7:45 PM · Browser-Support-Android-Google-Chrome, Readers-Web-Backlog (Design), UI-Standardization, VisualEditor (Current work), Patch-For-Review, MobileFrontend, VisualEditor-MediaWiki-Mobile
Jdlrobson added a comment to T198218: Generate list of most used special pages.

Does mobile domain include users using desktop on mobile? I would like to know the answer to
"which special pages are users using on a mobile phone that they feel the need to use desktop site for?"

Thu, Sep 20, 6:35 PM · Reading-analysis, Readers-Web-Backlog (Tracking), Product-Analytics
Jdlrobson moved T204746: It should be possible to opt into new page issues treatment via query string parameter from Needs Code Review to Needs QA on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Thu, Sep 20, 6:31 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Audiences-QA, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MinervaNeue
Jdlrobson moved T204144: ReadingDepth sometimes initialises before PageIssues leading to incorrect or missing ReadingDepth events from Needs Code Review to Needs QA on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Thu, Sep 20, 6:29 PM · Audiences-QA, MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Page-Issue-Warnings
Jdlrobson added a comment to T204609: Turn on page issues A/B test for Latvian wikipedia .

OK, let's look at the article https://lv.m.wikipedia.org/wiki/Filozofija , which currently is tagged as lacking references and receives between 20-80 pageviews per day.

However, it hasn't generated any PageIssues events so far:

SELECT COUNT(*) FROM event.pageissues 
WHERE year >0 
AND event.pageTitle = 'Filozofija';

_c0
0
1 row selected (58.149 seconds)
Thu, Sep 20, 6:16 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Wikimedia-Site-requests
Jdlrobson added a comment to T204609: Turn on page issues A/B test for Latvian wikipedia .

Interestingly, the top entry (on Pepin the Short) doesn't contain any visible page issues - rather, it seems there is an ambox class coming from this hidden infobox template: https://lv.wikipedia.org/wiki/Veidne:Infokaste%2B

Thu, Sep 20, 6:09 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Wikimedia-Site-requests
Jdlrobson updated subscribers of T204885: MobileFrontend's headless qunit tests need Sinon support.

@Jdrewniak @Niedzielski @nray and myself discussed and estimated this today.

Thu, Sep 20, 5:59 PM · Readers-Web-Backlog
Jdlrobson set the point value for T204885: MobileFrontend's headless qunit tests need Sinon support to 3.
Thu, Sep 20, 5:58 PM · Readers-Web-Backlog
Jdlrobson moved T202306: Remove EventLogging code from RelatedArticles from Needs More Work to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Thu, Sep 20, 5:51 PM · Performance-Team (Radar), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), RelatedArticles
Jdlrobson added a comment to T204609: Turn on page issues A/B test for Latvian wikipedia .

Not sure if this is relevant or not, but ~half the time I load the page I do see the page issue appear, with the treatment that is currently on production. Whereas other times it doesn't present itself at all:

whereas with other pages on the list, e.g. Ropažu sporta centrs, it alternates between the new and current version as expected:

Thu, Sep 20, 5:36 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Wikimedia-Site-requests
Jdlrobson moved T204910: Mobile history shows red-sad faces to logged-out viewers but grey-happy to logged-in from Needs Analysis to Triaged but Future on the Readers-Web-Backlog board.
Thu, Sep 20, 5:35 PM · Need-volunteer, Google-Code-in-2018, Easy, Readers-Web-Backlog, MobileFrontend
Jdlrobson added projects to T204910: Mobile history shows red-sad faces to logged-out viewers but grey-happy to logged-in: Easy, Google-Code-in-2018, Need-volunteer.
Thu, Sep 20, 5:35 PM · Need-volunteer, Google-Code-in-2018, Easy, Readers-Web-Backlog, MobileFrontend
Jdlrobson moved T204910: Mobile history shows red-sad faces to logged-out viewers but grey-happy to logged-in from To Triage to Needs Analysis on the Readers-Web-Backlog board.
Thu, Sep 20, 5:32 PM · Need-volunteer, Google-Code-in-2018, Easy, Readers-Web-Backlog, MobileFrontend
Jdlrobson moved T191987: Simplify CategoryOverlay code for readability from Blocked on Others to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Thu, Sep 20, 5:25 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MobileFrontend (MobileFrontend.js), Technical-Debt
Jdlrobson moved T203817: Continue webpack + test transition of mobile.startup from Needs More Work to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Thu, Sep 20, 4:09 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson added a comment to T173379: Remove mobileview API call for main page.

That wouldn't surprise me. Parsoid has no concept of the mobileformatter and I'm guessing the mobile content service doesn't optimise the page as android do not render the main page at all?

Thu, Sep 20, 4:05 PM · Page Content Service, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
Jdlrobson added a comment to T204143: ReadingDepth events are not being sent in browsers where navigator.sendBeacon should be supported but in practice isn't.

I didn't realise we were excluding all of Safari. That seems a bit extreme imo given we have seen this issue only on 11.1.1 on desktop and we could just exclude that user agent.

Thu, Sep 20, 2:25 PM · Patch-For-Review, Product-Analytics, Reading-analysis, Readers-Web-Backlog

Yesterday

Jdlrobson closed T202743: Add precommit tests to MobileFrontend repositories as Resolved.

Marking this as resolved.
I've submitted a patch for removing the HTML validator. Let's open new tasks as necessary, given precommit hook support now exists.
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MobileFrontend/+/461497 Remove validate HTML dev-script

Wed, Sep 19, 11:57 PM · MW-1.32-release-notes (WMF-deploy-2018-09-18 (1.32.0-wmf.22)), Patch-For-Review, MobileFrontend (MobileFrontend.js), Readers-Web-Backlog, User-Jdlrobson, MinervaNeue
Jdlrobson closed T202743: Add precommit tests to MobileFrontend repositories, a subtask of T195475: [EPIC] Automate asset bundling in MobileFrontend/Minerva, as Resolved.
Wed, Sep 19, 11:57 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Readers-Web-Kanbanana-Board-Old, User-Jdlrobson, Epic, MobileFrontend, MinervaNeue
Jdlrobson added a project to T188451: Migrate mobile editor to CSRF from edit token in TalkSectionAddOverlay: Google-Code-in-2018.
Wed, Sep 19, 11:53 PM · Google-Code-in-2018, MobileFrontend (MobileFrontend.js), Easy, Technical-Debt, Readers-Web-Backlog
Jdlrobson added a project to T204876: [Bug - development NOT production] Nearby TypeError when using JSONPForeignApi: Google-Code-in-2018.
Wed, Sep 19, 11:53 PM · Google-Code-in-2018, Need-volunteer, Easy, MobileFrontend, Readers-Web-Backlog
Jdlrobson added a project to T199351: [Bug] Lazily loaded inline images cause reflow - should retain thumbborder: Google-Code-in-2018.
Wed, Sep 19, 11:52 PM · Google-Code-in-2018, MobileFrontend, Readers-Web-Backlog
Jdlrobson added a project to T165034: MobileFormatter uses hardcoded "mw-parser-output" class which is configurable via ParserOptions: Google-Code-in-2018.
Wed, Sep 19, 11:52 PM · Google-Code-in-2018, MobileFrontend, Readers-Web-Backlog
Jdlrobson added a project to T197883: Update babel-preset-env and friends: Google-Code-in-2018.
Wed, Sep 19, 11:51 PM · Google-Code-in-2018, Technical-Debt, Page-Previews, Readers-Web-Backlog
Jdlrobson added a project to T198265: Special:MobileOptions is empty for third parties: Google-Code-in-2018.
Wed, Sep 19, 11:51 PM · Google-Code-in-2018, Reading-Web-Third-Party-Support, MobileFrontend, Readers-Web-Backlog, Mobile-Web-Settings
Jdlrobson updated the task description for T203818: Add code coverage testing and increase test coverage for some existing mobile.startup files.
Wed, Sep 19, 11:40 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson updated the task description for T203589: SWAT $wgPopupsBetaFeature reading web config cleanup.
Wed, Sep 19, 11:36 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, Patch-For-Review, Wikimedia-Site-requests
Jdlrobson closed T203589: SWAT $wgPopupsBetaFeature reading web config cleanup as Resolved.

This can skip QA and design review as it's removing dead code.

Wed, Sep 19, 11:36 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, Patch-For-Review, Wikimedia-Site-requests
Jdlrobson moved T203589: SWAT $wgPopupsBetaFeature reading web config cleanup from Needs Code Review to Ready for Signoff on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 11:36 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, Patch-For-Review, Wikimedia-Site-requests
Jdlrobson assigned T204609: Turn on page issues A/B test for Latvian wikipedia to Tbayer.

I'm seeing 0.015 events per second. Little impact on ReadingDepth.
@Tbayer can you look at the data coming in and see if there's anything that's not expected?

Wed, Sep 19, 11:35 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Wikimedia-Site-requests
Jdlrobson moved T204609: Turn on page issues A/B test for Latvian wikipedia from Ready for Signoff to Needs QA on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 11:33 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Wikimedia-Site-requests
Jdlrobson moved T204609: Turn on page issues A/B test for Latvian wikipedia from Needs Code Review to Ready for Signoff on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 11:33 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Wikimedia-Site-requests
Jdlrobson added a comment to T198264: Summary endpoint cannot handle brackets with whitespace inside.

I keep hitting this one in the wild, a constant reminder this bug is still open. Here's a recent example I saw today:


It would be great to address this sooner rather than later.

Wed, Sep 19, 11:21 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service, RESTBase-API
Jdlrobson added a comment to T197884: Respect Readers Web coding convention for test filenames.

This looks done and I think this can skip QA. I'll let @Jdrewniak confirm and assign me and I'm happy to sign off when that's the case.

Wed, Sep 19, 10:47 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Patch-For-Review, MobileFrontend (MobileFrontend.js), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt
Jdlrobson updated the task description for T197884: Respect Readers Web coding convention for test filenames.
Wed, Sep 19, 10:47 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Patch-For-Review, MobileFrontend (MobileFrontend.js), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt
Jdlrobson moved T197884: Respect Readers Web coding convention for test filenames from Doing to Ready for Signoff on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 10:46 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Patch-For-Review, MobileFrontend (MobileFrontend.js), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt
Jdlrobson added a comment to T204094: Minerva taint error: Calling method \BaseTemplate::set() in \SkinMinerva::prepareHeaderAndFooter that outputs using tainted argument.

Does a workaround exist? I'm hitting this a few times today.

Wed, Sep 19, 10:46 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Continuous-Integration-Config, MinervaNeue, phan-taint-check-plugin
Jdlrobson updated the task description for T197884: Respect Readers Web coding convention for test filenames.
Wed, Sep 19, 10:42 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Patch-For-Review, MobileFrontend (MobileFrontend.js), Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt
Jdlrobson added a comment to T203818: Add code coverage testing and increase test coverage for some existing mobile.startup files.

(the todo column was empty, I believe we informally agreed that it should never be empty)

Wed, Sep 19, 10:41 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson added a comment to T204839: Page change language icon is useless in disabled state.

That's not added by Minerva and doesn't show for me. I believe that's the gadget - "Add map popups to coordinates in the mobile website" so nothing readers web design had anything to do with, thus probably not a good comparison with the language button which we spent a whole quarter thinking about, designing and testing.

Wed, Sep 19, 10:40 PM · Readers-Web-Backlog (Design), MinervaNeue
Jdlrobson moved T203818: Add code coverage testing and increase test coverage for some existing mobile.startup files from Upcoming to Readers-Web-Kanbanana-Board-2018-19-Q1 on the Readers-Web-Backlog board.
Wed, Sep 19, 10:32 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson moved T191987: Simplify CategoryOverlay code for readability from Doing to Blocked on Others on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 10:24 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MobileFrontend (MobileFrontend.js), Technical-Debt
Jdlrobson moved T204746: It should be possible to opt into new page issues treatment via query string parameter from Doing to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 10:23 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Audiences-QA, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MinervaNeue
Jdlrobson updated the task description for T204746: It should be possible to opt into new page issues treatment via query string parameter.
Wed, Sep 19, 10:23 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Audiences-QA, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MinervaNeue
Jdlrobson moved T204746: It should be possible to opt into new page issues treatment via query string parameter from To Do to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 10:18 PM · MW-1.32-release-notes (WMF-deploy-2018-09-25 (1.32.0-wmf.23)), Audiences-QA, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MinervaNeue
Jdlrobson added a comment to T203814: Turn on MinervaErrorLogSamplingRate (Schema:WebClientError).

@Nuria so I completely agree with everything your saying.

Wed, Sep 19, 10:14 PM · Patch-For-Review, Wikimedia-Site-requests, Analytics, MobileFrontend (MobileFrontend.js), Readers-Web-Backlog
Jdlrobson updated subscribers of T204839: Page change language icon is useless in disabled state.

@Jdlrobson, should we always show the map button in a disabled state for pages without maps?

Wed, Sep 19, 10:04 PM · Readers-Web-Backlog (Design), MinervaNeue
Jdlrobson added a comment to T198963: Add "Did you know..." and "On this day" sections from desktop main page to mobile main page.

I don't have anything to add to what @Krinkle says... that's my understanding too.

Wed, Sep 19, 10:03 PM · Readers-Web-Backlog (Tracking), MobileFrontend, Mobile, SEO
Jdlrobson updated the task description for T204885: MobileFrontend's headless qunit tests need Sinon support.
Wed, Sep 19, 9:51 PM · Readers-Web-Backlog
Jdlrobson moved T204627: Double pokey on some page preview pop-ups from Needs Analysis to Needs Reproduction on the Readers-Web-Backlog board.

I still can't replicate this (even now trying Safari), can you please provide more information per https://phabricator.wikimedia.org/T204627#4596374 as maybe this is due to my browser/operating system combo?

Wed, Sep 19, 9:49 PM · Audiences-QA, Readers-Web-Backlog, Design, Page-Previews
Jdlrobson lowered the priority of T204792: Special:MobileLanguages emits warning "url key is undefined in language object" from High to Normal.

These logs seem to be intentional and were added in 456264d8 by @pmiazga
I'm guessing they should they go to a different logger channel? https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/includes/specials/SpecialMobileLanguages.php#L95

Wed, Sep 19, 9:46 PM · Readers-Web-Backlog, MobileFrontend, Wikimedia-production-error
Jdlrobson moved T204876: [Bug - development NOT production] Nearby TypeError when using JSONPForeignApi from Needs Analysis to Triaged but Future on the Readers-Web-Backlog board.
Wed, Sep 19, 9:42 PM · Google-Code-in-2018, Need-volunteer, Easy, MobileFrontend, Readers-Web-Backlog
Jdlrobson added projects to T204876: [Bug - development NOT production] Nearby TypeError when using JSONPForeignApi: Easy, Need-volunteer.
Wed, Sep 19, 9:42 PM · Google-Code-in-2018, Need-volunteer, Easy, MobileFrontend, Readers-Web-Backlog
Krinkle awarded T204885: MobileFrontend's headless qunit tests need Sinon support a Orange Medal token.
Wed, Sep 19, 9:38 PM · Readers-Web-Backlog
Jdlrobson renamed T204876: [Bug - development NOT production] Nearby TypeError when using JSONPForeignApi from [Bug] Nearby TypeError when using JSONPForeignApi to [Bug - development NOT production] Nearby TypeError when using JSONPForeignApi.
Wed, Sep 19, 9:36 PM · Google-Code-in-2018, Need-volunteer, Easy, MobileFrontend, Readers-Web-Backlog
Jdlrobson moved T204876: [Bug - development NOT production] Nearby TypeError when using JSONPForeignApi from Upcoming to Needs Analysis on the Readers-Web-Backlog board.
Wed, Sep 19, 9:33 PM · Google-Code-in-2018, Need-volunteer, Easy, MobileFrontend, Readers-Web-Backlog
Jdlrobson triaged T204885: MobileFrontend's headless qunit tests need Sinon support as High priority.
Wed, Sep 19, 9:33 PM · Readers-Web-Backlog
Jdlrobson moved T204885: MobileFrontend's headless qunit tests need Sinon support from Needs Analysis to Upcoming on the Readers-Web-Backlog board.
Wed, Sep 19, 9:33 PM · Readers-Web-Backlog
Jdlrobson edited projects for T204698: cloudvps: wikidata-page-banner project trusty deprecation, added: Readers-Web-Backlog (Tracking); removed Readers-Web-Backlog.

@aborrero if we don't hear from Sumit, i suggest we remove this instance, however I'm not sure what the protocol and grace period is for doing so.

Wed, Sep 19, 9:32 PM · Readers-Web-Backlog (Tracking), Wikidata, Wikidata-Page-Banner, Cloud-VPS
Jdlrobson moved T204698: cloudvps: wikidata-page-banner project trusty deprecation from To Triage to Needs Analysis on the Readers-Web-Backlog board.
Wed, Sep 19, 9:32 PM · Readers-Web-Backlog (Tracking), Wikidata, Wikidata-Page-Banner, Cloud-VPS
Jdlrobson created T204885: MobileFrontend's headless qunit tests need Sinon support.
Wed, Sep 19, 9:32 PM · Readers-Web-Backlog
Jdlrobson added a comment to T204691: MobileFrontend overlay toolbar can have no top border on Android.

Okay, some wires got crossed then :). I thought @Nirzar had approved the black color... :)

Wed, Sep 19, 9:10 PM · Browser-Support-Android-Google-Chrome, Readers-Web-Backlog (Design), UI-Standardization, VisualEditor (Current work), Patch-For-Review, MobileFrontend, VisualEditor-MediaWiki-Mobile
Jdlrobson moved T203817: Continue webpack + test transition of mobile.startup from Doing to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 9:06 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson moved T203817: Continue webpack + test transition of mobile.startup from Needs Code Review to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 8:23 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson added a project to T204867: Webpack + test transition of mobile.startup classes and instances (Button, Icon, icons, Panel, Section, Thumbnail, Page): MobileFrontend (MobileFrontend.js).
Wed, Sep 19, 7:14 PM · MobileFrontend (MobileFrontend.js), Readers-Web-Backlog
Jdlrobson created T204867: Webpack + test transition of mobile.startup classes and instances (Button, Icon, icons, Panel, Section, Thumbnail, Page).
Wed, Sep 19, 7:14 PM · MobileFrontend (MobileFrontend.js), Readers-Web-Backlog
Jdlrobson triaged T204866: Deprecate mobile.startup/user as Low priority.
Wed, Sep 19, 7:10 PM · MobileFrontend (MobileFrontend.js), Readers-Web-Backlog
Jdlrobson edited projects for T204866: Deprecate mobile.startup/user, added: MobileFrontend (MobileFrontend.js); removed MobileFrontend.
Wed, Sep 19, 7:09 PM · MobileFrontend (MobileFrontend.js), Readers-Web-Backlog
Jdlrobson created T204866: Deprecate mobile.startup/user.
Wed, Sep 19, 7:09 PM · MobileFrontend (MobileFrontend.js), Readers-Web-Backlog
Jdlrobson moved T203817: Continue webpack + test transition of mobile.startup from Doing to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 6:41 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson claimed T203817: Continue webpack + test transition of mobile.startup.
Wed, Sep 19, 6:27 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson moved T203100: Continue webpack + test transition of mobile.startup from Doing to Ready for Signoff on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.

I chatted with @nray after standup and we feel like the remaining work is captured in T204786, which we will discuss on Monday as a group (and then later estimate/tackle as deemed necessary).

Wed, Sep 19, 6:22 PM · MW-1.32-release-notes (WMF-deploy-2018-09-18 (1.32.0-wmf.22)), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MobileFrontend (MobileFrontend.js)
Jdlrobson updated the task description for T203100: Continue webpack + test transition of mobile.startup.
Wed, Sep 19, 6:21 PM · MW-1.32-release-notes (WMF-deploy-2018-09-18 (1.32.0-wmf.22)), Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), MobileFrontend (MobileFrontend.js)
Jdlrobson added a comment to T204088: exported puppet resources are not queryable: cannot create grafana graphs of EventLogging running in beta cluster.

I'm guessing this is not something trivial we can fix?
Is there another task I should be following relating to this?

Wed, Sep 19, 6:18 PM · Beta-Cluster-Infrastructure, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Operations, Puppet
Jdlrobson lowered the priority of T204606: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display from Normal to Low.

Since there is only one page with timeout issues in the whole of our projects I'm marking this as low.
All solutions seem kinda risky right now.

Wed, Sep 19, 6:15 PM · Wikimedia-production-error, MobileFrontend, Readers-Web-Backlog
Jdlrobson updated the task description for T204606: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display.
Wed, Sep 19, 6:15 PM · Wikimedia-production-error, MobileFrontend, Readers-Web-Backlog
Jdlrobson renamed T204606: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display from Disable LazyImageTransform on user namespace: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display to Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display.
Wed, Sep 19, 6:13 PM · Wikimedia-production-error, MobileFrontend, Readers-Web-Backlog
Jdlrobson moved T203817: Continue webpack + test transition of mobile.startup from To Do to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 6:03 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, MobileFrontend (MobileFrontend.js), User-Jdlrobson, MinervaNeue
Jdlrobson moved T204606: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display from Triaged but Future to Needs Analysis on the Readers-Web-Backlog board.
Wed, Sep 19, 5:59 PM · Wikimedia-production-error, MobileFrontend, Readers-Web-Backlog
Jdlrobson updated subscribers of T204691: MobileFrontend overlay toolbar can have no top border on Android.

but that changes the colour of the search bar as well as the status bar, which seems a bit extreme.

It doesn't seem more extreme to me than adding several lines of CSS that apply everywhere except iOS. The fact the status bar is no longer black (as you mentioned it was before) seems like a regression.

Wed, Sep 19, 5:55 PM · Browser-Support-Android-Google-Chrome, Readers-Web-Backlog (Design), UI-Standardization, VisualEditor (Current work), Patch-For-Review, MobileFrontend, VisualEditor-MediaWiki-Mobile
Jdlrobson renamed T204606: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display from Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display to Disable LazyImageTransform on user namespace: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display.
Wed, Sep 19, 5:41 PM · Wikimedia-production-error, MobileFrontend, Readers-Web-Backlog
Jdlrobson moved T204606: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display from Needs Analysis to Triaged but Future on the Readers-Web-Backlog board.

The easiest thing to do here is just turn off the transform on the talk namespace and then re-evaluate. I'm guessing this is not going to be a problem on real world articles as they'd be too slow to load and would be unusable.

Wed, Sep 19, 5:41 PM · Wikimedia-production-error, MobileFrontend, Readers-Web-Backlog
Jdlrobson updated the task description for T204606: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display.
Wed, Sep 19, 5:40 PM · Wikimedia-production-error, MobileFrontend, Readers-Web-Backlog
Jdlrobson renamed T204606: Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display from Some requests takes over 60 seconds to be processed by MobileFormatter, thus will not display to Some requests takes over 60 seconds to be processed by MobileFormatter when page has lots of images thus will not display.
Wed, Sep 19, 5:39 PM · Wikimedia-production-error, MobileFrontend, Readers-Web-Backlog
Jdlrobson updated the task description for T181545: "Other review tools" is not working when using Minerva on Recent Changes or JS is disabled on desktop.
Wed, Sep 19, 5:32 PM · Growth-Team, Patch-For-Review, Collaboration-Team-Triage (Collab-Team-This-Quarter), Edit-Review-Improvements-Integrated-Filters
Jdlrobson merged T204851: Special:RecentChanges “Other review tools“ JS disabled in MobileFrontend into T181545: "Other review tools" is not working when using Minerva on Recent Changes or JS is disabled on desktop.
Wed, Sep 19, 5:32 PM · Growth-Team, Patch-For-Review, Collaboration-Team-Triage (Collab-Team-This-Quarter), Edit-Review-Improvements-Integrated-Filters
Jdlrobson merged task T204851: Special:RecentChanges “Other review tools“ JS disabled in MobileFrontend into T181545: "Other review tools" is not working when using Minerva on Recent Changes or JS is disabled on desktop.
Wed, Sep 19, 5:32 PM · Growth-Team, MobileFrontend, Edit-Review-Improvements-RC-Page
Jdlrobson added a comment to T195494: Handle mobile domains in core.

Nope, an RFC has not been drafted. I'm happy to be a coauthor of an RFC, but given I'm primarily a frontend engineer, I would like some help with the technical PHP side of things and/or support from someone who has experience in writing RFCs.

Wed, Sep 19, 4:56 PM · Readers-Web-Backlog (Tracking), User-Jdlrobson, MediaWiki-General-or-Unknown, MobileFrontend
Jdlrobson added a comment to T204609: Turn on page issues A/B test for Latvian wikipedia .

Swatting at 4pm PST

Wed, Sep 19, 4:50 PM · Patch-For-Review, Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Wikimedia-Site-requests
Jdlrobson moved T203589: SWAT $wgPopupsBetaFeature reading web config cleanup from To Do to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1) board.
Wed, Sep 19, 4:49 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, Patch-For-Review, Wikimedia-Site-requests
Jdlrobson edited projects for T203589: SWAT $wgPopupsBetaFeature reading web config cleanup, added: Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1); removed Readers-Web-Backlog.

Swatting at 4pm PST

Wed, Sep 19, 4:49 PM · Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q1), Technical-Debt, Patch-For-Review, Wikimedia-Site-requests