Hacker News API Wrapper for Node (opens in new tab)

(github.com)

5 pointschasef11y ago5 comments

5 comments

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.

chasefOP11y 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.

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)

chasefOP11y ago

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

Nice!

j / k navigate · click thread line to collapse

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.

chasefOP11y 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.

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)

chasefOP11y ago

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

Nice!

j / k navigate · click thread line to collapse