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

Mint tokens rules adaptation #1568

Merged
merged 8 commits into from
Nov 15, 2022
Merged

Mint tokens rules adaptation #1568

merged 8 commits into from
Nov 15, 2022

Conversation

Mixa84
Copy link
Contributor

@Mixa84 Mixa84 commented Nov 14, 2022

What kind of PR is this?:

/kind feature

What this PR does / why we need it:

Restore old behavior for DAT tokens to be able to be minted by owner. When token is added to consortium then allow only foundation and consortium members to mint it, not the owner anymore.
Allowing on regtest everybody to mint any token, except when --regtest-minttoken-simulate-mainnet flag is supplied.

Minting rules after this PR:

 - only mintable by token owner
DAT token not in consortium:
 - token owner
 - foundation
DAT token in consortium:
 - foundation
 - consortium```

@Mixa84 Mixa84 added protocol Has network protocol changes kind/feature labels Nov 14, 2022
@Mixa84 Mixa84 requested a review from a team November 14, 2022 15:30
@Mixa84 Mixa84 self-assigned this Nov 14, 2022
@shohamc1
Copy link
Contributor

Fixes the minttoken issue on Jellyfish 👍🏻

@Bushstar Bushstar merged commit 254bd62 into master Nov 15, 2022
@Bushstar Bushstar deleted the feature/mint_tokens_change branch November 15, 2022 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/sync protocol Has network protocol changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants