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

NEP: Security Token Standard #98

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

joeqian10
Copy link

Security Token Standard draft created by the collaboration of Liquefy and NGD.

joeqian10 and others added 2 commits July 23, 2019 12:35
@WyattMufson
Copy link

@joeqian10 I opened a pull request to clean up some of the grammar/spelling: joeqian10#1

Cleaned up English and formatting
@joeqian10
Copy link
Author

@WyattMufson PR merged and thanks for reviewing it.

Copy link
Member

@vncoelho vncoelho left a comment

Choose a reason for hiding this comment

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

This is an interesting proposal and standard!

At the beginning, I had the feeling that NEP could handle Security Token standard.
However, this proposal has some peculiarities and applied functions. Sounds very good.

@joeqian10
Copy link
Author

@vncoelho I had the same feeling with you at first since we don't have a standard for partially fungible token (PFT). Security tokens (ST) are basically partially fungible tokens. When we start to create this ST standard, we have two ways: the first is to create a PFT standard and a ST standard based on it; the second is to use existing standards. Finally we chose the second one and came up this Main-Sub Standard idea.

@corollari
Copy link
Contributor

Submitted a PR fixing some typos.

@joeqian10
Copy link
Author

@corollari 一丁, thank you, and your PR is merged.

arhaha and others added 2 commits August 27, 2019 15:37
APIs:
addInvestorToWhitelist -> addInvestorsToWhitelist
removeInvestorFromWhitelist -> removeInvestorsFromWhitelist
addInvestorToBlacklist -> addInvestorsToBlacklist
removeInvestorFromBlacklist -> removeInvestorsFromBlacklist
mintToken  -> mint
burnToken  -> burn
events:
TokenMinted -> Minted
TokenBurned -> Burned
@erikzhang
Copy link
Member

All management methods should be removed from the standard.

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.

6 participants