search

openexchangerates / money.js

money.js is a tiny (1kb) javascript currency conversion library, for web & nodeJS
http://openexchangerates.github.io/money.js
MIT License
1.46k stars 127 forks source link

Introduce a new version of money.js #26

Closed ducdigital closed 8 years ago

ducdigital commented 8 years ago

This version is maintained here: https://github.com/ducdigital/money.js Feel free to give comment, add more tests and new feature in a PR.

In this version all API of money.js remain the same but introduce new features to this project.

Change Log

0.3.0

nsantiago2719 commented 8 years ago

I'm having a Uncaught ReferenceError: fx is not defined when I tried to implement this version. But if I rollback to v0.2 it fixes the problem. Don't know why though

ducdigital commented 8 years ago

Hi,

Can you redownload the files from here : https://github.com/ducdigital/money.js/blob/master/money.js

That error should no longer be anywhere in the code since I chaged the way it handle errors and the error should be clearer.

I suspect that the file you are using is an older file. Can you make sure that it is the latest from my repo?

Are you using it in frontend or backend? The frontend min file has been removed in my package and i suggest you use your favorite minifier to minify it if needed

On Tuesday, February 23, 2016, norman santiago <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

I'm having a Uncaught ReferenceError: fx is not defined when I tried to implement this version. But if I rollback to v0.2 it fixes the problem. Don't know why though

— Reply to this email directly or view it on GitHub https://github.com/openexchangerates/money.js/pull/26#issuecomment-187604836 .

nsantiago2719 commented 8 years ago

I re-download the file and even checked the source code rendered in the page. But the error is still there.