skylinezum / node-coinigy

Node wrapper for Coinigy's REST API
MIT License
18 stars 7 forks source link

Planning #2

Open awitherow opened 7 years ago

awitherow commented 7 years ago

I just signed up for Coinigy, and holy $"ยง&=) am I going to be staying. I want to utilise your library and help maintain it.

What is undone? Where do you need help? What are your ideas on where we can take this sdk? etc.

Let me know! ๐Ÿ‘

skylinezum commented 7 years ago

Hey, so currently everything works. And I've written some simple unit tests. I want to share with you some caveats that are not documented with coinigy from a user perspective. We can then discuss what you would like to help with.

Caveats

Now with that said. I still use them so here are the things I'm trying to do for the library and for getting features requested from coinigy.

Goals/Todo

Thanks for your interest in this little library I wrote super quickly :) Please let me know if there's anything you would like to add or have an interest in doing. I'll probably improve this library since I now know someone else is using it too.

awitherow commented 7 years ago

Dude this is awesome. It is nice to receive a well thought out and motivating reply ๐Ÿ™‡ ๐Ÿ™

The cancelall thing seems to be a pretty interesting one. I am chatting with one of the guys at coinigy and he was asking for some ideas. Perhaps we could try to get support/insights from their team and maybe even influence the creation of such an endpoint! Want me to write him about it?

I think the batch call helper function would be a really great addition... could be helpful as an open source contribution for the cancelall issue/feature request?

And the the quote about best/worst, I would like to take that out of the public and into more private. Could you please write me an email at au.witherow@gmail.com so we could chat about them a little more? I want to make sure the platform I am using is stable/secure/professional.

As you've written out, getting good test coverage is solid, as well as beefing up the documentation. I will attempt to get things running this week and dig through the code and write down observatory stuff in the readme and check out the error handling/tests.

On code structure, I will also give some feedback after looking through!

skylinezum commented 7 years ago

I think I will rewrite the functions to take an object as input instead of a long set of function params. so order won't matter and easier to handle optional keys.

awitherow commented 7 years ago

I am working on a project right now that is just using bittrex, kind of swallowing my time. Slow and steady wins the race though ๐Ÿข ping me in the PR about the object param.

christophe-hall commented 7 years ago

Guys, thanks for your work on this.

skylinezum commented 7 years ago

@awitherow + @christophe-hall

I've made breaking changes to take objects as input and clean up some other things. Let me know if everything is ok and if there's anything you'd like me to add.

williamkehl commented 7 years ago

Guys thank you so much for undertaking this, from the entire Coinigy dev team, we really appreciate it! I'm here to update you as we've added a new API call, "refreshOrders", which reaches out to the exchange to update/sync your open order status. It's identical to the "refreshBalance" call. Feel free to ping us on slack if you have any questions or issues at all!