Inconsistence in variable naming #57

Closed
bianjiang opened this Issue Jun 13, 2017 · 4 comments

Comments

Projects
None yet
3 participants
Contributor

bianjiang commented Jun 13, 2017

e.g.,

productId vs. product_id
orderId vs. order_id

A lot of places in the order are without the '_', while the GDAX api parameters are all with the underscore. This is somewhat confusing, especially the recent changes on the buy and sell function that takes raw GDAX api parameters.

Owner

danpaquin commented Jun 13, 2017

Agreed! This has been updated in PR #52 thanks to @acontry

Contributor

bianjiang commented Jun 13, 2017

Great, thanks!

Contributor

acontry commented Jun 13, 2017

I'm looking to implement more changes to clean up the API in the near future FYI. buy and sell need some work to make their usage clear.

Owner

danpaquin commented Jun 13, 2017

Agreed. Thanks to everyone for your support. I'm closing this issue out.

danpaquin closed this Jun 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment