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.

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

added UNN token #388

Closed
wants to merge 1 commit into from
Closed

added UNN token #388

wants to merge 1 commit into from

Conversation

gtaschuk
Copy link
Contributor

@gtaschuk gtaschuk commented Dec 3, 2021

@rabmarut
Copy link
Contributor

rabmarut commented Dec 3, 2021

As a rule, I don't think we should add unverified tokens to our lists. Without verification, we can't vet the contract for safety. Do you have a contact with their team who might be able to get the token verified?

https://etherscan.io/address/0x226f7b842E0F0120b7E194D05432b3fd14773a9D#code

@gtaschuk
Copy link
Contributor Author

gtaschuk commented Dec 4, 2021

UNN team says "We have a non-deterministic compile issue associated with our implementation of eip712 and use of an experimental abiencoder. Its a documented issue we cannot resolve" and gave the example of ethereum/solidity#9573

@gtaschuk
Copy link
Contributor Author

gtaschuk commented Dec 4, 2021

Per our listing guidelines, we have determined that we cannot add UNN because it's code is unverified https://balancer.gitbook.io/balancer/core-concepts/bal-liquidity-mining/exchange-and-reward-listing

@gtaschuk gtaschuk closed this Dec 4, 2021
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