Skip to content
This repository was archived by the owner on Apr 14, 2021. It is now read-only.

latest version does not include local data #227

Closed
jonathanong opened this issue Sep 20, 2016 · 5 comments
Closed

latest version does not include local data #227

jonathanong opened this issue Sep 20, 2016 · 5 comments

Comments

@jonathanong
Copy link

jonathanong commented Sep 20, 2016

cause may be : #199 9ad9a12

because they are not included, this instruction no longer works: https://github.com/andyearnshaw/Intl.js#intljs-and-browserifywebpack

not sure if the issue is that the instructions are outdated or if this change was accidental

@jonathanong
Copy link
Author

oh i see. you are requiring the js file, whereas i'd prefer to just require the json.

@jonathanong
Copy link
Author

let me know if this is a "will not fix"

@bensontrent
Copy link

Make sure you're referencing the locale-data/jsonp/.... folder as found in the instructions rather than locale-data/json as the json folder was considered less dangerous to be removed as it was never officially documented.

@caridy
Copy link
Collaborator

caridy commented Sep 20, 2016

yeah, if you really need the json files you will have to build them! I'm more curious about why do you need the json files. I will close this one, feel free to reopen if you have a strong reason :)

@10xLaCroixDrinker
Copy link

@caridy keeping the JSON in the tar would allow a user to add locale data for a macro-language like no-NO using a combination of data for nn-NO and nb-NO

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

No branches or pull requests

4 participants