Skip to content

Commit f5319e4

Browse files
authored
fix: retrieve all the wallets in the account (#842)
The default page size of 50 records may miss the wallet with funds which results in 'Insufficient funds' error. Currently, Coinbase list around 70 wallets and using maximum page size of 250 avoid the funds error. However, this is a temporary fix; until they add more than 250 wallets. This is an edge case as well and the error is only seen when a user has selected all the wallets when creating an API keys.
1 parent 5be74cf commit f5319e4

File tree

1 file changed

+1
-1
lines changed
  • models/exchange/coinbase

1 file changed

+1
-1
lines changed

models/exchange/coinbase/api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ def get_accounts(self) -> pd.DataFrame:
117117

118118
# GET /api/v3/brokerage/accounts
119119
try:
120-
df = self.auth_api("GET", "api/v3/brokerage/accounts")
120+
df = self.auth_api("GET", "api/v3/brokerage/accounts", "limit=250")
121121
except Exception:
122122
return pd.DataFrame()
123123

0 commit comments

Comments
 (0)