Page MenuHomePhabricator

ContentTranslation: Deprecate wgContentTranslationRESTBase config
Closed, ResolvedPublic

Description

Since we can now user Parsoid directly (ie $wgVirtualRestConfig['modules']['parsoid']), we can deprecate usage of wgContentTranslationRESTBase.

sx.wmflabs.org is using new setup for example.

Current usages

Not done

Event Timeline

Change 634956 had a related patch set uploaded (by KartikMistry; owner: KartikMistry):
[operations/mediawiki-config@master] WIP: Remove wgContentTranslationRESTBase config

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

We can remove wgContentTranslationRESTBase from CommonSettings.php. I believe it is already unused in production.

I think there is still value in keeping it in the CX extension. For development setups it is handy to be able to download articles from production. I think it should default to empty, and the code be changed to prefer that variable when present.

We can remove wgContentTranslationRESTBase from CommonSettings.php. I believe it is already unused in production.

I think there is still value in keeping it in the CX extension. For development setups it is handy to be able to download articles from production. I think it should default to empty, and the code be changed to prefer that variable when present.

Thanks. I'll update patch with notes.

Arrbee triaged this task as Medium priority.Nov 3 2020, 8:27 AM
Arrbee lowered the priority of this task from Medium to Low.
KartikMistry renamed this task from Deprecate use of wgContentTranslationRESTBase config to Deprecate wgContentTranslationRESTBase config.Nov 9 2020, 7:54 AM
KartikMistry updated the task description. (Show Details)

Change 634956 merged by jenkins-bot:
[operations/mediawiki-config@master] Remove wgContentTranslationRESTBase config

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

Mentioned in SAL (#wikimedia-operations) [2020-11-30T12:12:56Z] <urbanecm@deploy1001> Synchronized wmf-config/CommonSettings.php: 2922abe7b810f1b53b446af783dc9d51e6585225: Remove wgContentTranslationRESTBase config (T266213) (duration: 00m 57s)

KartikMistry renamed this task from Deprecate wgContentTranslationRESTBase config to ContentTranslation: Deprecate wgContentTranslationRESTBase config.Nov 30 2020, 12:32 PM
KartikMistry updated the task description. (Show Details)