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

comment issue #24

Open
ggs134 opened this issue May 16, 2018 · 5 comments
Open

comment issue #24

ggs134 opened this issue May 16, 2018 · 5 comments

Comments

@ggs134
Copy link
Member

ggs134 commented May 16, 2018

If comment is nedded to tokyo-schema, what am i supposed to do?

I reccommand to use "_comment" key, value pair.

For example,

token : {
    token_type : {
      _comment : ["Client does not want minime token"],
      is_minime : Boolean,
      token_option : {
        burnable : Boolean,          // self-burnable
        pausable : Boolean,          // pause token transfer. MiniMe token is pausable as default.
        no_mint_after_sale: Boolean, // no more token generation after sale
      }
    },
    token_name : String,
    token_symbol : String,
    decimals : Number,
  }
@4000D
Copy link
Contributor

4000D commented May 16, 2018

Where and why should "_comment" be used?

@ggs134
Copy link
Member Author

ggs134 commented May 16, 2018

Every JSON object if we want to.
Is this policy affects tokyo-generator?

Let's assume situation, what if we want to distinguish between dev team's account and onther's when it comes to ether distribution?

@4000D
Copy link
Contributor

4000D commented May 16, 2018

It is not allowed. tokyo-schema uses Joi, and it rejects unexpected { key, value } pair.
We can't add "_comment" fields to all possible spots.

@4000D
Copy link
Contributor

4000D commented May 16, 2018

"_comment" is used in sale.distributioin.token, sale.distributioin.ether, sale.stages.
Does other fields need to be "_comment"ed?

@ggs134
Copy link
Member Author

ggs134 commented May 16, 2018

If that job is too huge, don't take this proposal seriously.
Consider Cost(time)/Effect in terms of utility of _comment.

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

2 participants