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

DexNS support at TokenMint. #98

Open
Dexaran opened this issue Dec 25, 2017 · 5 comments
Open

DexNS support at TokenMint. #98

Dexaran opened this issue Dec 25, 2017 · 5 comments

Comments

@Dexaran
Copy link
Collaborator

Dexaran commented Dec 25, 2017

DexNS is a naming service that allows to register token abbreviations (symbols).

All the tokens that are registered at DexNS are automatically listed at Classic Ether Wallet.

DexNS allows to register name instantly in a context of a single transaction execution. This allows to integrate it with TokenMint. If a user wants to register his token at DexNS then he can enable the registerAtDexNS flag and pay a registration fee (0,1 ETC) but get his token automatically listed at CEW. If a user don't want to register his token at DexNS then he can disable the registerAtDexNS flag.

All the necessary smart-contract changes are applied at: https://github.com/EthereumCommonwealth/TokenMint/tree/master/contracts

@elaineo
Copy link
Contributor

elaineo commented Dec 26, 2017

@Dexaran I think that is a great idea. I see DexNS_interface.sol but not the function definitions. I think we would have the registry contract call registerName, is that correct?

@Dexaran
Copy link
Collaborator Author

Dexaran commented Dec 26, 2017

I would say that you have to call registerAndUpdateName function: https://github.com/EthereumCommonwealth/TokenMint/blob/master/contracts/DexNS_Interface.sol#L13

Otherwise, the TokenMint contract will become the owner of a Name, not a user.

@Dexaran
Copy link
Collaborator Author

Dexaran commented Dec 26, 2017

I have submitted a pull request for contracts.

@realcodywburns
Copy link
Owner

Merged

@elaineo
Copy link
Contributor

elaineo commented Mar 21, 2018

let's leave this open til the issues in the PR are fixed #99

@elaineo elaineo reopened this Mar 21, 2018
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

No branches or pull requests

3 participants