Page MenuHomePhabricator

Migrate a few projects to the new Validator framework
Open, NormalPublic

Description

With the new Validator framework being introduced we can start migrating a few projects to use it. Below are a handful of projects that have been chosen for initial migration,

  • iNaturalist
  • Dissem
  • Open Street Map - Website
  • WMCZ-Tracker

TODO: More to come

This process involves,

  • Migrating the existing checkers
  • Identifying any more checkers that need to be added
  • Migrating the Insertables
  • Deploying changes on Translatewiki.net
  • Getting feedback from the project

Another important goal is to have as less PHP code in the groups folder as possible.

After we've migrated the above, and taken feedback from the community we will work on migrating the rest of the projects.

Event Timeline

abi_ created this task.May 21 2019, 2:18 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 21 2019, 2:18 PM
abi_ updated the task description. (Show Details)May 23 2019, 12:39 PM
abi_ moved this task from Backlog to In Progress on the User-abi_ board.Jun 28 2019, 7:44 AM

Change 519783 had a related patch set uploaded (by Abijeet Patro; owner: Abijeet Patro):
[translatewiki@master] Adds validators for the Dissem papers and website

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

abi_ updated the task description. (Show Details)Jun 30 2019, 11:03 AM
Pginer-WMF triaged this task as Normal priority.Jul 1 2019, 10:50 AM
abi_ updated the task description. (Show Details)Jul 9 2019, 6:08 AM

Change 523725 had a related patch set uploaded (by Abijeet Patro; owner: Abijeet Patro):
[translatewiki@master] Adds validators for iNaturalist

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

While working on https://gerrit.wikimedia.org/r/c/translatewiki/+/524669 I observed two things:

  • Limiting the validator to only certain messages is not possible without PHP code. We should consider whether we can add a message key matching support.
  • Limiting insertables to only certain messages is not possible at all. Hence the MM|DD|YYYY validator is not currently showing them as insertables.

Change 519783 merged by jenkins-bot:
[translatewiki@master] Add validators for the Dissemin papers and website

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

abi_ updated the task description. (Show Details)Fri, Aug 30, 3:53 PM
abi_ added a comment.Sat, Aug 31, 5:28 AM

While working on https://gerrit.wikimedia.org/r/c/translatewiki/+/524669 I observed two things:

  • Limiting the validator to only certain messages is not possible without PHP code. We should consider whether we can add a message key matching support.
  • Limiting insertables to only certain messages is not possible at all. Hence the MM|DD|YYYY validator is not currently showing them as insertables.

I have created the task T231705: Limit validation to certain messages to track this.