-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TypeError: Cannot read property 'NA' of undefined #6
Comments
I've had a gander at the code. I can see that the missing information is in the |
changing the constructor to something like this makes it work: module.exports = class {
constructor(cfg = {}) {
this.config = Object.assign(require('./config/default.json'), cfg);
} |
also missing from the documentation is the fact that the client initialisation needs to be given credentials. something like this: const client = new Client(require('./config/sandbox.json')) with a little modification to the code it could be called without a parameter. my recommendation would be that for niladic calls the code look in |
if the above is acceptable I'm happy to submit a PR |
as a nice enhancement, I would make the call to |
also, I don't know if it's possible to create a USD card for Japan. probably not. in that case we should codify the currencies for the countries involved so the |
see 'Sundry fixes' PR |
followed the simple instructions in the README. I have a file with the sample code and when I run it I get the following:
what am I missing?
The text was updated successfully, but these errors were encountered: