Skip to content

Commit

Permalink
Fixed similar issue with history calls silently dropping params
Browse files Browse the repository at this point in the history
  • Loading branch information
arvicco committed Mar 15, 2016
1 parent 873e853 commit 72e0cd5
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions lib/bitfinex/historical_data.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ module Bitfinex
module HistoricalDataClient

# View all of your balance ledger entries.
#
#
# @param currency [string] (optional) Specify the currency, default "USD"
# @param params :since [time] (optional) Return only the history after this timestamp.
# @param params :until [time] (optional) Return only the history before this timestamp.
Expand All @@ -12,13 +12,13 @@ module HistoricalDataClient
# @example:
# client.history
def history(currency="usd", params = {})
check_params(params, %i{since until limit wallet})
check_params(params, %i{since until limit wallet})
params.merge!({currency: currency})
authenticated_post("history", params).body
authenticated_post("history", params: params).body
end

# View your past deposits/withdrawals.
#
#
# @param currency [string] (optional) Specify the currency, default "USD"
# @param params :method (optional) The method of the deposit/withdrawal (can be “bitcoin”, “litecoin”, “darkcoin”, “wire”)
# @param params :since (optional) Return only the history after this timestamp
Expand All @@ -30,11 +30,11 @@ def history(currency="usd", params = {})
def movements(currency="usd", params = {})
check_params(params, %i{method since until limit})
params.merge!({currency: currency})
authenticated_post("history/movements", params).body
authenticated_post("history/movements", params: params).body
end

# View your past trades.
#
#
# @param symbol The pair traded (BTCUSD, LTCUSD, LTCBTC)
# @param params :until [time] (optional) Return only the history before this timestamp.
# @param params :timestamp [time] (optional) Trades made before this timestamp won’t be returned
Expand All @@ -47,7 +47,7 @@ def movements(currency="usd", params = {})
def mytrades(symbol, params = {})
check_params(params, %i{until limit_trades reverse timestamp})
params.merge!({symbol: symbol})
authenticated_post("mytrades", params).body
authenticated_post("mytrades", params: params).body
end
end
end

0 comments on commit 72e0cd5

Please sign in to comment.