fvdm / nodejs-dnsimple Goto Github PK

• Restructure module
• Rewrite test script
• Rewrite readme
• Run tests
• Tag commits
• Create releases
• Move changelog to Github Releases

I'm trying to use your package with a new command line interface for dnsimple (http://ander.ly/1eSUh). However, I've found it doesn't work with the current version of the api.

The host name should now be api.dnsimple.com and the path should be prefixed with /v1/.

A couple of other issues I've run into are the callback signature doesn't follow the standard function(err, callback) convention.

For instance, instead of:

callback( result, status );

I'd like to change it to:

callback( status, result );

or:

callback( null, result );

and include status as a property on the result.

I have forked your repo and could submit these back as a PR, but didn't know if you were tied to that model.

Let me know.

Seriously, I don't get it.

lolwut

Hello,

I'm trying to use your module to check domain availability on my index.js page. However, I don't have much experience in node.js and I'm having hard time getting the the checker function to work.

My setup is running express as framework, sporting jade. This is what I have in the server-side so far:

var express = require('express')
  , routes = require('./routes')
  , dnsimple = require('./node_modules/dnsimple') 

....

// Configuration

dnsimple.api.email = 'j****@mail.com';
dnsimple.api.token = '*******************';

.....

So basically what I have tried to do is adding following into client side .jade file, inside script tags:

dnsimple.domains.show(domain);

where domain is post data of form. This however runs me into error saying "Uncaught ReferenceError: dnsimple is not defined ", which makes me confused where and how should I declare dnsimple function.

My current index.js looks like this:

exports.index = function(req, res, dnsimple){
  res.render('index', { title: 'Backnorder' })
};

I understand if you don't want to help a newbie, but any attempts to help would be really appreciated!

note to self

I'm considering to redesign the whole interface, making it all much much easier to code and maintain.

• Write test script in seperate branch: /Tests
• Connect to Travis CI
• Set up DNSimple Sandbox account
• Run successful basic test
• Add tests for individual methods

Part of v1 release (#5)

• Does the test script run?

Update:
In Windows 7 I needed to install Git for Windows and run the command below from git bash to at least install from the dev tree.

npm install git+https://github.com/fvdm/nodejs-dnsimple

First of all, thanks for publishing this. It is very helpful and saves me an hour or two. That is much appreciated.

However, please provide an update (with a significant new version) that changes the callback syntax from yours to the official node version, where the err is always the first parameter.

Thanks
Martin

Please wrap the API with config into an object so I can create multiple instances.

Thanks!