-
Notifications
You must be signed in to change notification settings - Fork 59
CIP-82 (DAML/Docs): Default optDevelopmentFundPercentage to 0.05 + releases notes #3576
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
CIP-82 (DAML/Docs): Default optDevelopmentFundPercentage to 0.05 + releases notes #3576
Conversation
21f3483 to
cfce301
Compare
…iningRound_StartIssuing in case is None (this avoid a one week rolling out) Signed-off-by: Jose Velasco <jose.velasco@intellecteu.com>
d1caeb7 to
f3c7424
Compare
IssuanceConfig.optDevelopmentFundPercentage is set during SV1Initializer bootstrapping Set it to None if not supported; otherwise keep the existing value in sv1Config.developmentFundPercentage. Signed-off-by: Jose Velasco <jose.velasco@intellecteu.com>
Signed-off-by: Jose Velasco <jose.velasco@intellecteu.com>
meiersi-da
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much @jose-velasco-ieu ! Very nice work.
@pawelperek-da works on the new SV UI which requires adapting to also support the two new config values. @pawelperek-da can I ask you to help @jose-velasco-ieu with high priority to fix his PR to pass the “NEW UI: Set Amulet Rules Configuration” assertion that fails in his his CI run on this PR?
Co-authored-by: Simon Meier <simon@digitalasset.com> Signed-off-by: Jose Velasco - IEU <jose.velasco@intellecteu.com>
Co-authored-by: Simon Meier <simon@digitalasset.com> Signed-off-by: Jose Velasco - IEU <jose.velasco@intellecteu.com>
|
@jose-velasco-ieu I've prepared a PR against the |
That's really good! Thank you @pawelperek-da |
Signed-off-by: Paweł Perek <pawel.perek@digitalasset.com>
21c947e
into
josevelasco/feature-cip-82
…leases notes (#3576) - Default optDevelopmentFundPercentage to 0.05 in AmuletRules_MiningRound_StartIssuing in case when None. - Account for development fund support when optDevelopmentFundPercentage is set during SV1Initializer bootstrapping. - Add release note/ PR reviewed in #3576 CI run #3576 Signed-off-by: Jose Velasco <jose.velasco@intellecteu.com>
…edDevelopmentFundCoupons (#3594) * Daml: Add support for DevelopmentFundCoupon PR reviewed in #3426 CI run #3451 Signed-off-by: Jose Velasco <jose.velasco@intellecteu.com> * CIP-82 (Scala): Add MergeUnclaimedDevelopmentFundCouponsTrigger (#3522) PR reviewed in #3522 CI run #3522 Signed-off-by: Jose Velasco <jose.velasco@intellecteu.com> * CIP-82 (DAML/Docs): Default optDevelopmentFundPercentage to 0.05 + releases notes (#3576) - Default optDevelopmentFundPercentage to 0.05 in AmuletRules_MiningRound_StartIssuing in case when None. - Account for development fund support when optDevelopmentFundPercentage is set during SV1Initializer bootstrapping. - Add release note/ PR reviewed in #3576 CI run #3576 Signed-off-by: Jose Velasco <jose.velasco@intellecteu.com> * [ci] Signed-off-by: Jose Velasco <jose.velasco@intellecteu.com> --------- Signed-off-by: Jose Velasco <jose.velasco@intellecteu.com>
Summary
optDevelopmentFundPercentageto 0.05 inAmuletRules_MiningRound_StartIssuingin case when None.optDevelopmentFundPercentageis set duringSV1Initializerbootstrapping. It is set it to None if not supported; otherwise keep the existing value in
sv1Config.developmentFundPercentage.Pull Request Checklist
Cluster Testing
/cluster_teston this PR to request it, and ping someone with access to the DA-internal system to approve it./hdm_teston this PR to request it, and ping someone with access to the DA-internal system to approve it.PR Guidelines
Fixes #n, and mention issues worked on using#nMerge Guidelines