-
-
Notifications
You must be signed in to change notification settings - Fork 142
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
Feat/xdc chain and xsswap #45
Feat/xdc chain and xsswap #45
Conversation
…h existing code (#3) * Drop patch packages * Update build files * Update vendor packages for building * ethers-xdc is building * It compiles * try again * Still trying * fix * clean up
…xdc_chain_and_xsswap
…way into feat/xdc_chain_and_xsswap
Hi @james-hummingbot could you please fix failing tests? Steps:
Actual: |
@nikspz Where do you see |
@james-hummingbot never mind that’s just a name of gateway folder. |
@nikspz Do you have any insight into the current error?
|
None, except it’s blocking PR gateway installation |
it's strange because that dependency already exists in the main branch https://github.com/hummingbot/gateway/blob/main/yarn.lock#L12120 |
Hi @james-hummingbot I tried to do some investigation and what I did is deleted the Tested using WSL (ubuntu20.04), will have a look with MacOS |
@rapcmia I am developing on Mac. My yarn version is |
Hi @james-hummingbot ahh yes both WSL and Mac using. version 1.22.19 as well. Here's the screenshot from Mac |
@rapcmia I guess the changes @fengtality made to workflow in the development branch fixed it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- Setup this PR and Feat/xdc chain and xdcswap hummingbot#6114
- Connect wallet both xinfin and apothem
- Tested
apothem
using WXDC and XSP - Modified tokenlist
xdc_tokens_apothem.json
- {wxdc, 0x2a5c77b016Df1b3b0AE4E79a68F8adF64Ee741ba}
- {xsp, 0x339c94081D1F7808FeEa3E6E5B1feA864c3cef43}
- Gateway connect xsswap
- Check wallet balance: ok
- Ran
gateway connector-tokens xxswap_xdc_apothem WXDC,XSWAP
ok - Ran `gateway approve-token {WXDC, XSP}
- No go, getting error
nonce already used
- As per advised, manually approved tokens via curl approve script, all ok
- No go, getting error
- Test curl
- Check port connection, ok
- Check for network status, ok
- Add xdc private key, ok
- Get all wallets, ok
- Delete xdc private key, ok
- Allowances check
- Approve {WXDC,XSP}, ok
- Test transaction polling, ok
- Get xdc swap price, ok
- Buy swap, ok
- Sell swap, ok
- Setup simple AMM-arb on client
- Setup
coingecko
as rate oracle source and addedxswap-protocol
- Observe profitability ok
- Observe order placed and filled events ok
- Check client history and CSV. Compare to apothem explorer, all ok
- Setup
Before submitting this PR, please make sure:
A description of the changes proposed in the pull request:
Adds XDC Chain and XSSwap connector into gateway.
Note: The large amount of changes is the result to having to include our own copy of
ethers-xdc
with changes made specifically for the XDC chainTests performed by the developer:
Added manual
curl
commands and included various unit tests. Tested XSSwap in hummingbot.Tips for QA testing:
Tested and approved by CoinAlpha QA team.
Ensure that XSSwap connector works as intended.
You will need to use this version of Hummingbot in PR #312 for it to work
You might need to use the
curl
commands to manual issue a token approvalNote: If testing on xinfin(mainnet) network, please verify that the WXDC address used is as follows:
xdc951857744785e80e2de051c32ee7b25f9c458c42
Tests performed by the developer:
Tips for QA testing: