Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement OCEX module in OB #1193

Open
ap211unitech opened this issue Apr 15, 2024 · 0 comments
Open

Implement OCEX module in OB #1193

ap211unitech opened this issue Apr 15, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@ap211unitech
Copy link
Member

ap211unitech commented Apr 15, 2024

📝 Feedback Description

Currently, OCEX transactions are being handled inside OB itself, core logic is present in OB, which makes code complex and provides higher chances of bugs. We have already created OCEX module in polkadex-ts, so we will be replacing all the direct OCEX calls with polkadex-ts OCEX module.

In future, If anything changes in OCEX pallet, we will just need to update inside polkadex-ts only.

Tasks -

  1. createProxyAccount
  2. removeProxyAccount
  3. getProxies - (readStrategy/getTradingAddresses function)
  4. deposit (i.e. transfer from funding to trading account)
  5. transfer from one account to another for any asset (useAssetTransfer hook)
  6. Need to refactor useCall hook - code duplication is there
@ap211unitech ap211unitech self-assigned this Apr 15, 2024
@pavankanteti pavankanteti added the enhancement New feature or request label May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Backlog
Development

No branches or pull requests

2 participants