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

Add AEUR token #249

Closed
wants to merge 3 commits into from
Closed

Add AEUR token #249

wants to merge 3 commits into from

Conversation

phraktle
Copy link

@phraktle phraktle commented Feb 20, 2019

Adding the Augmint Euro decentralized stablecoin.

For reference, see:

Copy link
Contributor

@estebanmino estebanmino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @phraktle, due to the community activity of this token on the network I'd recommend you to use the watchAsset feature. We are currently minimizing our usage of this token list, Please review our developer docs for our current best practices for displaying your token. You can find watchAsset feature here. Let me know if you have any doubt. Thanks.

@phraktle
Copy link
Author

phraktle commented Mar 4, 2019

Hi @estebanmino,

Thanks for the feedback. While the watchAsset feature is helpful, it's not actually a solution to the problem that prompted this PR, namely that when people with MetaMask wallets receive our tokens they do not see them listed properly.

You do have my full support on getting rid of centralized registries such as this. However, refusing a proper PR is actually putting projects at a UX disadvantage compared to others that came before (popular or not), so I would like to request that you reconsider your policies based on fairness and include new submissions that fulfill the requirements or remove all tokens listed in this registry.

@keo
Copy link

keo commented Mar 6, 2019

Please add this. Thank you.

@phraktle
Copy link
Author

@estebanmino the watchAsset API is inadequate even for the purpose it was meant to address. See ethereum/EIPs#1426 (comment)

Until Ethereum and Metamask can properly address the problem of detecting tokens in wallets, I urge Metamask maintainers to merge requests to add tokens that fulfill the listed criteria (such as AEUR), instead of further politicizing the process and making it a popularity contest.

@debreczeni
Copy link

debreczeni commented Mar 13, 2019

👍 Would love to see this! ❤️

@oyunrdn
Copy link

oyunrdn commented Mar 13, 2019

It would be great if you add Augmint Euro.

@godavid
Copy link

godavid commented Mar 13, 2019

Augmint EUR would be nice

@szerintedmi
Copy link

+1 one for adding Augmint AEUR

@hubudibu
Copy link

+1 please add, 💖 Augmint

@reyrodrigez
Copy link

+1 AugmintEUR, would be great

@estebanmino
Copy link
Contributor

Hi @phraktle, due to the community activity I'll close this PR, 73 holders is not enough to add it to MetaMask. I encourage you to give watch asset feature a try. Thanks.

@phraktle
Copy link
Author

phraktle commented Jun 19, 2019

@estebanmino how many holders are enough? did previously merged tokens all fulfill the same criteria?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants