Page MenuHomePhabricator

JoeWalsh (Joe Walsh)
Apps Engineering Manager

Projects (7)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
May 31 2016, 8:28 PM (217 w, 6 d)
Availability
Available
LDAP User
Joewalsh
MediaWiki User
JoeWalsh (WMF) [ Global Accounts ]

Recent Activity

Yesterday

JoeWalsh updated subscribers of T259149: [Spike] Estimate how many writes echo_push_subscription would be having.

Current active users (iOS/Android)
New users per month (iOS/Android)

Mon, Aug 3, 2:22 PM · Product-Infrastructure-Team-Backlog (Kanban), Push-Notification-Service, Spike

Wed, Jul 29

JoeWalsh closed T258977: Remove whitelist/blacklist/master in Wikipedia Product Infrastructure (PCS) codebase used by mobile apps, a subtask of T254646: Reconsidering how we name things, as Resolved.
Wed, Jul 29, 3:40 PM · MW-1.36-notes (1.36.0-wmf.2; 2020-07-28), MediaWiki-extensions-General, Wikimedia-General-or-Unknown, MediaWiki-General, MW-1.35-notes (1.35.0-wmf.39; 2020-06-30), Patch-For-Review, Voice & Tone
JoeWalsh closed T258977: Remove whitelist/blacklist/master in Wikipedia Product Infrastructure (PCS) codebase used by mobile apps as Resolved.
Wed, Jul 29, 3:40 PM · Product-Infrastructure-Team-Backlog, Voice & Tone

Tue, Jul 28

JoeWalsh closed T218379: Make onboarding components reusable as Declined.

Definitely worth doing if/when we revisit onboarding, but not worth keeping captured as a separate ticket

Tue, Jul 28, 4:22 PM · Technical-Debt, Wikipedia-iOS-App-Backlog
JoeWalsh closed T217411: Session & SchemeHandler improvements to prepare for mobile-html as Resolved.
Tue, Jul 28, 4:20 PM · Wikipedia-iOS-App-Backlog
JoeWalsh closed T164637: WikipediaRTL scheme doesn't seem to set date/time to format for Hebrew as Resolved.
Tue, Jul 28, 4:19 PM · WorkType-Maintenance, Wikipedia-iOS-App-Backlog
JoeWalsh closed T175722: Expand snapshot testing or add UI tests as Declined.
Tue, Jul 28, 4:19 PM · Wikipedia-iOS-App-Backlog
JoeWalsh closed T157523: Make Password reminder, Change pwd and 2FA view controllers use LoginFunnel as Resolved.
Tue, Jul 28, 4:18 PM · iOS-app-feature-Login, Wikipedia-iOS-App-Backlog
JoeWalsh closed T164636: Use babel for JS compiling and use similar babel settings as wikipedia-page-library as Invalid.
Tue, Jul 28, 4:17 PM · WorkType-Maintenance, wikimedia-page-library, Wikipedia-iOS-App-Backlog
JoeWalsh closed T164634: Refactor old crufty editing fetchers to Swift (old horrible delegate fetchers) as Resolved.
Tue, Jul 28, 4:16 PM · WorkType-Maintenance, Wikipedia-iOS-App-Backlog
JoeWalsh closed T164635: Bring the nice JS linting rules from wikipedia-page-library to the iOS app as Declined.
Tue, Jul 28, 4:16 PM · WorkType-Maintenance, wikimedia-page-library, Wikipedia-iOS-App-Backlog
JoeWalsh closed T164635: Bring the nice JS linting rules from wikipedia-page-library to the iOS app , a subtask of T163831: Investigate eslint-config-wikimedia, stylelint-config-wikimedia, as Declined.
Tue, Jul 28, 4:16 PM · Spike, wikimedia-page-library
JoeWalsh closed T171912: WMFPicOfTheDayCollectionViewCell and WMFSideScrollingCollectionViewCell image urls don't appear to be using wmf_maxScale variant urls as Invalid.
Tue, Jul 28, 4:15 PM · Wikipedia-iOS-App-Backlog
JoeWalsh closed T187081: Ensure the iOS app gracefully handles non-200 responses from any API call as Resolved.
Tue, Jul 28, 4:14 PM · Wikipedia-iOS-App-Backlog
JoeWalsh closed T185142: [BUG] Delay after tapping magnifying glass on article view as Resolved.

Unable to repro on 6.6.2, iPhone X

Tue, Jul 28, 4:14 PM · iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh closed T193183: Investigate invalid project name errors from reading list service as Resolved.
Tue, Jul 28, 4:11 PM · Wikipedia-iOS-App-Backlog
JoeWalsh closed T211224: [QA Feature Request] Dev/debug menu hidden in prod builds as Declined.
Tue, Jul 28, 4:11 PM · Wikipedia-iOS-App-Backlog

Mon, Jul 27

JoeWalsh renamed T256416: iOS 14 Updates from Investigate changes in iOS 14 to iOS 14 Updates.
Mon, Jul 27, 6:40 PM · Epic, iOS-app-v6.7-Bonefish-On-A-Bus, Wikipedia-iOS-App-Backlog
JoeWalsh added a comment to T257860: Update PageviewDefinition to only include /api/rest_v1/page/mobile-html requests with X-Analytics: pageview=1 in pageviews.

So we are all on the same page this will *reduce* the number of pageviews, see plot. For 2020/07/24 data for IOS the reduction is about 8% on data marked now as 'user'

Confirmed, that's expected given that we're now excluding requests to save for offline viewing and requests from versions older than 6.6.2.

Mon, Jul 27, 5:31 PM · Patch-For-Review, Analytics-Kanban, Analytics
JoeWalsh moved T258949: mobile-html: incorrect text color on United States infobox from To Do to Code Review on the Product-Infrastructure-Team-Backlog (Kanban) board.
Mon, Jul 27, 1:59 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Page Content Service
JoeWalsh moved T258949: mobile-html: incorrect text color on United States infobox from Needs triage to Kanban on the Product-Infrastructure-Team-Backlog board.
Mon, Jul 27, 1:59 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Page Content Service
JoeWalsh claimed T258949: mobile-html: incorrect text color on United States infobox.
Mon, Jul 27, 1:59 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Page Content Service
JoeWalsh created T258949: mobile-html: incorrect text color on United States infobox.
Mon, Jul 27, 1:59 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Page Content Service

Wed, Jul 22

JoeWalsh added a comment to T246362: Request creation of "entity-detection" VPS project.

Recreating it is an option. It'd probably be easier to recreate than downloading to a laptop.

Wed, Jul 22, 8:29 PM · Cloud-VPS (Project-requests)
JoeWalsh claimed T231987: iOS 13 Core Data value transformer deprecation.
Wed, Jul 22, 7:36 PM · iOS-app-v6.7-Bonefish-On-A-Bus, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh moved T231987: iOS 13 Core Data value transformer deprecation from Ready for Development to Needs Code Review on the iOS-app-v6.7-Bonefish-On-A-Bus board.
Wed, Jul 22, 7:36 PM · iOS-app-v6.7-Bonefish-On-A-Bus, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh added a comment to T246362: Request creation of "entity-detection" VPS project.

Hi @Andrew, 1 wouldn't be a problem. For 2 would the data be restored or would I need to recreate it from scratch?

Wed, Jul 22, 6:12 PM · Cloud-VPS (Project-requests)
JoeWalsh triaged T258301: Some image descriptions are not visible on dark mode as Medium priority.
Wed, Jul 22, 3:34 PM · Page Content Service, Product-Infrastructure-Team-Backlog
JoeWalsh triaged T258455: [BUG] mobile-html does not handle specific <p> with style properly as Low priority.
Wed, Jul 22, 3:33 PM · Page Content Service, Product-Infrastructure-Team-Backlog

Tue, Jul 21

JoeWalsh updated subscribers of T236731: Determine when enough iOS app users have upgraded to a version without mobile view to allow removal of the mobile view API.

@Jdlrobson in the last 30 days, 94% of sessions were on versions of the iOSapp that no longer require MobileView. @JMinor is there a particular % of sessions or some other metric you'd like to hit before we say this is resolved?

Tue, Jul 21, 3:43 PM · Wikipedia-iOS-App-Backlog
JoeWalsh updated the task description for T257944: [EPIC] Article description editing API.
Tue, Jul 21, 2:29 PM · Epic, Platform Engineering
JoeWalsh added a comment to T257944: [EPIC] Article description editing API.

@Tgr good point - If we go the PHP route, I don't think it needs to be an action API instead of a REST API. I'll update the description to clarify.

Tue, Jul 21, 2:25 PM · Epic, Platform Engineering
JoeWalsh updated the task description for T257944: [EPIC] Article description editing API.
Tue, Jul 21, 11:05 AM · Epic, Platform Engineering
JoeWalsh closed T258483: page/summary: Only return local short description overrides for enwiki, a subtask of T256817: [EPIC] Replace Wikidata descriptions on enwiki with local descriptions, as Declined.
Tue, Jul 21, 10:59 AM · Epic, Wikipedia-Android-App-Backlog (Android-app-release-v2.7.32x-Q-Qurabiya)
JoeWalsh closed T258483: page/summary: Only return local short description overrides for enwiki as Declined.

Declining this until there's a decision on T258485

Tue, Jul 21, 10:59 AM · Page Content Service, Product-Infrastructure-Team-Backlog
JoeWalsh added a subtask for T256817: [EPIC] Replace Wikidata descriptions on enwiki with local descriptions: T258485: On enwiki, action API requests with prop=description should only return the local description.
Tue, Jul 21, 10:50 AM · Epic, Wikipedia-Android-App-Backlog (Android-app-release-v2.7.32x-Q-Qurabiya)
JoeWalsh added a parent task for T258485: On enwiki, action API requests with prop=description should only return the local description: T256817: [EPIC] Replace Wikidata descriptions on enwiki with local descriptions.
Tue, Jul 21, 10:50 AM · Platform Engineering
JoeWalsh updated the task description for T258483: page/summary: Only return local short description overrides for enwiki.
Tue, Jul 21, 10:49 AM · Page Content Service, Product-Infrastructure-Team-Backlog
JoeWalsh created T258485: On enwiki, action API requests with prop=description should only return the local description.
Tue, Jul 21, 10:48 AM · Platform Engineering
JoeWalsh added a subtask for T256817: [EPIC] Replace Wikidata descriptions on enwiki with local descriptions: T258483: page/summary: Only return local short description overrides for enwiki.
Tue, Jul 21, 10:25 AM · Epic, Wikipedia-Android-App-Backlog (Android-app-release-v2.7.32x-Q-Qurabiya)
JoeWalsh added a parent task for T258483: page/summary: Only return local short description overrides for enwiki: T256817: [EPIC] Replace Wikidata descriptions on enwiki with local descriptions.
Tue, Jul 21, 10:25 AM · Page Content Service, Product-Infrastructure-Team-Backlog
JoeWalsh triaged T258483: page/summary: Only return local short description overrides for enwiki as High priority.
Tue, Jul 21, 10:24 AM · Page Content Service, Product-Infrastructure-Team-Backlog
JoeWalsh created T258483: page/summary: Only return local short description overrides for enwiki.
Tue, Jul 21, 10:24 AM · Page Content Service, Product-Infrastructure-Team-Backlog

Mon, Jul 20

JoeWalsh added a comment to T257921: Authors are removed from qqq.json file by Jenkins .

We re-export the strings from the source code, and our export doesn't preserve the comments. To fix, it would need to read the comments on the file before this line and re-add them to the newly written file.

Mon, Jul 20, 7:03 PM · translatewiki.net, Wikipedia-iOS-App-Backlog
JoeWalsh removed a project from T257767: All non-main namespace pages do not display title: Product-Infrastructure-Team-Backlog.

The app should be showing this page to mobile web in a webview, not mobile-html. Is this showing in mobile-html?

Mon, Jul 20, 6:47 PM · Wikipedia-Android-App-Backlog
JoeWalsh added a comment to T258295: Explore feed layout issue on iPad when navigating back from article.

Might need to trigger a recalculation of the collection view layout on viewWillAppear

Mon, Jul 20, 6:33 PM · iOS-app-v6.7.1-Bonefish-On-A-Fishing-Boat, Wikipedia-iOS-App-Backlog
JoeWalsh updated subscribers of T257860: Update PageviewDefinition to only include /api/rest_v1/page/mobile-html requests with X-Analytics: pageview=1 in pageviews.

Sounds good - @Charlotte, @JMinor, and @SNowick_WMF can help determine when we should switch to only counting pageview=1 requests. It might be feasible to make this switch fairly soon instead of trying to separate out by version. For code review, our tech leads @Tsevener (iOS) and @Dbrant (Android) can help

Mon, Jul 20, 4:42 PM · Patch-For-Review, Analytics-Kanban, Analytics
JoeWalsh updated the task description for T257860: Update PageviewDefinition to only include /api/rest_v1/page/mobile-html requests with X-Analytics: pageview=1 in pageviews.
Mon, Jul 20, 4:39 PM · Patch-For-Review, Analytics-Kanban, Analytics

Fri, Jul 17

JoeWalsh added a comment to T258210: iOS14 [Bug]: Fix top crash.

It looks like the crash is on webView.becomeFirstResponder() in ArticleViewController+ArticleWebMessageHandling. It was added in this commit but I don't recall why. Removing it doesn't seem to affect the W icon popover display. It's possible we can remove that line entirely.

Fri, Jul 17, 12:42 PM · iOS-app-v6.7-Bonefish-On-A-Bus, Wikipedia-iOS-App-Backlog

Thu, Jul 16

JoeWalsh moved T256069: [Bug] Table images disappear on scroll from Bug Backlog to Tracking on the Wikipedia-iOS-App-Backlog board.
Thu, Jul 16, 5:57 PM · MinervaNeue (Tracking), Reading-Web-Local-Wiki-Issues, Wikipedia-iOS-App-Backlog, iOS-app-Bugs
JoeWalsh added projects to T256069: [Bug] Table images disappear on scroll: MinervaNeue, Readers-Web-Backlog.

In mobile-html, removing this rule from base.css fixes it:

Thu, Jul 16, 5:56 PM · MinervaNeue (Tracking), Reading-Web-Local-Wiki-Issues, Wikipedia-iOS-App-Backlog, iOS-app-Bugs

Tue, Jul 14

JoeWalsh added a comment to T231987: iOS 13 Core Data value transformer deprecation.

@JMinor would it be OK if to fix this issue we reset everyone's explore feed cache? So instead of having up to 30 days of explore feed history cached, when users upgrade to 6.7, they would have to reload from the network. It would simplify things a lot to not have to do a migration from the legacy format to the new one for the explore feed

Tue, Jul 14, 7:24 PM · iOS-app-v6.7-Bonefish-On-A-Bus, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh updated the task description for T257944: [EPIC] Article description editing API.
Tue, Jul 14, 5:08 PM · Epic, Platform Engineering
JoeWalsh updated the task description for T257944: [EPIC] Article description editing API.
Tue, Jul 14, 5:04 PM · Epic, Platform Engineering
JoeWalsh added a subtask for T256817: [EPIC] Replace Wikidata descriptions on enwiki with local descriptions: T257944: [EPIC] Article description editing API.
Tue, Jul 14, 4:50 PM · Epic, Wikipedia-Android-App-Backlog (Android-app-release-v2.7.32x-Q-Qurabiya)
JoeWalsh added a parent task for T257944: [EPIC] Article description editing API: T256817: [EPIC] Replace Wikidata descriptions on enwiki with local descriptions.
Tue, Jul 14, 4:50 PM · Epic, Platform Engineering
JoeWalsh created T257944: [EPIC] Article description editing API.
Tue, Jul 14, 4:49 PM · Epic, Platform Engineering
JoeWalsh added a comment to T255475: Split box is not displayed within the Wikipedia iOS app.

It seems like this would be resolved by displaying the page issues as is requested in T248123. It's worth holding this until after T248123 is implemented to verify.

Tue, Jul 14, 4:00 PM · Page Content Service, Product-Infrastructure-Team-Backlog, Wikipedia-iOS-App-Backlog
JoeWalsh moved T256277: mobile-html: revisit 'notheme' exclusion and investigate alternatives for theming/theme exclusion from Doing to Code Review on the Product-Infrastructure-Team-Backlog (Kanban) board.
Tue, Jul 14, 3:36 PM · Product-Infrastructure-Team-Backlog (Kanban), Page Content Service

Mon, Jul 13

JoeWalsh updated the task description for T257227: Meta tag displayed in Japanese article infobox.
Mon, Jul 13, 6:56 PM · Parsoid-Rendering, Parsoid, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh updated the task description for T257227: Meta tag displayed in Japanese article infobox.
Mon, Jul 13, 6:56 PM · Parsoid-Rendering, Parsoid, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh edited projects for T257227: Meta tag displayed in Japanese article infobox, added: Parsoid; removed Product-Infrastructure-Team-Backlog.
Mon, Jul 13, 6:55 PM · Parsoid-Rendering, Parsoid, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh added a comment to T257227: Meta tag displayed in Japanese article infobox.

It looks like the tag exists in the Parsoid output

Mon, Jul 13, 6:54 PM · Parsoid-Rendering, Parsoid, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh added a subtask for T256508: API pageview counts for 'Mobile app' are incorrect since switch to mobile-html : T257860: Update PageviewDefinition to only include /api/rest_v1/page/mobile-html requests with X-Analytics: pageview=1 in pageviews.
Mon, Jul 13, 5:05 PM · Analytics-Kanban, Epic, Analytics, Product-Analytics
JoeWalsh added a parent task for T257860: Update PageviewDefinition to only include /api/rest_v1/page/mobile-html requests with X-Analytics: pageview=1 in pageviews: T256508: API pageview counts for 'Mobile app' are incorrect since switch to mobile-html .
Mon, Jul 13, 5:05 PM · Patch-For-Review, Analytics-Kanban, Analytics
JoeWalsh updated the task description for T257860: Update PageviewDefinition to only include /api/rest_v1/page/mobile-html requests with X-Analytics: pageview=1 in pageviews.
Mon, Jul 13, 4:49 PM · Patch-For-Review, Analytics-Kanban, Analytics
JoeWalsh created T257860: Update PageviewDefinition to only include /api/rest_v1/page/mobile-html requests with X-Analytics: pageview=1 in pageviews.
Mon, Jul 13, 4:49 PM · Patch-For-Review, Analytics-Kanban, Analytics
JoeWalsh closed T251352: change edit pencil link color for dark mode, a subtask of T252904: Issues with dark mode / non-white reading themes, as Resolved.
Mon, Jul 13, 3:41 PM · Wikipedia-iOS-App-Backlog
JoeWalsh closed T251352: change edit pencil link color for dark mode, a subtask of T242175: mobile-html: optimize accessibility, as Resolved.
Mon, Jul 13, 3:41 PM · Product-Infrastructure-Team-Backlog, Page Content Service
JoeWalsh closed T251352: change edit pencil link color for dark mode as Resolved.
Mon, Jul 13, 3:41 PM · Product-Infrastructure-Team-Backlog, Patch-For-Review, Page Content Service
JoeWalsh closed T251227: Change base10 color definition from #222 to #202122 as Resolved.

This was resolved as part of the theme work in this patch: https://gerrit.wikimedia.org/r/c/mediawiki/services/mobileapps/+/604823

Mon, Jul 13, 3:40 PM · Wikipedia-iOS-App-Backlog (Design), Product-Infrastructure-Team-Backlog, Wikipedia-Android-App-Backlog

Fri, Jul 10

JoeWalsh added a subtask for T256508: API pageview counts for 'Mobile app' are incorrect since switch to mobile-html : T256507: Add `X-Analytics: pageview=1` header to page view requests.
Fri, Jul 10, 1:51 PM · Analytics-Kanban, Epic, Analytics, Product-Analytics
JoeWalsh added a parent task for T256507: Add `X-Analytics: pageview=1` header to page view requests: T256508: API pageview counts for 'Mobile app' are incorrect since switch to mobile-html .
Fri, Jul 10, 1:51 PM · iOS-app-v6.7-Bonefish-On-A-Bus, Wikipedia-Android-App-Backlog (Android-app-release-v2.7.31x-P-Pryanik), Wikipedia-iOS-App-Backlog
JoeWalsh updated subscribers of T256507: Add `X-Analytics: pageview=1` header to page view requests.

@Charlotte @JMinor @SNowick_WMF the Android app release that added the X-Analytics: pageview=1 header added it to all mobile-html requests, not just when the page is loaded into an article view. So when a user syncs reading lists, or saves an article for offline, the header is included. This is how pageviews were counted previously. I assumed that in adding the header, we would limit it to requests made by the article view when the user views a page. That was the intent of this ticket, apologies if that wasn't clear. The iOS version we have staged for release with this fix only adds it when the user loads the page in an article view, not with offline requests. We should resolve this discrepancy before the iOS app goes out.

Fri, Jul 10, 1:50 PM · iOS-app-v6.7-Bonefish-On-A-Bus, Wikipedia-Android-App-Backlog (Android-app-release-v2.7.31x-P-Pryanik), Wikipedia-iOS-App-Backlog

Thu, Jul 9

JoeWalsh updated the task description for T257461: Multiple duplicate mobile-html requests from individual iOS app clients.
Thu, Jul 9, 10:55 PM · iOS-app-v6.7-Bonefish-On-A-Bus, Wikipedia-iOS-App-Backlog
JoeWalsh added a comment to T256516: Re-process webrequests from 2020-05-18 so that page views from latest Wikipedia app releases are counted.

That makes sense, thanks so much for pulling the data! We'll be able to use this to ensure we have a complete fix for the problem.

Thu, Jul 9, 10:53 PM · Analytics, Product-Analytics
JoeWalsh triaged T257389: User-Agent sent by the iOS app to mobile-html is incorrect as High priority.
Thu, Jul 9, 4:59 PM · iOS-app-v6.7-Bonefish-On-A-Bus, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh triaged T257461: Multiple duplicate mobile-html requests from individual iOS app clients as High priority.
Thu, Jul 9, 4:58 PM · iOS-app-v6.7-Bonefish-On-A-Bus, Wikipedia-iOS-App-Backlog
JoeWalsh updated the task description for T257461: Multiple duplicate mobile-html requests from individual iOS app clients.
Thu, Jul 9, 4:56 PM · iOS-app-v6.7-Bonefish-On-A-Bus, Wikipedia-iOS-App-Backlog

Wed, Jul 8

JoeWalsh updated the task description for T257082: mobile-html: mw:leadImage meta tag removes language information from SVG, causing the wrong language to be displayed in an SVG lead image in the apps.
Wed, Jul 8, 3:51 PM · Page Content Service, Product-Infrastructure-Team-Backlog, Wikipedia-Android-App-Backlog
JoeWalsh renamed T257082: mobile-html: mw:leadImage meta tag removes language information from SVG, causing the wrong language to be displayed in an SVG lead image in the apps from SVG language not matching with interface language to mobile-html: mw:leadImage meta tag removes language information from SVG, causing the wrong language to be displayed in an SVG lead image in the apps.
Wed, Jul 8, 3:49 PM · Page Content Service, Product-Infrastructure-Team-Backlog, Wikipedia-Android-App-Backlog
JoeWalsh created T257461: Multiple duplicate mobile-html requests from individual iOS app clients.
Wed, Jul 8, 2:35 PM · iOS-app-v6.7-Bonefish-On-A-Bus, Wikipedia-iOS-App-Backlog
JoeWalsh added a comment to T256516: Re-process webrequests from 2020-05-18 so that page views from latest Wikipedia app releases are counted.

Thank you for flagging these issues before re-processing the data and for providing the additional context and evidence.

Wed, Jul 8, 10:51 AM · Analytics, Product-Analytics
JoeWalsh renamed T257389: User-Agent sent by the iOS app to mobile-html is incorrect from User-Agent sent by the app to mobile-html is incorrect to User-Agent sent by the iOS app to mobile-html is incorrect.
Wed, Jul 8, 1:03 AM · iOS-app-v6.7-Bonefish-On-A-Bus, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh added a parent task for T257389: User-Agent sent by the iOS app to mobile-html is incorrect: T256508: API pageview counts for 'Mobile app' are incorrect since switch to mobile-html .
Wed, Jul 8, 1:03 AM · iOS-app-v6.7-Bonefish-On-A-Bus, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh added a subtask for T256508: API pageview counts for 'Mobile app' are incorrect since switch to mobile-html : T257389: User-Agent sent by the iOS app to mobile-html is incorrect.
Wed, Jul 8, 1:02 AM · Analytics-Kanban, Epic, Analytics, Product-Analytics
JoeWalsh updated the task description for T256508: API pageview counts for 'Mobile app' are incorrect since switch to mobile-html .
Wed, Jul 8, 1:02 AM · Analytics-Kanban, Epic, Analytics, Product-Analytics
JoeWalsh updated the task description for T257389: User-Agent sent by the iOS app to mobile-html is incorrect.
Wed, Jul 8, 12:10 AM · iOS-app-v6.7-Bonefish-On-A-Bus, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh updated the task description for T257389: User-Agent sent by the iOS app to mobile-html is incorrect.
Wed, Jul 8, 12:10 AM · iOS-app-v6.7-Bonefish-On-A-Bus, iOS-app-Bugs, Wikipedia-iOS-App-Backlog

Tue, Jul 7

JoeWalsh added a comment to T256516: Re-process webrequests from 2020-05-18 so that page views from latest Wikipedia app releases are counted.

There's at least one bug with the updated definition and it's my fault. The iOS app is sending the wrong User-Agent on pageview requests. When reviewing the patch that updated the server-side definition, I verified the User-Agent matched, but only before the request was handed off to the system web view. The web view changes the User-Agent to it's internal one, which is something along the lines of:
Mozilla/5.0 (iPhone; CPU iPhone OS 13_5 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Mobile/15E148
And not the app default of:
WikipediaApp/6.6.1.0 (iOS 12.4; Phone)

Tue, Jul 7, 11:57 PM · Analytics, Product-Analytics
JoeWalsh created T257389: User-Agent sent by the iOS app to mobile-html is incorrect.
Tue, Jul 7, 11:54 PM · iOS-app-v6.7-Bonefish-On-A-Bus, iOS-app-Bugs, Wikipedia-iOS-App-Backlog
JoeWalsh added a comment to T255579: mobile-html: title should be encoded to call the /page/related API.

Works for me

Tue, Jul 7, 5:14 PM · Patch-For-Review, Wikipedia-Android-App-Backlog, Product-Infrastructure-Team-Backlog (Kanban), Page Content Service
JoeWalsh added a comment to T256508: API pageview counts for 'Mobile app' are incorrect since switch to mobile-html .

@Nuria sounds good, I'd happy to talk about that. I set up a meeting to discuss.

Tue, Jul 7, 10:31 AM · Analytics-Kanban, Epic, Analytics, Product-Analytics

Jul 1 2020

JoeWalsh triaged T256894: [Bug] mobile-html: mapframe links don't work as Medium priority.
Jul 1 2020, 5:27 PM · Page Content Service, Product-Infrastructure-Team-Backlog
JoeWalsh updated the task description for T256894: [Bug] mobile-html: mapframe links don't work.
Jul 1 2020, 4:41 PM · Page Content Service, Product-Infrastructure-Team-Backlog
JoeWalsh created T256894: [Bug] mobile-html: mapframe links don't work.
Jul 1 2020, 4:41 PM · Page Content Service, Product-Infrastructure-Team-Backlog
JoeWalsh moved T256884: Content-type version mismatch is disabling caching of mobile-html-offline-resources responses from Code Review to To Deploy on the Product-Infrastructure-Team-Backlog (Kanban) board.
Jul 1 2020, 4:07 PM · Product-Infrastructure-Team-Backlog (Kanban), Page Content Service
JoeWalsh moved T254490: Consider increasing s-maxage for /page/mobile-html-offline-resources from Code Review to To Deploy on the Product-Infrastructure-Team-Backlog (Kanban) board.
Jul 1 2020, 4:07 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Page Content Service
JoeWalsh added a comment to T254490: Consider increasing s-maxage for /page/mobile-html-offline-resources.

I think unless the cache duplication is causing an issue, we should keep the contract of the API and go with option (1). We have plans to lazy-load content, which would require those pieces of content to exist in the offline-resources endpoint

Jul 1 2020, 4:06 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Page Content Service
JoeWalsh moved T256277: mobile-html: revisit 'notheme' exclusion and investigate alternatives for theming/theme exclusion from To Do to Doing on the Product-Infrastructure-Team-Backlog (Kanban) board.
Jul 1 2020, 3:45 PM · Product-Infrastructure-Team-Backlog (Kanban), Page Content Service