fix(#patch); bancor-v3; add other possible stable target tokens for tradeOutputBySourceAmount #2371
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue:
targetToken
asDAI
address[getTokenPriceUSD] try_tradeOutputBySourceAmount(0xeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee, 0x6b175474e89094c44da98b954eedeac495271d0f, 1000000000000000000) reverted
Fix: Added
USDC
andUSDT
to be also considered astargetToken
s fortradeOutputBySourceAmount
callDeployment: https://okgraph.xyz/?q=dhruv-chauhan%2Fbancor-v3-ethereum
Query: https://api.thegraph.com/subgraphs/id/QmYLawGQf6rdA7o9BfaT4zfutFN8xPxNGGiFZHmxwamiSu/graphql?query=%7B%0A++swaps+%28orderBy%3A+timestamp%2C+orderDirection%3Adesc%29+%7B%0A++++hash%0A++++tokenIn+%7B%0A++++++name%0A++++++id%0A++++%7D%0A++++amountIn%0A++++amountInUSD%0A++++tokenOut+%7B%0A++++++name%0A++++++id%0A++++%7D%0A++++amountOut%0A++++amountOutUSD%0A++%7D%0A%7D