Skip to content

Further development on the Margin Trading API#226

Open
acravenho wants to merge 3 commits into
ccxt:masterfrom
acravenho:ac-development
Open

Further development on the Margin Trading API#226
acravenho wants to merge 3 commits into
ccxt:masterfrom
acravenho:ac-development

Conversation

@acravenho

Copy link
Copy Markdown

Added the following endpoints:

  • margin trading order history
  • transfer
  • borrow (loan)
  • max borrow
  • repay
  • repay history
  • margin order history
  • margin assets
  • max transferable

@balthazar balthazar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you mind only pushing the new margin endpoint rather than style changes that conflict with my prettier config?

@bennycode

Copy link
Copy Markdown
Contributor

@acravenho can you allow maintainers to edit your PR? This would help us adjusting the Prettier styling and merging your PR.

@acravenho

Copy link
Copy Markdown
Author

@bennyn No problem at all. Sorry, I've been super busy.

@bennycode

Copy link
Copy Markdown
Contributor

@balthazar

Copy link
Copy Markdown
Collaborator

@bennyn I do not, but you could fork his branch and submit a new PR with your updates, so you don't have to wait on it

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants