Hacker News API Wrapper for Node

by chasef • 12 years ago

5 5 12 years ago

chasef2• 12 years ago

Just started whipping this up. Would love if people wanted to help contribute to make it awesome. Thanks!

buddy0815• 12 years ago

Nice!

jbraithwaite2• 12 years ago

Your requests are not async and so the item will always be `undefined`. You should add a callback to your functions.

    hn.item(8863, function(err, item){
        console.log(item);
    });

Edit: Actually, there is a lot that isn't correct about this library.. you should add a disclaimer that this is a WIP and is not in a usable state.

chasef2• 12 years ago

Yes WIP (just started throwing it together now), not stable yet. I'll add that. Thanks.

A lot still needs to be fixed and done. Hoping this is a good way to get people involved. Would love any contributions you might want to make.

UPDATE: Made some updates. Give it a look now.

jbraithwaite2• 12 years ago

Much better. I would also suggest you check out this library for ideas on how to better structure your code:

(This isn't my library)

Related Stories