Since the change is about Latin script, may I request testing with Vietnamese as well? Where latin script has its unique characteristics.

This is a good catch. I should not have used a title like "Proxima Centauri" for lot of testing because that has same title in source and target languages.

Can you confirm if you were logged in your mobile browser already? Also, what happens if you access the page from a private browsing session?

Need to add our tag to a valid list of tags for VE $wgWMEEditCampaigns as per T209132. Also to CX

This is due to nonloading our our javascript to that Desktop page due to limiting the corresponding RL module to 'mobile' only. It is addressed in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ExternalGuidance/+/490314

we don't have any tests yet to have any meaningful data. So I just added a phpunit test to begin with: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ExternalGuidance/+/490585

. What I was trying to add is for local wiki, distinguish create a new page when there is no page with the same title exist, vs expand the existing article when there exist a page with the same title,

The eg.wmflabs.org has no article. It is pretending that all these articles are present using a simulation trick in MobileFrontend($wgMFContentProviderClass). his page is rendered from server and it checks if that title in MW database of the instance. It is not. Hence it always says 'create new one'. To verify, try this: https://eg.wmflabs.org/w/index.php?title=Special:ExternalGuidance&from=en&to=id&page=Main_Page&service=Google

In T212414#4946442, @chelsyx wrote:

@santhosh Thanks so much for the event logging! https://meta.wikimedia.org/w/index.php?title=Schema:ExternalGuidance&oldid=18870706

I think it would be helpful if we can add the following information to the schema. What do you think?

• I think it would be helpful if we can distinguish whether user is create a new page, or edit existing page on the local wiki. To do that, we can 1) add a field named "type" with enum [new, existing] for all events (if possible), or 2) add an new action named "edit_existing"

Cause is same as T215606: Start an article action not working. Use the mobile version as explained in the comments there

This is due to our limitations in setting up a realistic environment. To get this working - use any of the following

1. use https://eg.wmflabs.org/wiki/Special:ExternalGuidance?from=en&to=id&page=Arròs+negre&service=Google
2. Access it from mobile or use mobile simulation from browser inpsect tool
3. Click on the "Mobile version" link at the bottom of the page.

@sbassett, @Reedy Are you planning to take another look on this? Or are we done? All the patches related to your comments are merged now. Thanks!

To my understanding, Graphite doesn't support aggregation (e.g. by month) on the dashboard.

In T212414#4930500, @chelsyx wrote:

• MediaWiki.ExternalGuidance.init.serviceName.fromLang.toLang is emitted when a page is requested by the translation service, correct?

In T214152#4925599, @sbassett wrote:

1. In SiteMapper::getPageURL, it looks like $title isn't being validated/sanitized and is just replacing $2 within //$1.wikipedia.org/wiki/$2. Given that Html::rawElement is used within specials/SpecialExternalGuidance.php and it's populating $sourcePage from the request via getVal(), I'm thinking there could be a potential for injection here, unless I'm just missing where this is being sent to the parser or sanitized, etc.

Thanks @cscott, @ssastry. The content inside the span tricked me, I should have checked the character inside it. In further testing, I found that the MT engine behavior is randomabout handling these inputs. Some of them changes nbsp to space, ndash to hyphen in random. I added a guard against these changes now.

@cscott Translation service is not required to reproduce the issue.

1. Give A B in https://en.wikipedia.org/api/rest_v1/#!/Transforms/post_transform_wikitext_to_html_title_revision. It gives

<p id="mwAg">A<span typeof="mw:Entity" id="mwAw"> </span>B</p>

2. Copy that and past into https://en.wikipedia.org/api/rest_v1/#!/Transforms/post_transform_html_to_wikitext_title_revision. It gives

A B

The original article has the following wikitext

auf dem sich u. a.

To help understanding what the project does,access our labs instance where this extension is deployed using google translate. You should see broken edit links, and page actions removed by a contribution invite banner. Which will take you to one of our wikis, explaining steps to contribute. (Currently that target wiki is same labs wiki due to constraints in testing)

In T214152#4892886, @Bawolff wrote:

Just wondering, is there a reason for the doing the new SiteMapper stuff, instead of using core's WikiMap (and related) class? (I haven't looked at WikiMap in a while, so the answer might just be that WikiMap really sucks)

There are multiple way to solve this. A special resource loader module or a new API are options. But it comes with the cost of extra fetches from server apart from RL module.

4 events are tracked as counters with the keys given below:

There is a javascript library http://github.com/wikimedia/mediawiki-html-sanitizer created based on parsoid's html sanitization code

Hi, For the ExternalGuidance project, we have a usecase where when a content is presented in external context such as Google translate, the search feature in mobile frontend should work. Currently when our content is presented in a foreign domain, sometimes under an iframe like google translate, the search feature in mobile frontend fails because of CORS. See T212314: Review additional elements exposed by external services that may not work

Bumping priority since this is blocking all CI of ExternalGuidance patches. Just need review from CI team.

Two items that need fix:

So far we hide all pen icons(edit links) and the top bar containing language selector, mark as favorite, download, and edit.