globalize icon indicating copy to clipboard operation
globalize copied to clipboard

Make the constructor case-insensitive

Open danielt998 opened this issue 1 year ago • 1 comments
trafficstars

If I run the following:

console.log("Globalize curr with currencyFormatter, ja-jp:" + Globalize("ja-jp").currencyFormatter("BGN", {
        compact: "short"
      })(112310));

      console.log("Globalize curr with currencyFormatter, ja-JP:" + Globalize("ja-JP").currencyFormatter("BGN", {
        compact: "short"
      })(112310));

I get

Globalize curr with currencyFormatter, ja:BGN 112K
Globalize curr with currencyFormatter, ja:BGN 11万

the latter of which is correct. I think the constructor should automatically capitalise the locale code correctly as this feels like something that could catch a lot of people out (I might be happy to raise a PR myself)

danielt998 avatar Apr 01 '24 09:04 danielt998

Also I think it would be helpful to emit a warning when a locale passed in is invalid, this is such an easy thing to miss

danielt998 avatar Apr 01 '24 09:04 danielt998