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.

Sign up for GitHub

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

Jump to bottom

P2.2: Direct Withdrawal of Native Tokens #956

Open
zktony opened this issue May 8, 2024 · 0 comments · Fixed by #959
Open

P2.2: Direct Withdrawal of Native Tokens #956

zktony opened this issue May 8, 2024 · 0 comments · Fixed by #959
Assignees
@Gauthamastro

Comments

@zktony
Copy link
Contributor

zktony commented May 8, 2024

Direct Withdrawal of Native Tokens:

  • Implement functionality for direct withdrawal of native tokens from the Orderbook to their respective native parachain.
  • Allow direct withdrawal of native tokens from the Orderbook to non-native parachains (e.g., withdrawing DOT to Astar).
@pavankanteti pavankanteti changed the title Direct Withdrawal of Native Tokens P2.2 - Direct Withdrawal of Native Tokens May 8, 2024
@pavankanteti pavankanteti changed the title P2.2 - Direct Withdrawal of Native Tokens P2.2: Direct Withdrawal of Native Tokens May 8, 2024
@Gauthamastro Gauthamastro mentioned this issue May 13, 2024
Merged
4 tasks
@Gauthamastro Gauthamastro linked a pull request May 13, 2024 that will close this issue
Direct Deposit/Withdrawal #959
Merged
4 tasks
Gauthamastro added a commit that referenced this issue May 20, 2024
@Gauthamastro
Direct Deposit/Withdrawal (#959)
6046b3a 
## Describe your changes

1. Registers main account and deposits the assets on detection of direct
deposit XCM type
2. Modified Ob Withdrawal to use Multilocation for direct withdrawal
from OB.

## Issue ticket number and link
#956

## Checklist before requesting a review
- [ ] I have performed a self-review of my code.
- [ ] If it is a core feature, I have added thorough tests.
- [ ] I removed all Clippy and Formatting Warnings. 
- [ ] I added required Copyrights.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Gauthamastro Gauthamastro

Labels
None yet
Projects
Polkadex Backend
Status: Review
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Direct Deposit/Withdrawal Polkadex-Substrate/Polkadex
2 participants
@Gauthamastro @zktony