ngLocale es-AR

[BartoGabriel]

I am Argentine. Our currency is not the Euro (https://github.com/angular/angular.js/blob/master/src/ngLocale/angular-locale_es-ar.js#L66)
Our currency symbol is "$". The symbol is placed before the number.

Example:
$ 14.212,82

[jeffbcross]

Thanks for pointing it out, want to submit a PR?

[caitp]

We still get these from Closure Library, https://github.com/google/closure-library/blob/master/closure/goog/i18n/numberformatsymbolsext.js#L3938 (they are getting them from CLDR)

But, since we don't update the locales each release, a PR wouldn't hurt

[lgalfaso]

It looks like what is needed is also to include the extended number formats, this is

diff --git a/i18n/update-closure.sh b/i18n/update-closure.sh
index 4a9290d..576926c 100755
--- a/i18n/update-closure.sh
+++ b/i18n/update-closure.sh
@@ -14,4 +14,5 @@ curl "$I18N_BASE/currency.js" > closure/currencySymbols.js
 curl "$I18N_BASE/datetimesymbols.js" > closure/datetimeSymbols.js
 curl "$I18N_BASE/datetimesymbolsext.js" > closure/datetimeSymbolsExt.js
 curl "$I18N_BASE/numberformatsymbols.js" > closure/numberSymbols.js
+curl "$I18N_BASE/numberformatsymbolsext.js" >> closure/numberSymbols.js
 curl "$I18N_BASE/pluralrules.js" > closure/pluralRules.js

then update and generate the locales. I am not 100% happy with the change as this will also add ~100 new locales, but it is still better than having the wrong information in the locales we already ship

@caitp @jeffbcross WDYT?

[caitp]

there's wrong information in closure's numberSymbolsExt actually, so I'm not sure how that would really help. They don't supply the currency symbol for a lot of those latin countries, which is why we're getting issues about "argentina does not use the euro!" and stuff (because it just inherits from the default 'ES' locale).

Honestly we really just need to ditch closure and go with cldr directly, but I haven't been working on that for a few weeks, work is stale