Page MenuHomePhabricator

Clean up of old dashboard code
Closed, ResolvedPublic

Description

After T386719: Deploy unified dashboard and T389970: Remove access to old dashboard and related code , the next steps involve:

  • An observation period of 1-2 weeks to check for any issues raised by wikis or inconsistent translation activity in the data.
  • If the above step passes, we will proceed to clean up all related code.

Event Timeline

PWaigi-WMF updated the task description. (Show Details)
PWaigi-WMF removed a project: Epic.
PWaigi-WMF moved this task from Backlog to Prioritized on the LPL Hypothesis board.
PWaigi-WMF moved this task from Prioritized to Incoming on the LPL Hypothesis board.

Change #1161034 had a related patch set uploaded (by Nik Gkountas; author: Nik Gkountas):

[mediawiki/extensions/ContentTranslation@master] CX: Remove "cxsuggestionlist" endpoint and usages

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

Change #1161035 had a related patch set uploaded (by Nik Gkountas; author: Nik Gkountas):

[mediawiki/extensions/ContentTranslation@master] CX: Remove old dashboard code

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

Change #1161034 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] CX: Remove "cxsuggestionlist" endpoint and usages

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

SBisson moved this task from Prioritized to In-progress on the LPL Hypothesis board.

@ngkountas Anything else left in this task?

Change #1161035 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] CX: Remove old dashboard code

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

Change #1164976 had a related patch set uploaded (by Nik Gkountas; author: Nik Gkountas):

[mediawiki/extensions/ContentTranslation@master] CX: Remove manage-lists.php script

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

Change #1164977 had a related patch set uploaded (by Nik Gkountas; author: Nik Gkountas):

[mediawiki/extensions/ContentTranslation@master] CX SuggestionListManager: Clean up unused methods

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

@SBisson I added another couple of patches that removes (very) old code, related to the dashboard. If we merge these patches, I think we can close this task as done.

Change #1164976 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] CX: Remove manage-lists.php script

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

Change #1164977 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] CX SuggestionListManager: Clean up unused methods

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

Nikerabbit moved this task from Needs QA to Product Signoff on the LPL Hypothesis board.

It looks like the cx-translator-header message should also have been removed with this task. I'll submit a small follow up and then we can resolve for good.

Change #1167877 had a related patch set uploaded (by Sbisson; author: Sbisson):

[mediawiki/extensions/ContentTranslation@master] Remove unused message cx-translator-header

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

Change #1167877 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] Remove unused message cx-translator-header

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

I believe this is now complete