-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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/approve tokens changes #6005
Conversation
sync/ stg -> master
sync / development -> staging
Development
sync/ development -> staging
fix / gateway source
sync / staging -> master
This reverts commit 94291f1.
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.
check those observations!
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
- Installation source/docker, ok
- Setup simple pancakeswap-bsc on source
- Test with tokens that needs to be approved
- Confirmed tokens needs to be enabled on connector
- Run
gateway approve-token [connector] [token]
- Displayed
Token XXXX approval tx is pending
- Displayed
Token XXXX is approved for spending for xxxx for wallet xxxx
on both log and output panel
- Displayed
- Successfully approved and amm-arb worked ok
- Test docker with pangolin-avalanche
- same test done with source, all ok
- Displayed help on
gateway approve-token -h
- Test invalid tokens, getting expected error for unsupported tokens
- Execute
gateway approve-token
while AMM-arb is running, successfully reproduceTokens [token] is not approved for spending. Please use manual approval.
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!
Before submitting this PR, please make sure:
A description of the changes proposed in the pull request:
Tests performed by the developer:
Tips for QA testing: