Skip to content

add switch over time field to configs - #6968

Open
JYC11 wants to merge 4 commits into
canton-network:staging-0.8.0from
obsidiansystems:jaeyoon/switch-over-times-config
Open

add switch over time field to configs#6968
JYC11 wants to merge 4 commits into
canton-network:staging-0.8.0from
obsidiansystems:jaeyoon/switch-over-times-config

Conversation

@JYC11

@JYC11 JYC11 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

fixes: #4249

Signed-off-by: JYC11 <jaeyoon.c@gmail.com>

@meiersi-da meiersi-da left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks a lot. This goes in the right direction.

Apart from my comments below I'd like to see a small extension to the test cases for DsoConfig, AmuletConfig, and ExternalPartyConfig which shows that a new switch-over time can be set. Just to be double-sure that the propagation works correctly.

Note also that you'll need to adjust the defaults for the config UI implementation and tests. CI will tell you where.

Comment thread daml/splice-amulet/daml/Splice/ExternalPartyConfigState.daml Outdated
Comment thread daml/dars.lock
Comment thread daml/splice-dso-governance/daml/Splice/DsoRules.daml Outdated
@JYC11
JYC11 force-pushed the jaeyoon/switch-over-times-config branch from c705202 to 78a424f Compare August 27, 2026 06:51
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
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

Successfully merging this pull request may close these issues.

7 participants