symfony-bundle icon indicating copy to clipboard operation
symfony-bundle copied to clipboard

[RFC] Deprecate extract, upload, and download commands before 1.0

Open welcoMattic opened this issue 3 years ago • 12 comments

As Symfony 5.3 brings Translation Providers, which are the native equivalent of Adapters here, I wonder to deprecate them and thus the associated commands (+ extract, which also could be merged with the Symfony one, see https://github.com/symfony/symfony/issues/41411).

After Adapters and commands removal, the bundle keeps a reason to live, because of the improved Translation Panel in Profiler will never be accepted in Symfony's core, and the edit-in-place feature which are both taking there place here, in an external bundle.

My point is to remove all duplicated features with Symfony from the bundle, and keeps what will belongs to the bundle only, then we could plan a 1.0 version.

Please @php-translation/deciders and @php-translation/members give me your thoughts about that :pray:

welcoMattic avatar Jun 18 '21 06:06 welcoMattic

I think it's a good idea. I don't think we want to duplicate the functionality, so, we can deprecate the code that was already implemented in Symfony, and later drop the deprecated code when we will drop support for Symfony <5.3 IMO

bocharsky-bw avatar Jun 18 '21 20:06 bocharsky-bw

Maybe create an overview / comparison of all functionality first? Or are the Symfony implementations functionally identical to the ones in this bundle?

rvanlaak avatar Jun 22 '21 08:06 rvanlaak

I became curious also how the translation functionality will evolve further. :thinking: Are there any chances to reduce confusion because of potentially different command implementations?

elfring avatar Jul 06 '21 06:07 elfring

@elfring which command implementation do you refer?

welcoMattic avatar Jul 06 '21 07:07 welcoMattic

Example: update or extract

elfring avatar Jul 06 '21 08:07 elfring

If you talk about commands in php-translation/symfony-bundle, there will no more commands here, we will consider Symfony's one as enough stable to use them. For update/extract command, see https://github.com/symfony/symfony/issues/41411

welcoMattic avatar Jul 06 '21 09:07 welcoMattic

Will any more extensions be reconsidered for this software area? :thinking:

elfring avatar Jul 06 '21 09:07 elfring

Like what? I suggest to remove all double features between this bundle and Symfony, but keeping what's existing only in the bundle (EditInPlace, Web UI, and improved translation Profiler panel)

welcoMattic avatar Jul 06 '21 09:07 welcoMattic

You point issues on Symfony, not on php-translation bundle, so I don't understand what you want to explain?

welcoMattic avatar Jul 06 '21 10:07 welcoMattic

I assumed (for example) that the bundled extract command was improved in comparison to the Symfony update command. How will the software evolve further here?

elfring avatar Jul 06 '21 11:07 elfring

We plan to move improvements of extract command of the bundle to Symfony, if it's possible. If not, we will keep the extract command in the bundle (maybe renamed, to avoid conflicts with the Symfony one)

welcoMattic avatar Jul 06 '21 12:07 welcoMattic