search

deitch / jsorm-i18n

i18n library for JavaScript: calendar conversions, timezones, i18n resource bundles, currency rendering
20 stars 2 forks source link

Added support for BTC (8 decimal place formatting) #2

Closed cabbiepete closed 12 years ago

cabbiepete commented 12 years ago

Now use significantFigures to round and 0 pad decimal part of the string if required.

deitch commented 12 years ago

I had to redo the formatting slightly, and reran the tests for node and browser, but all looks good. I pushed it out.

Thanks for adding BTC!

cabbiepete commented 12 years ago

Awesome happy to help, wish I could do the same for the numberformat class we use in our PHP code lol.

Pete irc: cabbiepete

On 31 January 2012 14:19, Avi Deitcher < reply@reply.github.com

wrote:

I had to redo the formatting slightly, and reran the tests for node and browser, but all looks good. I pushed it out.

Thanks for adding BTC!

Reply to this email directly or view it on GitHub: https://github.com/deitch/jsorm-i18n/pull/2#issuecomment-3731731

deitch commented 12 years ago

LOL!

Too busy with my consulting dayjob, plus my business venture(s), plus the open-source libs I maintain or contribute, to start something in php.

Really, it was cool to add BTC.

Wonder if I could do a currency converter based off of xe.com...

On Tue, Jan 31, 2012 at 4:24 AM, Peter Simmons < reply@reply.github.com

wrote:

Awesome happy to help, wish I could do the same for the numberformat class we use in our PHP code lol.

Pete irc: cabbiepete

On 31 January 2012 14:19, Avi Deitcher < reply@reply.github.com

wrote:

I had to redo the formatting slightly, and reran the tests for node and browser, but all looks good. I pushed it out.

Thanks for adding BTC!

Reply to this email directly or view it on GitHub: https://github.com/deitch/jsorm-i18n/pull/2#issuecomment-3731731

Reply to this email directly or view it on GitHub: https://github.com/deitch/jsorm-i18n/pull/2#issuecomment-3732364