Skip to content

Conversation

@moodysalem
Copy link
Contributor

@moodysalem moodysalem commented Jun 3, 2020

i think bytes32 returndata if decoded as a string will probably work fine

@vercel
Copy link

vercel bot commented Jun 3, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/uniswap/uniswap-frontend/f3cmzlmbi
✅ Preview: https://uniswap-frontend-git-fork-moodysalem-use-token-cleanup.uniswap.now.sh

@netlify
Copy link

netlify bot commented Jun 3, 2020

Deploy preview for uniswap ready!

Built with commit 52bbabd

https://deploy-preview-855--uniswap.netlify.app

@moodysalem
Copy link
Contributor Author

moodysalem commented Jun 4, 2020

imo we should only include name and symbol

done

@moodysalem moodysalem merged commit 104be83 into Uniswap:v2 Jun 4, 2020
@moodysalem moodysalem deleted the use-token-cleanup branch June 4, 2020 02:15
crisog pushed a commit to Levinswap/levinswap-interface that referenced this pull request Feb 5, 2021
* use single call to get token information

* delete the bytes32 overload

* console log statement

* add a bunch of tests to actions.ts for multicall

* fix to work with bytes32 symbols/names

* only include name/symbol

* enforce lowercase calldata
daniel-dev1990 pushed a commit to daniel-dev1990/uniswap-interface that referenced this pull request Jul 8, 2021
* use single call to get token information

* delete the bytes32 overload

* console log statement

* add a bunch of tests to actions.ts for multicall

* fix to work with bytes32 symbols/names

* only include name/symbol

* enforce lowercase calldata
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants