bearND (Bernd Sitzmann)
Reading Infrastructure Services dev

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Sunday

  • Clear sailing ahead.

User Details

User Since
Nov 19 2014, 8:27 PM (170 w, 1 d)
Availability
Available
IRC Nick
bearND
LDAP User
BearND
MediaWiki User
Unknown

Recent Activity

Today

bearND updated the task description for T187936: List of issues of 1.3.0 summary endpoint results.
Fri, Feb 23, 7:17 AM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND moved T186838: Handle dewiki hatnotes from Deployed to Code Review on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Fri, Feb 23, 7:14 AM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
bearND added a comment to T183833: [Bug report] Removing parentheses breaks chemical formulas.

The other ones are missing several components. Here are the text versions of the formulas:

  • Nefrit: Ca2(Mg,Fe)5Si8O22(OH)2
  • Nephrite: Ca2 5Si8O22(OH)2
  • Olivín: (Mg,Fe)2[SiO4]
  • Olivine: 2SiO4
Fri, Feb 23, 1:13 AM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Readers-Web-Backlog (Tracking), Page-Previews
bearND added a comment to T183833: [Bug report] Removing parentheses breaks chemical formulas.

@Vachovec1 Thanks. These are even better test cases. We also need to remove the following attributes: rel, href, title. I'm going to explore running the html-sanitizer step before (in addition to after) the parentheses stripping since that whitelists attributes we want to keep instead of a blacklist of attributes we want to remove.

Fri, Feb 23, 12:57 AM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Readers-Web-Backlog (Tracking), Page-Previews
bearND added a comment to T183833: [Bug report] Removing parentheses breaks chemical formulas.

I'm running the compare script. How urgent is this to get deployed?

Fri, Feb 23, 12:02 AM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Readers-Web-Backlog (Tracking), Page-Previews
bearND claimed T183833: [Bug report] Removing parentheses breaks chemical formulas.
Fri, Feb 23, 12:01 AM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Readers-Web-Backlog (Tracking), Page-Previews
bearND moved T183833: [Bug report] Removing parentheses breaks chemical formulas from To Do to To Deploy on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Fri, Feb 23, 12:00 AM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Readers-Web-Backlog (Tracking), Page-Previews
bearND moved T183833: [Bug report] Removing parentheses breaks chemical formulas from Needs triage to Kanban on the Reading-Infrastructure-Team-Backlog board.
Fri, Feb 23, 12:00 AM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Readers-Web-Backlog (Tracking), Page-Previews

Yesterday

bearND added a comment to T183833: [Bug report] Removing parentheses breaks chemical formulas.

We don't want to be too aggressive when removing attributes since we don't want to remove needed attributes for other elements, like img, maybe figure and more.

Thu, Feb 22, 11:42 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Readers-Web-Backlog (Tracking), Page-Previews
bearND added a comment to T183833: [Bug report] Removing parentheses breaks chemical formulas.

Ugh, this is going to get ugly. The reason why (CH<sub id="mwCw">2</sub>) is stripped is because it does include a space -- in the HTML, before the id attribute. I can strip the id attributes beforehand but this doesn't guarantee that there wouldn't be other attributes leading to a strip of the parenthetical.

Thu, Feb 22, 11:23 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Readers-Web-Backlog (Tracking), Page-Previews
bearND added a comment to T183833: [Bug report] Removing parentheses breaks chemical formulas.

@Jdlrobson Sure. Will look into it. Thanks for the test case.

Thu, Feb 22, 10:44 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Readers-Web-Backlog (Tracking), Page-Previews
bearND added a comment to T171052: Add disambiguation page handling in Page Summary API.

The MCS version does expose the revision for summaries but RESTBase does not. Not sure why. If I had to guess it may have to do with storage overhead. From what I remember is that there wasn't a use case for it.

Thu, Feb 22, 10:17 PM · Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Mobile-Content-Service, Page-Previews
bearND added a comment to T171052: Add disambiguation page handling in Page Summary API.

@Jdlrobson f you don't want to wait you can purge the page with ?action=purge.

Thu, Feb 22, 10:10 PM · Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Mobile-Content-Service, Page-Previews
bearND updated subscribers of T171052: Add disambiguation page handling in Page Summary API.

Yes, this one is still at 1.2.0.

Thu, Feb 22, 10:08 PM · Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Mobile-Content-Service, Page-Previews
bearND added a comment to T188005: HTML previews displaying changes made by gadgets.

@Jdlrobson I think MCS could still chose to drop any span.mw-redirects from summary output, knowing that these came from a.mw-redirects. Since the summary changes the links to spans.

Thu, Feb 22, 9:19 PM · Reading-Web-Local-Wiki-Issues, Page-Previews, Readers-Web-Backlog
bearND moved T179947: Improve geo coordinate reporting in MCS from To Deploy to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Thu, Feb 22, 9:17 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)
bearND moved T172005: Extract categories as a structured array in the metadata endpoint from To Deploy to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Thu, Feb 22, 9:17 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban)
bearND moved T172002: Extract Page Issues as a structured array from To Deploy to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Thu, Feb 22, 9:17 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban)
bearND moved T186838: Handle dewiki hatnotes from Code Review to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Thu, Feb 22, 9:17 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
bearND added a comment to T187936: List of issues of 1.3.0 summary endpoint results.

Same on Safari on macOS.

Thu, Feb 22, 6:06 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND added a comment to T187936: List of issues of 1.3.0 summary endpoint results.

@ssastry To see the difference you may need to look closely. I've increased the font size so it's easier to see. This is with Chrome. Note the difference in spacing where the diacritics are:

Thu, Feb 22, 5:17 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND added a comment to T187936: List of issues of 1.3.0 summary endpoint results.

So, for reading purposes we should be able to get rid of them and flatten them completely.

Thu, Feb 22, 4:55 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND added a comment to T188005: HTML previews displaying changes made by gadgets.

Just explaining what's happening here technically:
Links with redirects get a special class in Parsoid: <a class="mw-redirect">.
When MCS creates the summary it turns the <a> tags into <span> tags,
so the result is: <span class="mw-redirect">. MCS removes the id attributes but class and style attributes would be preserved.

Thu, Feb 22, 4:46 PM · Reading-Web-Local-Wiki-Issues, Page-Previews, Readers-Web-Backlog
bearND added a comment to T187955: Page preview shows icon instead of thumbnail.

@ovasileva I found this on en:VPT: https://en.wikipedia.org/wiki/Wikipedia:Village_pump_(technical)#Hovercards. I haven't found any others yet.

Thu, Feb 22, 4:14 PM · MW-1.31-release-notes (WMF-deploy-2018-02-27 (1.31.0-wmf.23)), Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Page-Previews, Readers-Web-Backlog
bearND added a project to T187936: List of issues of 1.3.0 summary endpoint results: Parsoid.
Thu, Feb 22, 4:04 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND added a comment to T187936: List of issues of 1.3.0 summary endpoint results.

Another couple of things:

  1. do we know why some articles are appearing without a summary? (eswiki: http://wpsummary.surge.sh/1.2.0-b0be98c/html/es.html#27, http://wpsummary.surge.sh/1.2.0-b0be98c/html/es.html#3, cswiki: http://wpsummary.surge.sh/1.2.0-b0be98c/html/cs.html#22)
Thu, Feb 22, 4:03 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND updated the task description for T187936: List of issues of 1.3.0 summary endpoint results.
Thu, Feb 22, 3:38 AM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND added a comment to T182321: Show HTML summaries on cswiki.

The cswiki page summaries have been re-rendered with the correct thumbnail size. We're still working through other wikis but for cs the blank thumbnail image issue should be resolved.

Thu, Feb 22, 3:26 AM · Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikimedia-Site-requests, Page-Previews, Readers-Web-Backlog
bearND added a comment to T187955: Page preview shows icon instead of thumbnail.

@Jdlrobson Yep. The change was that the new summary was requesting a larger thumbnail image (for reasons unbeknownst to me), which resulted in the original image URL being returned for the thumbnail field as well. That one doesn't have the px- piece in the filename.

Thu, Feb 22, 3:06 AM · MW-1.31-release-notes (WMF-deploy-2018-02-27 (1.31.0-wmf.23)), Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Page-Previews, Readers-Web-Backlog
bearND updated subscribers of T187955: Page preview shows icon instead of thumbnail.

We should change both the server and the client side.

  • On the server side, MCS should request smaller thumbnails for the summaries. The MCS patch is up ^ and will be deployed shortly. Once that is deployed @Pchelolo is going to run a dump of summaries over enwiki and cswiki (hopefully in parallel).
  • The client should remove the assumption of having px- in the thumbnail URL.
Thu, Feb 22, 12:15 AM · MW-1.31-release-notes (WMF-deploy-2018-02-27 (1.31.0-wmf.23)), Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Page-Previews, Readers-Web-Backlog

Wed, Feb 21

bearND added a comment to T187955: Page preview shows icon instead of thumbnail.

Yes, it's in rest.js.

Wed, Feb 21, 11:10 PM · MW-1.31-release-notes (WMF-deploy-2018-02-27 (1.31.0-wmf.23)), Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Page-Previews, Readers-Web-Backlog
bearND added a comment to T182321: Show HTML summaries on cswiki.

Correct. I can repro it on FF showing blank space. What I showed above was with Chrome.

Wed, Feb 21, 10:53 PM · Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikimedia-Site-requests, Page-Previews, Readers-Web-Backlog
bearND added a comment to T187955: Page preview shows icon instead of thumbnail.

E.g., when hovering over Timeline_of_the_far_future. The summary has the thumbnail
https://upload.wikimedia.org/wikipedia/commons/a/aa/Red_Giant_Earth_warm.jpg, which would have worked.
Instead it tries to download https://upload.wikimedia.org/wikipedia/commons/a/aa/320px-d_Giant_Earth_warm.jpg, which gets a 404.
Note the s/Red/320px-d/.

Wed, Feb 21, 10:47 PM · MW-1.31-release-notes (WMF-deploy-2018-02-27 (1.31.0-wmf.23)), Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Page-Previews, Readers-Web-Backlog
bearND added a comment to T182321: Show HTML summaries on cswiki.

I can also reproduce the same on a few enwiki pages, see new subtask T187955.

Wed, Feb 21, 10:39 PM · Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikimedia-Site-requests, Page-Previews, Readers-Web-Backlog
bearND triaged T187955: Page preview shows icon instead of thumbnail as High priority.
Wed, Feb 21, 10:37 PM · MW-1.31-release-notes (WMF-deploy-2018-02-27 (1.31.0-wmf.23)), Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Page-Previews, Readers-Web-Backlog
bearND added a comment to T182321: Show HTML summaries on cswiki.

The thumbnail in this page preview looks definitely wrong:
From https://cs.wikipedia.org/wiki/Donald_Trump hover over "prezident Spojených států amerických"

Wed, Feb 21, 10:19 PM · Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikimedia-Site-requests, Page-Previews, Readers-Web-Backlog
bearND added a comment to T182321: Show HTML summaries on cswiki.

I suspect that this is a page preview (web client) issue. The /page/summary JSON responses for these pages look fine to me. All have the thumbnail property:

Wed, Feb 21, 10:06 PM · Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikimedia-Site-requests, Page-Previews, Readers-Web-Backlog
bearND added a comment to T182321: Show HTML summaries on cswiki.

The summaries for all pages on cswiki are now rendered using MCS.

Wed, Feb 21, 8:17 PM · Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikimedia-Site-requests, Page-Previews, Readers-Web-Backlog
bearND updated the task description for T187936: List of issues of 1.3.0 summary endpoint results.
Wed, Feb 21, 8:14 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND added a project to T187936: List of issues of 1.3.0 summary endpoint results: Page Content Service.
Wed, Feb 21, 8:11 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND updated the task description for T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.
Wed, Feb 21, 8:01 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND updated the task description for T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.
Wed, Feb 21, 8:00 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND removed the point value for T187936: List of issues of 1.3.0 summary endpoint results.
Wed, Feb 21, 7:58 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND created T187936: List of issues of 1.3.0 summary endpoint results.
Wed, Feb 21, 7:57 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service
bearND added a comment to T187841: Decide on protection output format for the metadata endpoint.

I'd like to hear from client teams what they think. If would be great to hear from devs familiar with the creation of the action=mobileview API.

Wed, Feb 21, 4:41 PM · Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND updated the task description for T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.
Wed, Feb 21, 5:02 AM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND added a comment to T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.

Looks like the backlog of summary rerenders is worked off now, and I can see pages from the Recent Changes list getting the new MCS summary treatment automatically.

Wed, Feb 21, 5:01 AM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Tue, Feb 20

bearND added a comment to T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.

I guess we're in an unplanned, super-slow-rollout mode right now. While it works for manual purges, new pages showing up in recent changes don't get updated to 1.3.0 yet.

Tue, Feb 20, 7:28 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND added a comment to T182321: Show HTML summaries on cswiki.

Here's the latest comparison run for the Czech html extracts from TextExtracts vs the MCS summary implementation: http://wpsummary.surge.sh/1.2.0-b0be98c/html/cs.html. See also T179875#3986429.

Tue, Feb 20, 5:07 PM · Readers-Web-Kanbanana-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikimedia-Site-requests, Page-Previews, Readers-Web-Backlog
bearND added a comment to T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.

I've ran another compare run between the old 1.2.0 (TextExtracts) version and the latest (MCS repo b0be98c).

Tue, Feb 20, 4:50 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND updated the task description for T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.
Tue, Feb 20, 4:20 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND moved T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete from Blocked to Doing on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Tue, Feb 20, 4:20 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND moved T181636: Content service incorrectly reports article as "deleted" from Needs triage to Tracking on the Reading-Infrastructure-Team-Backlog board.
Tue, Feb 20, 4:03 PM · Services (done), Reading-Infrastructure-Team-Backlog
bearND removed a project from T181636: Content service incorrectly reports article as "deleted": Mobile-Content-Service.
Tue, Feb 20, 4:02 PM · Services (done), Reading-Infrastructure-Team-Backlog

Sat, Feb 17

bearND updated subscribers of T177428: Develop Metadata JSON API.

@Dbrant @Mhurd @JoeWalsh Are you guys ok using just the HTML version for page issues and hatnotes? In other words can we drop the plain text version of these?

Sat, Feb 17, 1:56 AM · MW-1.31-release-notes (WMF-deploy-2018-02-27 (1.31.0-wmf.23)), Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND edited projects for T181636: Content service incorrectly reports article as "deleted", added: Parsoid, Services; removed Page Content Service.

That's coming from the Parsoid response: https://en.wikipedia.org/api/rest_v1/page/html/Public_choice

Sat, Feb 17, 1:39 AM · Services (done), Reading-Infrastructure-Team-Backlog

Fri, Feb 16

bearND updated the task description for T187475: Extra square in 'In the news' section.
Fri, Feb 16, 12:05 AM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)

Thu, Feb 15

bearND added a comment to T187384: Content service assigns incorrect ids to sections in certain pages..

Adding to that, I think for now the Android app should treat sections with negative section ids as uneditable, and wouldn't even show the pencil icon for these. On this article, editing the "India" section in the Android app works but none of its subheadings.

Thu, Feb 15, 9:23 PM · Reading-Infrastructure-Team-Backlog, Page Content Service, Mobile-Content-Service, Android-app-Bugs, Wikipedia-Android-App-Backlog
bearND created T187507: Add section attribute with page title for transcluded templates.
Thu, Feb 15, 9:21 PM · Parsoid, Reading-Infrastructure-Team-Backlog, Page Content Service, Mobile-Content-Service, Wikipedia-Android-App-Backlog
bearND added a comment to T187384: Content service assigns incorrect ids to sections in certain pages..

@Dbrant The -1 or -2 can theoretically happen on any page. Practically I have only observed that on non-main namespaces (talk or project are other namespaces that come to mind), though. This comes from Parsoid[1]. IIUC those section ids mean that the section is not editable, mainly due to some template transclusion of a page with some HTML heading tags happening. If you want we could make a patch in MCS to change the values to something else. The problem is figuring out what that something else should be.

Thu, Feb 15, 7:13 PM · Reading-Infrastructure-Team-Backlog, Page Content Service, Mobile-Content-Service, Android-app-Bugs, Wikipedia-Android-App-Backlog
bearND added a comment to T187430: Duplicate usage examples in Wiktionary page definition endpoint.

@jberkel This is slightly off topic but since you seem to be interested in the Wiktionary definitions endpoint I was wondering whether you had any thoughts on T133051#3976643.

Thu, Feb 15, 6:52 PM · Wikipedia-Android-App-Backlog, Wiktionary, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
bearND added a comment to T133051: [BUG] Wiktionary links to Appendix:Glossary but doesn't show results.

@cooltey RESTBase automatically resolves redirects for you. So that shouldn't be a problem.

Thu, Feb 15, 6:49 PM · Wikipedia-Android-App-Backlog, Android-app-Bugs, Patch-For-Review, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
bearND closed T187475: Extra square in 'In the news' section as Resolved.
Thu, Feb 15, 6:45 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)
bearND moved T187475: Extra square in 'In the news' section from To Deploy to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Thu, Feb 15, 6:44 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)
bearND moved T187475: Extra square in 'In the news' section from Code Review to To Deploy on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Thu, Feb 15, 5:47 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)
bearND added a comment to T187475: Extra square in 'In the news' section.

After fix:

Thu, Feb 15, 5:46 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)
bearND moved T187475: Extra square in 'In the news' section from Doing to Code Review on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Thu, Feb 15, 5:35 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)
bearND updated the task description for T187475: Extra square in 'In the news' section.
Thu, Feb 15, 5:30 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)
bearND created T187475: Extra square in 'In the news' section.
Thu, Feb 15, 5:27 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)
bearND awarded T187430: Duplicate usage examples in Wiktionary page definition endpoint a Like token.
Thu, Feb 15, 2:41 PM · Wikipedia-Android-App-Backlog, Wiktionary, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
bearND added a comment to T172005: Extract categories as a structured array in the metadata endpoint.

Updated title and description of this task to reflect the current plan.

Thu, Feb 15, 3:57 AM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban)
bearND renamed T172005: Extract categories as a structured array in the metadata endpoint from Extract categories as a structured array to Extract categories as a structured array in the metadata endpoint.
Thu, Feb 15, 3:56 AM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban)

Wed, Feb 14

bearND added a comment to T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.

Update: since the main guys for this in the Services team are out today and tomorrow the earliest would be Tuesday, Feb 20.

Wed, Feb 14, 3:59 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Mon, Feb 12

bearND added a comment to T187113: Ensure that all network handlers can gracefully handle HTTP Success codes other than 200..

Just to be explicit, the issue was probably not the non-200 error code but the empty payload.

Mon, Feb 12, 9:19 PM · Technical-Debt, Wikipedia-Android-App-Backlog
bearND added a subtask for T184753: Use cached page leads when creating page summaries to reduce MCS load: T187098: Extra span at beginning of mobile-sections-lead.
Mon, Feb 12, 6:11 PM · Reading-Infrastructure-Team-Backlog, Patch-For-Review, Services (blocked), Page Content Service, Reading Epics (Platform JS CSS and HTML consolidation)
bearND added a parent task for T187098: Extra span at beginning of mobile-sections-lead: T184753: Use cached page leads when creating page summaries to reduce MCS load.
Mon, Feb 12, 6:11 PM · Reading-Infrastructure-Team-Backlog (Kanban)
bearND updated the task description for T187098: Extra span at beginning of mobile-sections-lead.
Mon, Feb 12, 5:51 PM · Reading-Infrastructure-Team-Backlog (Kanban)
bearND created T187098: Extra span at beginning of mobile-sections-lead.
Mon, Feb 12, 5:49 PM · Reading-Infrastructure-Team-Backlog (Kanban)
bearND added a comment to T186933: 204 summary responses cause Android app to crash.

Sounds good. I think we should be able to get that one in by today's deployment window if we all agree on what value we should use for the type field in the cases we were returning 204 before. I brought up none as a strawman proposal. Is that ok or should we use a different value?

Mon, Feb 12, 5:16 PM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND added a comment to T186933: 204 summary responses cause Android app to crash.

Thanks! How about we change the spec to return 200s and empty extracts in the cases we were returning 204s?
Also let the apps still prepare for 204s in the future.

Mon, Feb 12, 3:05 PM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Sun, Feb 11

bearND added a comment to T186933: 204 summary responses cause Android app to crash.

Maybe the interaction of page previews and link previews is different enough to warrant more thought/discussion about the behavior differences. There's a delay before page previews appear on web but none before link previews in the Android app because if Link Previews are enabled that's currently the only way to follow most links. We don't want a delay here because that would slow down navigation in the Android app. (The iOS app is still another story since one has to hard press the link to get the link preview there.)

Sun, Feb 11, 10:16 PM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Sat, Feb 10

bearND added a comment to T186944: extract and extract_html empty for some page summary responses.

I think this is probably due to T186927 which happened today. Looks like some of the summary results need to be purged/re-rendered. I guess the ones that had a rerender triggered during the time the bug existed and until the fix was deployed. Not sure if it's worth forcing a rerender on all summaries if we still go ahead with the switchover to summary 1.3.0 soon. We were planning to do that on Monday it might be delayed, see T186933.

Sat, Feb 10, 5:44 AM · Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
bearND added a comment to T186933: 204 summary responses cause Android app to crash.

Well, it's really up to @Fjalapeno and @phuedx to hash out if we do go ahead on Monday or later. One possible option would be to change the spec to return 200 but a special type for when not to show previews. That should be BWC for the apps. Or we allow more time for the apps to patch things. Or we say that this only affects a few pages and is rare enough that we can go ahead with this on Monday. (Although the Android crash is bad, of course.)

Sat, Feb 10, 2:08 AM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND added a comment to T186933: 204 summary responses cause Android app to crash.

@Dbrant I think we want to not show a link preview in this case instead of the error I'm seeing with this patch:

Sat, Feb 10, 12:23 AM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND added a comment to T186933: 204 summary responses cause Android app to crash.

@Dbrant Nice! Maybe there is hope for the current spec after all. When do you think this could be released? Could this be back-ported to older Android versions which don't get the updates?

Sat, Feb 10, 12:11 AM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Fri, Feb 9

bearND updated the task description for T186933: 204 summary responses cause Android app to crash.
Fri, Feb 9, 11:23 PM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND awarded T186927: mw-parser-output divs leaking into mobileview output again a Yellow Medal token.
Fri, Feb 9, 11:18 PM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), MediaWiki-Parser, Patch-For-Review, Android-app-Bugs, Wikipedia-iOS-App-Backlog, iOS-app-Bugs, Wikipedia-Android-App-Backlog, MobileFrontend
bearND updated the task description for T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.
Fri, Feb 9, 11:10 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND added a comment to T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.

Due to T186933 we should postpone the summary switchover until a proper solution has been agreed and deployed.

Fri, Feb 9, 11:10 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND triaged T186933: 204 summary responses cause Android app to crash as High priority.
Fri, Feb 9, 11:06 PM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND added a comment to T133051: [BUG] Wiktionary links to Appendix:Glossary but doesn't show results.

Hmm, for which Glossary word (on what page) does it work? It doesn't work for "mantis"->"mantisees"-> "plural", as @cooltey mentioned earlier.

Fri, Feb 9, 5:10 PM · Wikipedia-Android-App-Backlog, Android-app-Bugs, Patch-For-Review, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
bearND added a comment to T186761: Normalize messages in service-runner.

Thanks, the Kibana dashboard for MCS over the last 12 hours looks much cleaner already. :)

Fri, Feb 9, 4:57 PM · Services (done), service-runner
bearND added a comment to T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.

Sorry, we found another issue, see new subtask, which delayed the switchover to Monday morning.

Fri, Feb 9, 1:20 AM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Thu, Feb 8

bearND updated the task description for T179875: Update RESTBase to get summary content from MCS Summary 1.3 endpoint when development is complete.
Thu, Feb 8, 5:09 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
bearND added a comment to T184000: Magic word on English WP to override display of Wikidata short description.

Are there any older app versions out there which might use a different method to get the data? Does Android use MCS?

Thu, Feb 8, 4:08 PM · Reading-Infrastructure-Team-Backlog (Kanban), MediaWiki-extensions-WikibaseClient, Wikidata, Community-Tech
bearND added a comment to T184753: Use cached page leads when creating page summaries to reduce MCS load.

@Mholloway What is it blocked on?

Thu, Feb 8, 3:53 PM · Reading-Infrastructure-Team-Backlog, Patch-For-Review, Services (blocked), Page Content Service, Reading Epics (Platform JS CSS and HTML consolidation)

Wed, Feb 7

bearND added a comment to T184000: Magic word on English WP to override display of Wikidata short description.

@Tgr The iOS app does it directly through action=mobileview but that should be easy enough to change there to page_props as well if necessary. The harder part would be changing the description editing workflow on the apps.
Question: is this only for English WP?

Wed, Feb 7, 11:32 PM · Reading-Infrastructure-Team-Backlog (Kanban), MediaWiki-extensions-WikibaseClient, Wikidata, Community-Tech
bearND added a comment to T186627: [EPIC] Deprecate the MobileView API.

@Jdlrobson MCS is currently using action=mobileview for the main pages since the content of the main page content is not responsive. Hoping for T138622#3416133 to get done. Then we could get rid of the special handling there.

Wed, Feb 7, 10:50 PM · Epic, Readers-Web-Backlog
bearND closed T186745: MW API requests to mediawiki.org (without the www subdomain) fail as Resolved.
Wed, Feb 7, 10:40 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, service-template-node, Mobile-Content-Service
bearND added a comment to T186745: MW API requests to mediawiki.org (without the www subdomain) fail.

Some comments between @Mholloway and myself on IRC -services channel earlier with minor edits:

Wed, Feb 7, 10:40 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, service-template-node, Mobile-Content-Service