Skip to content

feat(compiler): open the serializer API for i18n#14281

Closed
marclaval wants to merge 1 commit intoangular:masterfrom
marclaval:i18nCustomSerializer
Closed

feat(compiler): open the serializer API for i18n#14281
marclaval wants to merge 1 commit intoangular:masterfrom
marclaval:i18nCustomSerializer

Conversation

@marclaval
Copy link
Copy Markdown
Contributor

This PR opens the serializer API of i18n, in order to allow developers to do something like this:

  ./node_modules/.bin/ng-xi18n --i18nSerializer=./custom_serializer
 ./node_modules/.bin/ngc --i18nFile=src/messages.fr.json --locale=fr --i18nSerializer=./custom_serializer

The serializer is loaded with a require(), so the path can be relative, absolute or a dependency.

@marclaval marclaval force-pushed the i18nCustomSerializer branch 2 times, most recently from dd5a823 to f74e9c9 Compare February 3, 2017 13:59
@marclaval marclaval added area: i18n Issues related to localization and internationalization action: review The PR is still awaiting reviews from at least one requested reviewer labels Feb 3, 2017
@marclaval marclaval requested review from hansl and vicb February 3, 2017 14:36
@vicb
Copy link
Copy Markdown
Contributor

vicb commented Feb 3, 2017

I'm generally 👍 on this but I think it should be delayed as much as possible until there is a concrete use case because once the API is public we can not change it any more. Do you have such a use case ?

Also #14290 update the API to also return the bundle locale.

@mhevery
Copy link
Copy Markdown
Contributor

mhevery commented Feb 10, 2017

Needs rebase

@marclaval marclaval force-pushed the i18nCustomSerializer branch from f74e9c9 to f560e26 Compare February 13, 2017 10:56
@ocombe ocombe mentioned this pull request May 2, 2017
20 tasks
@vicb
Copy link
Copy Markdown
Contributor

vicb commented Aug 1, 2017

Closing for now as there is no urge to open this API

@vicb vicb closed this Aug 1, 2017
@marclaval marclaval deleted the i18nCustomSerializer branch December 11, 2017 10:12
@marclaval marclaval restored the i18nCustomSerializer branch December 11, 2017 10:13
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer area: i18n Issues related to localization and internationalization cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants