Remember that Android users will also see this - Android has far fewer users than iOS currently using this. But message should be generic enough to work for iOS and Android users, and include a mailto link to give us feedback if they can't upgrade.

As of now, planning on inserting a section at the top of every article regarding this API - explaining it will be turned off starting on [insert date], please upgrade your app, and to get in touch w/ us (via a provided email address) if you have concerns. The nice thing is this works for both iOS and Android.

How are we going to handle translation for this? Something we need to consider.

Also adding @Johan to this ticket, so he's aware.

Assuming inside the API endpoint response is where the message will be rendered, we can use the PHP msg function and translation can happen via translate wiki. Can help provide a code sample if that's helpful.

@JMinor any updates here? I'd really like to get this code removed before June. Sam has now left my team so it's really just me maintaining it and I have no interest in doing that. I am happy to help code review or provide the patch to do this, if someone from apps team is willing to test it.

Hey sorry for the slow follow through. We have a design and plan but
haven't prioritized the implementation. Let me check on capacity and get
back to you with timing or plan for getting it done.

Thanks Josh!

While looking into this, I found that PCS uses the MobileView API for all requests for Chinese Wikipedia (zh.wikipedia.org). I'm not sure the history of this, but it complicates the retirement of this system. I'm adding Content Transformers to this ticket, as we're going to need their support to figure out how to remove this dependency.

From mobile-html.js, line 87 (router.get('/page/mobile-html/:title/:revision?/:tid?', (req, res)):

if (!mobileviewHtml.shouldUseMobileview(req)) {
			return getMobileHtmlFromParsoid(req, res);
		} else {
			return getMobileHtmlFromMobileview(req, res);
		}

and

function shouldUseMobileview(req) {
	return wikiLanguage.getLanguageCode(req.params.domain) === 'zh';
}

The code has another path that uses Mobileview for main pages, but that code is only called from the /mobile-sections/ API, which doesn’t seem to be used by Android or iOS anymore.

@JMinor @JTannerWMF @schoenbaechler @OTichonova @cmadeo - I've started a mediawiki page that this message will link to. Please feel free to edit it or build it out: https://www.mediawiki.org/wiki/Wikimedia_Apps/Sunsetting_MobileView

Thanks @MattCleinman

Change 762545 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/extensions/MobileFrontend@master] Show a warning message to clients using the old mobile view API

https://gerrit.wikimedia.org/r/762545

Test wiki created on Patch demo by MCleinman (WMF) using patch(es) linked to this task:

https://patchdemo.wmflabs.org/wikis/7cb36f9e5e/w/

I finally got an old version of the app to compile and send just some requests to the patch demo server. (If all requests go there, the iOS app breaks in horrible ways.)

This is what we see:

The app does some work to hoist the move paragraph of content above Infoboxes, so it makes sense that it also pushes the first paragraph content above this message. I think that's okay. However, the only CSS that the app has (at least, for this old version of the app) for warningbox gives some padding. Since this is a temporary thing that is about to be killed, maybe we could manually add some background color here in the HTML we're adding in? (Also open to better ways, but I think a quick and dirty solution is probably fine here.) Maybe something like medium orange? (We need it to be readable on dark and light themes.)

Since this is a temporary thing that is about to be killed, maybe we could manually add some background color here in the HTML we're adding in?

The updated patch adds some inline styles to do exactly this. Can you take a look?

Test wiki created on Patch demo by MCleinman (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/bf0a905b9b/w/

Test wiki created on Patch demo by MCleinman (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/bf0c890466/w/

Looks great to me, thanks @Jdlrobson!

This is screenshots of the message. It appears after the first paragraph of the intro. (Ignore junk on top - that's from the test wiki.) @JMinor / @JTannerWMF does this look okay to you?

Yes, looks good. Thanks for picking this up and sticking with it @MattCleinman

Thanks @JMinor ! Just wanting to check in on the Android side:
@JTannerWMF @Dbrant does Android have any older clients that use the MobileView API that would see these banners?

On Android the number of installs that still use mobileview is virtually nonexistent. Even the oldest versions of the app that aren't using the modern mobile-html (around 0.7% of installs) are using the mobile-sections-lead and mobile-sections-remaining endpoints, which are parsoid-based. (Matt correctly noted that those endpoints do indeed touch mobileview in the case of Main pages, but that can also be removed and replaced by parsoid, since that was just for styling purposes.)

Nevertheless we will be prepared to field any support questions from any impacted users, should they come up.

We're still waiting for this to be deployed to production, right? Is there anything we can do to assist with it's deployment?

I figured this was blocked on T236733: mobile-html: Remove mobile view API dependency when parsoid supports language variants on zhwiki

Otherwise chinese apps would show banner for all page views.

Am I mistaken?

No, you're 100% correct. Thanks, total Friday brainfart over here.

Change 762545 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@master] Show a warning message to clients using the old mobile view API

https://gerrit.wikimedia.org/r/762545

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Monterey
Browser: Chrome
Device: MBP
Emulated Device:NA

Test Artifact(s):

QA Steps

Visit https://en.wikipedia.beta.wmflabs.org/wiki/Special:ApiSandbox#action=mobileview&format=json&page=Conflict-title-0.7231249569571601-I%C3%B1t%C3%ABrn%C3%A2ti%C3%B4n%C3%A0liz%C3%A6ti%C3%B8n&sections=all&prop=text%7Csections
✅ AC1: Confirm the message "Please upgrade your app." appears in the response

Test wiki on Patch demo by MCleinman (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/bf0a905b9b/w/

Test wiki on Patch demo by MCleinman (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/bf0c890466/w/

Test wiki on Patch demo by MCleinman (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/7cb36f9e5e/w/