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

Onboarding: token settings #17

Closed
0xGabi opened this issue Mar 9, 2021 · 6 comments
Closed

Onboarding: token settings #17

0xGabi opened this issue Mar 9, 2021 · 6 comments
Assignees

Comments

@0xGabi
Copy link
Member

0xGabi commented Mar 9, 2021

BYOT

1. Address of the existing token (e.g. `TKN`,`Token Name`)
2. New Garden token. Add a help bubble with the following text:  `We recommend sticking with the default syntax here unless you have a good reason not to` 
    1. Symbol (the existing token symbol appears by default with a lowercase g in front: `gTKN`)
    2. Name (the name of the existing token appears by default with the word Garden in front: `Garden Token Name`)

Native

  1. Symbol
  2. Name
  3. List of initial holders (seeds) specifying address and quantity for each one.
  4. Initial amount of tokens in the common pool
    1. Add a help bubble with the following text: This is the shared resource of the community which will be managed by the token holders using Conviction Voting
@0xGabi 0xGabi transferred this issue from 1Hive/gardens-ui Apr 6, 2021
@0xGabi 0xGabi changed the title Onboarding: create new token view Onboarding: token settings Jun 11, 2021
@0xGabi 0xGabi assigned pavloMandryk and unassigned PJColombo Jun 16, 2021
@sembrestels
Copy link
Member

I would remove "Initial amount of tokens in the common pool" since it should match issuance.maxRatio * token.totalSupply since the beginning.

@0xGabi
Copy link
Member Author

0xGabi commented Jun 23, 2021

I am not sure if I fully understood. Does it means that given a list of seed holder we can derive the Common Pool initial balance with: issuance.maxRatio * token.totalSupply where totalSupply is the sum of the balances of the list of seed holders?

@sembrestels
Copy link
Member

The complete formula would be:

targetRatio = commonPoolAmount / initialTotalSupply = commonPoolAmout / (seedsAmount + commonPoolAmount)

If we solve for commonPool we obtain the following formula:

commonPoolAmount = seedsAmount * targetRatio / (1 - targetRatio)

We can sum up all initial tokenholders to obtain seedsAmount and obtain targetRatio from the issuance configuration. targetRatio can not be set to one and we should disable the next button in that case, but that's part of the issuance policy screen.

@0xGabi
Copy link
Member Author

0xGabi commented Jun 24, 2021

Make sense. I probably not fully understand how issuance works. But I wonder if there are use cases where a community would like to set up a custom balance for the Common Pool?

@sembrestels
Copy link
Member

Yes, actually that's the case of 1hive. We started being few seeds, and wanted to grow quick at the beginning, so we set up a high inflation to delude initial token holdings.

So maybe a community wants to start with a lot of GDN in their common pool to distribute fast using conviction voting, but they want that over time, the relation commonPool/totalSupply to reduce.

For this case, I would set up an new slider in the issuance policy screen (maybe in advanced?) called initialRatio, so people can decide how to do the initial distribution of the token.

@0xGabi
Copy link
Member Author

0xGabi commented Jul 5, 2021

For this case, I would set up an new slider in the issuance policy screen (maybe in advanced?) called initialRatio, so people can decide how to do the initial distribution of the token.

I agree. Most likely we want to support this on the issuance app.

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

4 participants