Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove FbtCommonConstants from babel-plugin-fbt #110

Closed
fruchtose opened this issue Nov 12, 2019 · 1 comment
Closed

Remove FbtCommonConstants from babel-plugin-fbt #110

fruchtose opened this issue Nov 12, 2019 · 1 comment
Assignees

Comments

@fruchtose
Copy link

馃殌 Feature Proposal

babel-plugin-fbt currently contains the following dictionary of Facebook-specific strings in the library, used for the common API: https://github.com/facebookincubator/fbt/blob/cf5743af96e48ce49cda9f3258c5d1102684f1b7/packages/babel-plugin-fbt/FbtCommonConstants.js

Developers should be able to register their own set of common strings, instead.

Motivation

This proposal relates to #108. For the common string API to be useful, developers need to be able to say which strings are constant.

Example

A developer could provide a JSON file which contains a dictionary of common strings. For instance, this might be part of the collectFBT command:

node ./node_modules/babel-plugin-fbt/bin/collectFBT --commonSrc ./config/i18n/commonStrings.json --manifest < .fbt/src_manifest.json > .fbt/source_strings.json

Pitch

This feature would be useful for the same reason #108 would be useful: it would empower teams to do more with FBT.

@jrwats jrwats self-assigned this Nov 20, 2019
@jrwats
Copy link
Contributor

jrwats commented Dec 2, 2019

26c5cd8

and published in: babel-plugin-fbt@0.12.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants