Skip to content
This repository has been archived by the owner on Sep 27, 2022. It is now read-only.

Documentation seems to contain minor errors #255

Open
michael-spengler opened this issue Jan 21, 2021 · 1 comment
Open

Documentation seems to contain minor errors #255

michael-spengler opened this issue Jan 21, 2021 · 1 comment

Comments

@michael-spengler
Copy link

const result = await perpetual.api.getAccountBalances({
  account: '0x52bc44d5378309ee2abf1539bf71de1b7d7be3b5',
});

--> "account" shall be replaced by "accountOwner"

see https://docs.dydx.exchange/#perpetual-get-account-balances

If you let me know where I shall change such things, I can do so as a contributor as I like DyDx. If you would like to support the general DeFi Package, feel free to fix stuff like that: michael-spengler/decentralized-finance#7 (comment)

:)

@michael-spengler
Copy link
Author

michael-spengler commented Jan 21, 2021

@BrendanChou It would be cool, if we could provide role model usage examples instead of pseudo code like when it comes to order matching.

I can offer to provide that within the DeFi Package. It would be cool if you could link to the DeFi Package in return in order to foster a mutually supportive collaboration.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

1 participant