transloco icon indicating copy to clipboard operation
transloco copied to clipboard

feature(transloco): Use MessageFormatTranspiler with FunctionalTranspiler

Open MikeDabrowski opened this issue 1 year ago • 3 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Which Transloco package(s) will this feature affect?

MessageFormat

Is your feature request related to a problem? Please describe

Is there a way to combine the MessageFormatTranspiler and FunctionalTranspiler? Current impl of MessageFormatTranspiler extends DefaultTranspiler. I havent found any hint in the docs apart creating custom one myself.

Describe the solution you'd like

No response

Describe alternatives you've considered

CustomTranspiler extends FunctionalMessageFormatTranspiler extends FunctionalTranspiler.

I just started writing it, not sure if it will work.

Additional context

No response

I would like to make a pull request for this feature

Yes 🚀

MikeDabrowski avatar Sep 22 '23 08:09 MikeDabrowski

@MikeDabrowski I'll have to consider how since it wasn't raised before. I would love to hear your functional transpiler use case! I don't get much comments about it.

shaharkazaz avatar Sep 26 '23 07:09 shaharkazaz

We have an alert type component - just a rectangle with background and text we display to notify user of some important things.

That already has translation that uses markdown. Now UX has the idea that we need to show additionally a list of errors there. If But those errors each has to be translated as well. I thought maybe I could just append the translated errors for markdown to render them. But I thought the logic to do so would be best moved to such function.

I tried to merge Functional and MessageFormat ones but the fn didn't work.

In the end (we need this done by the end of this week) I just created new component and have two regular translations.

MikeDabrowski avatar Sep 26 '23 08:09 MikeDabrowski

We just got another feature where this fn would be useful - need to replace some parts of translations based on dynamic config.

MikeDabrowski avatar Oct 16 '23 11:10 MikeDabrowski