Page MenuHomePhabricator

[parent] Purge article after saving item edit
Closed, ResolvedPublic

Description

The API to use is action=purge. The page is specified via titles, not pageids or revids. From T227758#5555144, it sounds like we want to specify forcelinkupdate as well.

The progress bar continues to be shown until this is done.

An error in the purge API call may be tracked, but has no other effect (we don’t show it to the user, it doesn’t prevent the rest of the action to proceed).

Event Timeline

Tonina_Zhelyazkova_WMDE renamed this task from Purge article after saving item edit to [parent] Purge article after saving item edit.Mar 4 2020, 8:29 AM

This is done, except that error handling is currently a TODO comment, not tracking. Do we want to leave this task open (or add a subtask) for tracking purge errors?

We can decide on an approach during/after the daily (e.g. mw.log), create a task then.

Pablo-WMDE moved this task from To do to Doing on the Wikidata-Bridge-Sprint-15 board.

Discussed in the daily: will add a mw.track call in case of purge error. Maybe maybe a console error as well.

Change 578323 had a related patch set uploaded (by Pablo Grass (WMDE); owner: Pablo Grass (WMDE)):
[mediawiki/extensions/Wikibase@master] bridge: add test utility to mock the tracker service

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

Change 578324 had a related patch set uploaded (by Pablo Grass (WMDE); owner: Pablo Grass (WMDE)):
[mediawiki/extensions/Wikibase@master] bridge: track in case purging goes wrong

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

Change 578361 had a related patch set uploaded (by Pablo Grass (WMDE); owner: Pablo Grass (WMDE)):
[mediawiki/extensions/Wikibase@master] bridge: document way to subscribe to tracking events

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

Change 578323 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] bridge: add test utility to mock the tracker service

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

Moving this to done as what it describes is covered - Michael found one possible follow-up which I'll create a patch for.

Change 578324 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] bridge: track in case purging goes wrong

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

Change 578361 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] bridge: document way to subscribe to tracking events

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