Closed glennfu closed 6 years ago
Haven’t been keeping up with the API. Thanks!
Np! I'm enjoying your library and happy to have found a spot to contribute.
Ok go ahead and release a new gem version tonight so others can benefit from your contribution.
Actually, just spotted something. Seems that in methods, you are using :price instead of :all_prices endpoint reference. New PR? :)
That's a good question. Rest api here:
Shows that the /v3/price endpoint with no arguments will return all prices. /v3/allPrices does not exist. However /v1/allPrices exists and seems to return the same thing as /v3/price. Which is preferable?
Hmm.... if /v3/price works for the new method, then I don't see a reason to have the /allPrices endpoint that isn't a dependency on any other methods. We should probably just remove it, unless it becomes relevant later on.
Is there something structurally different that needs to happen for it vs the regular singular /price endpoint where you pass params to it?
With this endpoint added, #21 can be solved with 2 api calls.