php-translation / symfony-bundle

Symfony integration for Translations
MIT License
327 stars 93 forks source link

[Xliff] Allow "approving" translations #64

Open rvanlaak opened 7 years ago

rvanlaak commented 7 years ago

When translations are new, or when they have been edited lately, a workflow to approve them would help to ensure translation quality. The Xliff standard does support this out of the box, the bundle can help with that. Also see XliffExporter.

Scenario 1: new translation

Scenario 2: updating translation via edit in place or profiler

Scenario 3: updating translation web ui

Future enhancement could be to require a certain ROLE in order to approve a translation.

Nyholm commented 7 years ago

I like the ability of a support function. This is however only a xliff feature. But xliff is the standard =)

👍

To implement this we need to make changes to the CatalogueManager and the CatalougeMessage. If the translation is approved or not will go in the meta flag on the Common\Message so third party adapters may reach it

rvanlaak commented 7 years ago

If we can agree on the technical specs / changes, I'd like to help with the implementation right away 👍

One UI thing I mentioned already is that the web UI uses colors, but I think they are related to the completion percentage (?). These colors should change: gray or red is missing, green is approved, yellow is not approved yet.

Nyholm commented 7 years ago

Make many smaller PRs, they are easier to review and agree upon. Start with just adding this support to the CatalogueManager end CatalogueMessage.

Then you may make two more PRs, one for WebUI and one for EditInPlace.

rvanlaak commented 7 years ago

I first need to finish our migration to xliff, so I can probably start working on this in the beginning of next week.

Nyholm commented 7 years ago

FYI. Using symfony-storage 0.3.0 will allow storing "approved".

rvanlaak commented 7 years ago

@Nyholm any status update on the symfony-storage? Did it get any additional features that would be able to help us even better while implementing this?

Nyholm commented 6 years ago

Yeah. You've probably seen it already. But there is a Metadata class. If you use XLIFF then you will be able to save the metadata.