Update the Wikipedia Zero exit interstitial to duplicate the functionality that was recently added to the web UI.
This includes:
Per-partner overrides of default text for Title, warning, optional button label and link.
Update the Wikipedia Zero exit interstitial to duplicate the functionality that was recently added to the web UI.
This includes:
Per-partner overrides of default text for Title, warning, optional button label and link.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Dbrant | T115484 Display definitions of highlighted words from Wiktionary | |||
| Resolved | • bearND | T126934 Roll out Restbase to production | |||
| Resolved | Dbrant | T120116 Better analytics for preloads and Wikipedia Zero support. | |||
| Resolved | • Mholloway | T123748 Add EventLogging in the Android App to track external link clicks per partner | |||
| Declined | Yurik | T115398 For Wikipedia Zero partners - Create ability to measure number of external link clicks per partner | |||
| Resolved | • Mholloway | T119126 Replicate custom Zero exit interstitial behavior in Android App | |||
| Resolved | • Mholloway | T123744 Update ApiZeroBanner to make custom partner info and exit warnings available to the apps |
@dr0ptp4kt @DFoy Can you link to the task(s) that represent this work as it was done on the Web side? (Or a link to the specification for how this data is received and formatted, as well as any design mocks)
Note to future self filling in details: point to some relevant files to aid in the coding task.
The design mocks have been shared. Subtask T115398 regarding the EL schema added for consult from data analyst. @Dbrant, @Mholloway: do you want a separate task for the actual EL wireup?
@dr0ptp4kt Sure, I can write one up and add as a blocker to T115398, free to edit as necessary.
Change 264859 had a related patch set uploaded (by Mholloway):
Add exit titles and warnings to API message response
@dr0ptp4kt We deferred on estimating this at the most recent Android estimation meeting because the title seems broad but the description is narrow. Does this task encompass just plugging in the strings soon to be available pending the API-side changes? Or is there broader work around testing the mobile web and emulating its behavior (for example, when switching wifi networks)?
We were on the fence about whether to pull this task into the current sprint but it's looking to me like we might have the bandwidth, at least under the narrow interpretation of the task.
@Mholloway, thanks for checking.
Yes, the client side code should apply a different dialog depending on the current config.
If the user has two SIM cards, both for zero-rated operators, but one operator has the non-customized config, yet the other has the customized config, should the user be using the app and switch between the two operators with SIM switching, the app should show the appropriate dialog for that user when she taps on a third party link (assuming the option in More... / settings is checked for warning about leaving, as usual).
Change 264859 merged by jenkins-bot:
Add exit titles and warnings to API message response
Relevant links for posterity:
https://www.mediawiki.org/wiki/Wikipedia_Zero
https://www.mediawiki.org/wiki/Wikipedia_Zero/Test_cases
Change 269548 had a related patch set uploaded (by Mholloway):
Add support for custom carrier exit interstitial messages
Change 269548 merged by jenkins-bot:
Add support for custom carrier exit interstitial messages
Let's not call this a win yet. I'm currently still working on some of the implementation of the new stuff, though the groundwork was laid in the patch just merged.
Sorry this has been so disjointed, but there have been kind of a lot of "gotchas."
On the state transition piece, I've verified that state transitions are made as expected across the following changes:
A couple of follow-up changes related to updating the Zero state when the app language changes are in progress:
T126815: [Bug] Going from a zero-rated site ("Wikipedia language") to a non-zero-rated site does not trigger Wikipedia Zero state change event
T126997: [Bug] Switching the Wikipedia language while Zero is enabled does not update the nav menu zero message
Change 271079 had a related patch set uploaded (by Mholloway):
Add bannerUrl to zero config API response
@Mholloway There is not enough information in this ticket for a third party to be able to test. Please add preconditions and steps to verify if you want this ticket tested by QA.