Jdlrobson (Jon Robson)
User

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Sunday

  • Clear sailing ahead.

User Details

User Since
Nov 6 2014, 11:05 PM (150 w, 12 h)
Availability
Available
LDAP User
Unknown
MediaWiki User
Jdlrobson

Recent Activity

Fri, Sep 8

Jdlrobson removed a project from T172902: Display short url version of links on desktop and PDF print styles: Patch-For-Review.

@ovasileva i thought we are stalling this as this is scope creep to print work

@Nirzar See https://phabricator.wikimedia.org/T172902#3589643. The patch is just exploratory work to see how we might do this - a proof of concept. It has no chance of getting merged or going any further without us committing time.

Fri, Sep 8, 9:49 PM · Proton, MediaWiki-extensions-ShortUrl, Vector, Readers-Web-Backlog
Jdlrobson added a comment to T138719: MediaWiki should have a service for short URLs.

Any ideas for how to do this @Legoktm? We've been brainstorming ways over in the phabricator ticket T172902 for the print mode and it would be good to get some input on how best to do this (the patch you just reviewed was a strawman proposal to show one way it might be done). I like the idea of something more generic but I'm not sure what that would look like.

Fri, Sep 8, 7:48 PM · MediaWiki-extensions-UrlShortener, MediaWiki-General-or-Unknown
Jdlrobson updated the task description for T172902: Display short url version of links on desktop and PDF print styles.
Fri, Sep 8, 7:47 PM · Proton, MediaWiki-extensions-ShortUrl, Vector, Readers-Web-Backlog
Jdlrobson updated the task description for T175063: Undefined notices for Collection extension.
Fri, Sep 8, 7:42 PM · Unplanned-Sprint-Work, Readers-Web-Kanban-Board, MW-1.30-release-notes (WMF-deploy-2017-09-19 (1.30.0-wmf.19)), Collection, Readers-Web-Backlog, Wikimedia-log-errors
Jdlrobson updated subscribers of T153125: Hide last modifier bar on revision pages.

issues have been addressed! @pmiazga is on it!

Fri, Sep 8, 7:10 PM · Readers-Web-Kanban-Board, Need-volunteer, Easy, MinervaNeue, Design, Readers-Web-Backlog
Jdlrobson reassigned T174598: Reading web extensions are incompatible with QUnit 2 from Jdlrobson to bmansurov.

@bmansurov on irc said he can have a go at signing this off

Fri, Sep 8, 7:05 PM · Readers-Web-Kanban-Board, Unplanned-Sprint-Work, MW-1.30-release-notes (WMF-deploy-2017-09-05 (1.30.0-wmf.17)), Readers-Web-Backlog, MobileFrontend
Jdlrobson closed T173600: Improve treatment of hatnotes, a subtask of T143535: [EPIC] Improve article notes , as Resolved.
Fri, Sep 8, 7:04 PM · Readers-Web-Backlog (Design), Epic, Reading-Community-Engagement, Design
Jdlrobson closed T173600: Improve treatment of hatnotes as Resolved.

Per https://phabricator.wikimedia.org/T173600#3585910 and https://phabricator.wikimedia.org/T173600#3592479

Fri, Sep 8, 7:04 PM · Readers-Web-Kanban-Board, Design (RW-Design-Debt), Readers-Web-Backlog
Jdlrobson moved T153125: Hide last modifier bar on revision pages from Needs More Work to Needs Code Review on the Readers-Web-Kanban-Board board.
Fri, Sep 8, 7:04 PM · Readers-Web-Kanban-Board, Need-volunteer, Easy, MinervaNeue, Design, Readers-Web-Backlog
Jdlrobson removed a project from T174598: Reading web extensions are incompatible with QUnit 2: Patch-For-Review.
Fri, Sep 8, 6:46 PM · Readers-Web-Kanban-Board, Unplanned-Sprint-Work, MW-1.30-release-notes (WMF-deploy-2017-09-05 (1.30.0-wmf.17)), Readers-Web-Backlog, MobileFrontend
Jdlrobson updated the task description for T174598: Reading web extensions are incompatible with QUnit 2.
Fri, Sep 8, 6:46 PM · Readers-Web-Kanban-Board, Unplanned-Sprint-Work, MW-1.30-release-notes (WMF-deploy-2017-09-05 (1.30.0-wmf.17)), Readers-Web-Backlog, MobileFrontend
Jdlrobson added a comment to T174598: Reading web extensions are incompatible with QUnit 2.

https://gerrit.wikimedia.org/r/374908 has a +1.
I am going on vacation later today.

Fri, Sep 8, 5:21 PM · Readers-Web-Kanban-Board, Unplanned-Sprint-Work, MW-1.30-release-notes (WMF-deploy-2017-09-05 (1.30.0-wmf.17)), Readers-Web-Backlog, MobileFrontend
Jdlrobson moved T175008: Justified text appearing inconsistent due to large infobox when printing with new styles in firefox from Needs Code Review to Doing on the Readers-Web-Kanban-Board board.
Fri, Sep 8, 5:20 PM · MW-1.30-release-notes (WMF-deploy-2017-09-19 (1.30.0-wmf.19)), Readers-Web-Kanban-Board, Patch-For-Review, Proton, Readers-Web-Backlog
Jdlrobson moved T174957: Infobox breaking toc in new print styles from Needs Code Review to Doing on the Readers-Web-Kanban-Board board.
Fri, Sep 8, 5:20 PM · Vector, Readers-Web-Kanban-Board, Patch-For-Review, Proton, Readers-Web-Backlog
Jdlrobson removed a project from T169074: Remove the mfLazyLoadReferences and mfLazyLoadImages cookie: Patch-For-Review.
Fri, Sep 8, 5:19 PM · Readers-Web-Kanban-Board, Technical-Debt (RW-Tech-Debt), Readers-Web-Backlog, Easy, MobileFrontend
Jdlrobson created T175381: Remove references to non-existent mfLazyLoadReferences cookies.
Fri, Sep 8, 5:19 PM · Readers-Web-Backlog (Tracking), Operations, Puppet
Jdlrobson updated the task description for T166714: Code hygiene ketchup (tracking).
Fri, Sep 8, 5:09 PM · Category, MW-1.30-release-notes (WMF-deploy-2017-08-08_(1.30.0-wmf.13)), Tracking, Readers-Web-Kanban-Board
Jdlrobson moved T169074: Remove the mfLazyLoadReferences and mfLazyLoadImages cookie from Needs Code Review to Ready for Signoff on the Readers-Web-Kanban-Board board.
Fri, Sep 8, 5:00 PM · Readers-Web-Kanban-Board, Technical-Debt (RW-Tech-Debt), Readers-Web-Backlog, Easy, MobileFrontend
Jdlrobson updated the task description for T175286: Do side by side comparison of old summary endpoint against new summary endpoint.
Fri, Sep 8, 2:41 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Reading Epics (Page Previews), Services (watching), Page-Previews
Jdlrobson moved T30063: Collection extension should have an option "move articles and chapters together" on [[Special:Book]] from Backlog to Later on the Readers-Web-Backlog (Tracking) board.
Fri, Sep 8, 2:37 PM · Readers-Web-Backlog (Tracking), Design, Collection
Jdlrobson moved T175286: Do side by side comparison of old summary endpoint against new summary endpoint from Backlog to Watching on the Readers-Web-Backlog (Tracking) board.
Fri, Sep 8, 2:37 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Reading Epics (Page Previews), Services (watching), Page-Previews
Jdlrobson added a comment to T162910: Update Collection not to use deprecated wfSetupSession call.

Interesting. I didn't realise we didn't allow for sessions for anonymous users. I'm curious how this works with Varnish now.

Fri, Sep 8, 2:11 PM · Readers-Web-Backlog (Tracking), Collection, Technical-Debt, Wikimedia-log-errors
Jdlrobson reopened T175368: Create personal user board User-Jdlrobson as "Open".
Fri, Sep 8, 2:09 PM · Project-Admins
Jdlrobson reopened T175368: Create personal user board User-Jdlrobson, a subtask of T555: Per-user projects for personal work in progress tracking, as Open.
Fri, Sep 8, 2:09 PM · Upstream, Phabricator (Upstream)
Jdlrobson created T175368: Create personal user board User-Jdlrobson.
Fri, Sep 8, 2:01 PM · Project-Admins
Jdlrobson moved T174957: Infobox breaking toc in new print styles from To Do to Needs Code Review on the Readers-Web-Kanban-Board board.
Fri, Sep 8, 1:58 PM · Vector, Readers-Web-Kanban-Board, Patch-For-Review, Proton, Readers-Web-Backlog
Jdlrobson moved T175008: Justified text appearing inconsistent due to large infobox when printing with new styles in firefox from To Do to Needs Code Review on the Readers-Web-Kanban-Board board.

Why has this and T174957 been moved to do? The work has been done (so this probably should be code review), but I'm reluctant to review this until https://phabricator.wikimedia.org/T174957#3585819) has an answer. @MBinder_WMF how would we capture this on a kanban board?

Fri, Sep 8, 1:57 PM · MW-1.30-release-notes (WMF-deploy-2017-09-19 (1.30.0-wmf.19)), Readers-Web-Kanban-Board, Patch-For-Review, Proton, Readers-Web-Backlog
Jdlrobson added a comment to T175092: Determine if .navboxes should be stripped from the page content delivered by the Mobile Content Service and Page Content Service.

> We are building a new service that will probably be serving all the HTML for all our clients for the foreseeable future…we don't want to have to make breaking changes in 2 years to support something we never took a few moments to discus on Phabricator early on in the process.

Fri, Sep 8, 1:01 PM · Page Content Service, Mobile-Content-Service, Reading-Infrastructure-Team-Backlog (Kanban)
Jdlrobson renamed T175092: Determine if .navboxes should be stripped from the page content delivered by the Mobile Content Service and Page Content Service from Determine if large tables should be stripped from the page content delivered by the Mobile Content Service and Page Content Service to Determine if .navboxes should be stripped from the page content delivered by the Mobile Content Service and Page Content Service.
Fri, Sep 8, 12:50 PM · Page Content Service, Mobile-Content-Service, Reading-Infrastructure-Team-Backlog (Kanban)
Jdlrobson added a comment to T172291: Launch page previews A/B test on enwiki and dewiki.

@phuedx .... would it make sense to resolve this and move T172322 into blocked?

Fri, Sep 8, 12:50 PM · Patch-For-Review, Readers-Web-Kanban-Board, Page-Previews, Readers-Web-Backlog
Jdlrobson renamed T175005: Articles with tables larger than page will shrink aspect ratio and adjust font-size from [6 Hours] New print styles displaying articles with varying font size to Articles with tables larger than page will shrink aspect ratio and adjust font-size.
Fri, Sep 8, 12:11 PM · Readers-Web-Backlog (Design), Proton

Thu, Sep 7

Jdlrobson added a comment to T163472: [EPIC] Provide a way to download articles in PDF on the mobile website.

can we generate it and show the size and then let the person download? that was meant to be the flow.

Thu, Sep 7, 10:40 PM · Readers-Web-Backlog, Proton, Epic, New-Readers
Jdlrobson updated subscribers of T175005: Articles with tables larger than page will shrink aspect ratio and adjust font-size.

I still say we keep it in tracking or backlog.

What is the goal at doing this?
See also https://phabricator.wikimedia.org/T174955#3588362 from @phuedx
Phabricator is not a great place for tracking things that cannot be fixed or we don't know how to fix. It makes it harder to actually find things that should be worked on.

Thu, Sep 7, 10:37 PM · Readers-Web-Backlog (Design), Proton
Jdlrobson added a comment to T175064: RelatedArticles not displaying on desktop wikivoyage: Cannot read property 'getBucket' of undefined console error.

Change should roll out in 1.30.0-wmf.18 which I think will be live on 13th.

Thu, Sep 7, 9:58 PM · Unplanned-Sprint-Work, Readers-Web-Kanban-Board, Readers-Web-Backlog, RelatedArticles
Jdlrobson added a comment to T171833: Deploy new book renderer to all projects side by side with OCG.

We can't do the SWAT deploy until T173018 is done (otherwise we'll be creating a broken experience) so I've merged these to get them done together.

Thu, Sep 7, 8:44 PM · Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson merged T171833: Deploy new book renderer to all projects side by side with OCG into T173018: Add an option in Special:Book to download PDFs generated by ElectronPdfService.
Thu, Sep 7, 8:43 PM · Proton, Patch-For-Review, Unplanned-Sprint-Work, Collection, Readers-Web-Backlog
Jdlrobson merged task T171833: Deploy new book renderer to all projects side by side with OCG into T173018: Add an option in Special:Book to download PDFs generated by ElectronPdfService.
Thu, Sep 7, 8:43 PM · Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson updated the task description for T173018: Add an option in Special:Book to download PDFs generated by ElectronPdfService.
Thu, Sep 7, 8:43 PM · Proton, Patch-For-Review, Unplanned-Sprint-Work, Collection, Readers-Web-Backlog
Jdlrobson moved T172902: Display short url version of links on desktop and PDF print styles from Needs Analysis to Upcoming on the Readers-Web-Backlog board.

^ the above patch implements one of the proposals. Would be good to talk about it vs other proposals in next grooming session.

Thu, Sep 7, 7:56 PM · Proton, MediaWiki-extensions-ShortUrl, Vector, Readers-Web-Backlog
Jdlrobson updated the task description for T172902: Display short url version of links on desktop and PDF print styles.
Thu, Sep 7, 7:55 PM · Proton, MediaWiki-extensions-ShortUrl, Vector, Readers-Web-Backlog
Jdlrobson updated the task description for T172902: Display short url version of links on desktop and PDF print styles.
Thu, Sep 7, 7:40 PM · Proton, MediaWiki-extensions-ShortUrl, Vector, Readers-Web-Backlog
Jdlrobson changed the status of T171836: Apply new print styles for books from Stalled to Open.

I assume this is a precursor to T169732 (ie. the same but only for beta cluster)? If so this doesn't seem stalled. There should be no reason not to enable them on beta cluster... although I'm not sure how useful this is. There are very few articles on the beta cluster which render nicely and we're testing on live data on https://reading-web-staging.wmflabs.org.

Thu, Sep 7, 7:23 PM · Wikimedia-Site-requests, Proton, Readers-Web-Backlog
Jdlrobson changed the status of T171836: Apply new print styles for books, a subtask of T171833: Deploy new book renderer to all projects side by side with OCG, from Stalled to Open.
Thu, Sep 7, 7:23 PM · Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson merged T171956: Allow PDF rendering to be triggered only by electron for single-article PDFs into T165954: Allow electron to render single PDF's on all projects.
Thu, Sep 7, 7:16 PM · Wikimedia-Site-requests, Patch-For-Review, Readers-Web-Kanban-Board, Electron-PDFs, Readers-Web-Backlog
Jdlrobson merged task T171956: Allow PDF rendering to be triggered only by electron for single-article PDFs into T165954: Allow electron to render single PDF's on all projects.
Thu, Sep 7, 7:16 PM · Patch-For-Review, Readers-Web-Kanban-Board, Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson added a comment to T171833: Deploy new book renderer to all projects side by side with OCG.

Ah got it. I thought we were doing this as part of T173018... does it make sense to have two tasks here? We won't be able to sign off T173018 until the swat happens will we?

Thu, Sep 7, 7:10 PM · Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson updated the task description for T175161: Special:MobileHistory warning: Using deprecated fallback handling for comment rev_comment [Called from CommentStore::getCommentInternal in /Users/jrobson/git/core/includes/CommentStore.php at line 200].
Thu, Sep 7, 6:53 PM · MW-1.30-release-notes (WMF-deploy-2017-09-05 (1.30.0-wmf.17)), Wikimedia-log-errors, Unplanned-Sprint-Work, Readers-Web-Kanban-Board, Readers-Web-Backlog, MobileFrontend
Jdlrobson moved T175161: Special:MobileHistory warning: Using deprecated fallback handling for comment rev_comment [Called from CommentStore::getCommentInternal in /Users/jrobson/git/core/includes/CommentStore.php at line 200] from Upcoming to Ready for Signoff on the Readers-Web-Kanban-Board board.
Thu, Sep 7, 6:51 PM · MW-1.30-release-notes (WMF-deploy-2017-09-05 (1.30.0-wmf.17)), Wikimedia-log-errors, Unplanned-Sprint-Work, Readers-Web-Kanban-Board, Readers-Web-Backlog, MobileFrontend
Jdlrobson edited projects for T175161: Special:MobileHistory warning: Using deprecated fallback handling for comment rev_comment [Called from CommentStore::getCommentInternal in /Users/jrobson/git/core/includes/CommentStore.php at line 200], added: Readers-Web-Kanban-Board, Unplanned-Sprint-Work; removed Patch-For-Review.

Pulling into sprint since this has been merged and is being SWATTed today (presumably due to the noise in logstash) and it's important we verify the fix.

Thu, Sep 7, 6:51 PM · MW-1.30-release-notes (WMF-deploy-2017-09-05 (1.30.0-wmf.17)), Wikimedia-log-errors, Unplanned-Sprint-Work, Readers-Web-Kanban-Board, Readers-Web-Backlog, MobileFrontend
Jdlrobson added a comment to T171833: Deploy new book renderer to all projects side by side with OCG.

Could we update which configuration variable that would be? It's very vague right now to know what this is referring to.

Thu, Sep 7, 6:49 PM · Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson added a comment to T174751: Lazily loaded images do not get preserved on archive.org.

It does not appear to be running JavaScript resources... only inline JavaScript.

Thu, Sep 7, 6:48 PM · MobileFrontend, Internet-Archive, Readers-Web-Backlog
Jdlrobson added a comment to T174751: Lazily loaded images do not get preserved on archive.org.

I guess the question then is.. if it's caching and running JavaScript why does that not work?

Thu, Sep 7, 6:47 PM · MobileFrontend, Internet-Archive, Readers-Web-Backlog
Jdlrobson updated subscribers of T171956: Allow PDF rendering to be triggered only by electron for single-article PDFs.

What exactly needs deploying here? ElectronPdf is already deployed to all wikis.

Thu, Sep 7, 6:44 PM · Patch-For-Review, Readers-Web-Kanban-Board, Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson moved T171956: Allow PDF rendering to be triggered only by electron for single-article PDFs from To Triage to Needs Analysis on the Readers-Web-Backlog board.
Thu, Sep 7, 6:42 PM · Patch-For-Review, Readers-Web-Kanban-Board, Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson updated subscribers of T171833: Deploy new book renderer to all projects side by side with OCG.

This seems like a dupe of what @bmansurov is working on in T173018 ?

Thu, Sep 7, 6:42 PM · Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson moved T171833: Deploy new book renderer to all projects side by side with OCG from To Triage to Needs Analysis on the Readers-Web-Backlog board.
Thu, Sep 7, 6:42 PM · Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson added a comment to T171832: Deploy new book renderer to all projects.

Is there any deploy necessary here? I thought everything is deployed already - it's just a case of the service has not been fully built yet?

Thu, Sep 7, 6:42 PM · Wikimedia-Site-requests, Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson moved T171832: Deploy new book renderer to all projects from To Triage to Needs Analysis on the Readers-Web-Backlog board.
Thu, Sep 7, 6:41 PM · Wikimedia-Site-requests, Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson added a project to T171832: Deploy new book renderer to all projects: Wikimedia-Site-requests.
Thu, Sep 7, 6:41 PM · Wikimedia-Site-requests, Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson moved T171836: Apply new print styles for books from To Triage to Needs Analysis on the Readers-Web-Backlog board.
Thu, Sep 7, 6:41 PM · Wikimedia-Site-requests, Proton, Readers-Web-Backlog
Jdlrobson edited projects for T171836: Apply new print styles for books, added: Wikimedia-Site-requests; removed Electron-PDFs.

Do you mean "new Vector print styles" e.g. the ones we've been working on in T154965?

Thu, Sep 7, 6:41 PM · Wikimedia-Site-requests, Proton, Readers-Web-Backlog
Jdlrobson moved T111231: Page previews for Wikidata from Needs Analysis to Tracking on the Readers-Web-Backlog board.
Thu, Sep 7, 6:39 PM · Readers-Web-Backlog (Tracking), Wikimedia-Site-requests, User-aude, Beta-Feature, Story, Page-Previews, MediaWiki-extensions-WikibaseRepository, Wikidata
Jdlrobson moved T146944: Mobile-sections replacement from Needs Analysis to Tracking on the Readers-Web-Backlog board.
Thu, Sep 7, 6:38 PM · Readers-Web-Backlog (Tracking), Reading-Infrastructure-Team-Backlog, Reading Epics (Platform JS CSS and HTML consolidation), User-mobrovac, RESTBase-API, Services (watching), Wikipedia-Android-App-Backlog, Mobile-Content-Service
Jdlrobson moved T142527: Page actions should use semantic 'a' rather than 'li' as a button from Needs Analysis to Triaged but Future on the Readers-Web-Backlog board.
Thu, Sep 7, 6:38 PM · MinervaNeue, Readers-Web-Backlog, Front-end-Standards-Group, MediaWiki-Interface, Technical-Debt (RW-Tech-Debt), SEO, Accessibility
Jdlrobson changed the status of T142527: Page actions should use semantic 'a' rather than 'li' as a button from Stalled to Open.
Thu, Sep 7, 6:38 PM · MinervaNeue, Readers-Web-Backlog, Front-end-Standards-Group, MediaWiki-Interface, Technical-Debt (RW-Tech-Debt), SEO, Accessibility
Jdlrobson moved T128326: De-deploy the RelatedSites extension (used on Wikivoyage) from Needs Analysis to Tracking on the Readers-Web-Backlog board.
Thu, Sep 7, 6:36 PM · Readers-Web-Backlog (Tracking), Community-Liaisons, User-notice, MediaWiki-extensions-RelatedSites, Technical-Debt, Wikimedia-Site-requests
Jdlrobson updated the task description for T174670: Remove banner from saved books.
Thu, Sep 7, 6:36 PM · Proton, Readers-Web-Backlog
Jdlrobson changed the status of T173454: Lazy loading code should be skin agnostic from Open to Stalled.

I've written a proof of concept patch, but while doing this I uncovered the fact that this is blocked by T167713.
I have updated the description with a proposed solution.
I think this is ready to discuss/estimate at our convenience.

Thu, Sep 7, 6:35 PM · Patch-For-Review, Readers-Web-Backlog, MinervaNeue (Desktop)
Jdlrobson changed the status of T173454: Lazy loading code should be skin agnostic, a subtask of T167713: MinervaNeue's starter modules should we grouped by page not feature, from Open to Stalled.
Thu, Sep 7, 6:35 PM · MinervaNeue, Technical-Debt (RW-Tech-Debt), Readers-Web-Backlog
Jdlrobson updated the task description for T167713: MinervaNeue's starter modules should we grouped by page not feature.
Thu, Sep 7, 6:31 PM · MinervaNeue, Technical-Debt (RW-Tech-Debt), Readers-Web-Backlog
Jdlrobson updated the task description for T173454: Lazy loading code should be skin agnostic.
Thu, Sep 7, 6:28 PM · Patch-For-Review, Readers-Web-Backlog, MinervaNeue (Desktop)
Jdlrobson added a parent task for T173454: Lazy loading code should be skin agnostic: T167713: MinervaNeue's starter modules should we grouped by page not feature.
Thu, Sep 7, 6:26 PM · Patch-For-Review, Readers-Web-Backlog, MinervaNeue (Desktop)
Jdlrobson updated the task description for T173454: Lazy loading code should be skin agnostic.
Thu, Sep 7, 6:26 PM · Patch-For-Review, Readers-Web-Backlog, MinervaNeue (Desktop)
Jdlrobson moved T175286: Do side by side comparison of old summary endpoint against new summary endpoint from To Triage to Tracking on the Readers-Web-Backlog board.
Thu, Sep 7, 6:19 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Reading Epics (Page Previews), Services (watching), Page-Previews
Jdlrobson updated the task description for T173454: Lazy loading code should be skin agnostic.
Thu, Sep 7, 6:17 PM · Patch-For-Review, Readers-Web-Backlog, MinervaNeue (Desktop)
Jdlrobson updated subscribers of T172291: Launch page previews A/B test on enwiki and dewiki.

This has been lingering in sign off for some time and I wonder if there is a better way to capture the remaining problems, in a "verify A/B test card" that lives in the blocked column and has a clearer outline on what the remaining problem is here. The A/B test has been launched so it's misleading to leave this open (cc @MBinder_WMF ).

Thu, Sep 7, 5:58 PM · Patch-For-Review, Readers-Web-Kanban-Board, Page-Previews, Readers-Web-Backlog
Jdlrobson assigned T174430: Create a staging environment for OCG/Proton to phuedx.

Looks like Sam might be signing this off? Please re-assign if that's not the case.

Thu, Sep 7, 5:56 PM · Proton, Readers-Web-Kanban-Board, Readers-Web-Backlog, OCG-General
Jdlrobson assigned T175064: RelatedArticles not displaying on desktop wikivoyage: Cannot read property 'getBucket' of undefined console error to Niedzielski.

Stephen would you mind signing this off since I'll be away on vacation (note the "verify fix" steps for now and later in the description)?

Thu, Sep 7, 5:55 PM · Unplanned-Sprint-Work, Readers-Web-Kanban-Board, Readers-Web-Backlog, RelatedArticles
Jdlrobson updated the task description for T175064: RelatedArticles not displaying on desktop wikivoyage: Cannot read property 'getBucket' of undefined console error.
Thu, Sep 7, 5:54 PM · Unplanned-Sprint-Work, Readers-Web-Kanban-Board, Readers-Web-Backlog, RelatedArticles
Jdlrobson reassigned T174147: Update banner on books page from Jdlrobson to ovasileva.

Yes but you please verify this on the beta cluster. Any issues we'll want to catch and fix before the train rolls out on Tuesday:
https://en.wikipedia.beta.wmflabs.org/w/index.php?title=Special:ElectronPdf&page=Albert+Einstein&action=show-selection-screen&coll-download-url=%2Fw%2Findex.php%3Ftitle%3DSpecial%3ABook%26bookcmd%3Drender_article%26arttitle%3DAlbert%2BEinstein%26returnto%3DAlbert%2BEinstein%26oldid%3D364759%26writer%3Drdf2latex

Thu, Sep 7, 5:53 PM · Patch-For-Review, Readers-Web-Kanban-Board, Proton, Electron-PDFs, Readers-Web-Backlog
Jdlrobson moved T153125: Hide last modifier bar on revision pages from Needs QA to Needs Code Review on the Readers-Web-Kanban-Board board.

Per acceptance criteria there is still some refactoring for review:
https://gerrit.wikimedia.org/r/#/c/376281/

Thu, Sep 7, 5:52 PM · Readers-Web-Kanban-Board, Need-volunteer, Easy, MinervaNeue, Design, Readers-Web-Backlog
Jdlrobson moved T175161: Special:MobileHistory warning: Using deprecated fallback handling for comment rev_comment [Called from CommentStore::getCommentInternal in /Users/jrobson/git/core/includes/CommentStore.php at line 200] from Needs Analysis to Upcoming on the Readers-Web-Backlog board.
Thu, Sep 7, 4:33 PM · MW-1.30-release-notes (WMF-deploy-2017-09-05 (1.30.0-wmf.17)), Wikimedia-log-errors, Unplanned-Sprint-Work, Readers-Web-Kanban-Board, Readers-Web-Backlog, MobileFrontend
Jdlrobson added a comment to T175161: Special:MobileHistory warning: Using deprecated fallback handling for comment rev_comment [Called from CommentStore::getCommentInternal in /Users/jrobson/git/core/includes/CommentStore.php at line 200].

That was an easy fix..

Thu, Sep 7, 4:32 PM · MW-1.30-release-notes (WMF-deploy-2017-09-05 (1.30.0-wmf.17)), Wikimedia-log-errors, Unplanned-Sprint-Work, Readers-Web-Kanban-Board, Readers-Web-Backlog, MobileFrontend
Jdlrobson added a comment to T174957: Infobox breaking toc in new print styles.

@Jdlrobson, hi Jon. Sorry for the noob question but why do we need display: table-cell on .tocnumber, .toctext

It looks like this is used to control how the table of contents displays on long lines. I don't know how attached we are to that. We could easily disable this in mobile.



And yes this is related and another possible solution. The problem being that if the table does not express a width, it will consider the table of contents a table and assume it cannot break the content flow.

Thu, Sep 7, 4:27 PM · Vector, Readers-Web-Kanban-Board, Patch-For-Review, Proton, Readers-Web-Backlog
Jdlrobson added a comment to T175092: Determine if .navboxes should be stripped from the page content delivered by the Mobile Content Service and Page Content Service.

Why are we overcomplicating this? Just remove them like we do everywhere else. We can change this decision later if the Navbox template works on desktop, but that's not happening any time soon. Let's not over-engineer this.

Thu, Sep 7, 4:23 PM · Page Content Service, Mobile-Content-Service, Reading-Infrastructure-Team-Backlog (Kanban)
Jdlrobson added a comment to T175286: Do side by side comparison of old summary endpoint against new summary endpoint.

@Jdlrobson is development on the new end point still stalled while OCG is worked out? Or is it moving forward?

I'm working on it part time, but you should think of it as stalled. That said, I strongly believe the new endpoint is superior to the old and that's what I'd like to prove.

Thu, Sep 7, 4:21 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Reading Epics (Page Previews), Services (watching), Page-Previews
Jdlrobson updated the task description for T175286: Do side by side comparison of old summary endpoint against new summary endpoint.
Thu, Sep 7, 4:15 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Reading Epics (Page Previews), Services (watching), Page-Previews
Jdlrobson closed T168848: Bootstrap an initial version of the Page Summary API in MCS, a subtask of T113094: [EPIC] The Page Summary API needs to provide useful content for the majority of articles, as Resolved.
Thu, Sep 7, 4:15 PM · Reading-Infrastructure-Team-Backlog, Reading Epics (Page Previews), Services (watching), Readers-Web-Backlog, Epic, Tracking, Page-Previews
Jdlrobson closed T168848: Bootstrap an initial version of the Page Summary API in MCS as Resolved.

Thanks! Im not sure if a deploy only on beta cluster is useful now - we've shown this can be done when needed.
I'd suggest next step is for @bearND to do the side by side comparison: T175286

Thu, Sep 7, 4:15 PM · Reading-Infrastructure-Team-Backlog, Mobile-Content-Service, Readers-Web-Backlog, Page-Previews
Jdlrobson created T175286: Do side by side comparison of old summary endpoint against new summary endpoint.
Thu, Sep 7, 4:14 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Reading Epics (Page Previews), Services (watching), Page-Previews
Jdlrobson updated the task description for T174018: Port Minerva's browser tests to Selenium.
Thu, Sep 7, 3:25 PM · MW-1.30-release-notes (WMF-deploy-2017-08-29 (1.30.0-wmf.16)), Release-Engineering-Team (Kanban), MinervaNeue, Readers-Web-Backlog, User-zeljkofilipin
Jdlrobson merged T148973: Identify our browser test coverage into T160238: Rewrite MobileFrontend, QuickSurveys, Page previews automated browser tests in Node.js.
Thu, Sep 7, 3:23 PM · QuickSurveys, Page-Previews, MobileFrontend, Patch-For-Review, User-zeljkofilipin, Readers-Web-Backlog
Jdlrobson merged task T148973: Identify our browser test coverage into T160238: Rewrite MobileFrontend, QuickSurveys, Page previews automated browser tests in Node.js.
Thu, Sep 7, 3:23 PM · Readers-Web-Backlog, Technical-Debt (RW-Tech-Debt), MobileFrontend
Jdlrobson updated the task description for T160238: Rewrite MobileFrontend, QuickSurveys, Page previews automated browser tests in Node.js.
Thu, Sep 7, 3:23 PM · QuickSurveys, Page-Previews, MobileFrontend, Patch-For-Review, User-zeljkofilipin, Readers-Web-Backlog
Jdlrobson removed a project from T174430: Create a staging environment for OCG/Proton: Patch-For-Review.
Thu, Sep 7, 2:55 PM · Proton, Readers-Web-Kanban-Board, Readers-Web-Backlog, OCG-General
Jdlrobson removed a project from T168848: Bootstrap an initial version of the Page Summary API in MCS: Patch-For-Review.
Thu, Sep 7, 2:55 PM · Reading-Infrastructure-Team-Backlog, Mobile-Content-Service, Readers-Web-Backlog, Page-Previews
Jdlrobson added a comment to T162256: [EPIC] Port Selenium tests from Ruby to Node.js on Reading Web extensions.
const UserLoginPage = require( '../../../../../tests/selenium/userlogin.page' );
UserLoginPage.login( username, password );

great that was what I was looking for.

Thu, Sep 7, 2:40 PM · Release-Engineering-Team (Kanban), Readers-Web-Backlog, User-zeljkofilipin, RelatedArticles, MobileFrontend, Epic
Jdlrobson removed a project from T173600: Improve treatment of hatnotes: Patch-For-Review.
Thu, Sep 7, 2:39 PM · Readers-Web-Kanban-Board, Design (RW-Design-Debt), Readers-Web-Backlog
Jdlrobson removed a project from T175064: RelatedArticles not displaying on desktop wikivoyage: Cannot read property 'getBucket' of undefined console error: Patch-For-Review.
Thu, Sep 7, 2:39 PM · Unplanned-Sprint-Work, Readers-Web-Kanban-Board, Readers-Web-Backlog, RelatedArticles
Jdlrobson closed T56679: PLURAL support for Coll-load local book, a subtask of T40781: Messages needing PLURAL support (tracking), as Resolved.
Thu, Sep 7, 2:38 PM · Tracking, I18n, MediaWiki-Interface