Skip to content

SV UI: Add SV frontend UI for switch-over times - #7062

Open
JYC11 wants to merge 8 commits into
canton-network:mainfrom
obsidiansystems:jaeyoon/sv-ui-switch-over-times
Open

SV UI: Add SV frontend UI for switch-over times#7062
JYC11 wants to merge 8 commits into
canton-network:mainfrom
obsidiansystems:jaeyoon/sv-ui-switch-over-times

Conversation

@JYC11

@JYC11 JYC11 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

fixes: #4249

Switch-over times let SVs agree on a single time to change Scan-served APIs so that BFT reads stay consistent. This information is stored in key-value pairs with keys being what's changing and value for when everyone changes the API. This PR is the 2nd part of the work in issue 4249 where users are allowed to vote on when to set the switch-over time.

the UI part of the issue is implemented here
Amulet Config and DSO Rules Config changes are near identical so screenshots are only from Amulet Config proposal
Screenshot From 2026-09-02 15-32-01
Screenshot From 2026-09-02 15-32-22
Screenshot From 2026-09-02 15-32-57
Screenshot From 2026-09-02 15-33-16

@JYC11 JYC11 changed the title Add SV frontend UI for switch-over times SV UI: Add SV frontend UI for switch-over times Sep 2, 2026
@pawelperek-da
pawelperek-da self-requested a review September 2, 2026 15:33
@JYC11
JYC11 force-pushed the jaeyoon/sv-ui-switch-over-times branch from f2d79e0 to 7d4d640 Compare September 3, 2026 01:23
Comment thread apps/sv/frontend/src/components/forms/formValidators.ts
)}
</form.AppField>
))}
{dsoConfigChanges

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.

I was able to lock the proposal even though I don't have any errors, I don't know what's the exact repro but it seem easy to trigger

Image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I experienced the same bug when working on other forms for another PR but I had no idea how to replicate it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Perhaps this could be a separate github issue?

Comment thread apps/sv/frontend/src/components/forms/formValidators.ts
Comment thread apps/sv/frontend/src/__tests__/formValidators.test.ts
@JYC11

JYC11 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

updated switch over time renders

from null (no switch over times) to switch over times set
Screenshot From 2026-09-04 12-06-25

from some switch over times to another switcher over times
Screenshot From 2026-09-04 12-22-47

from some switch over times to no switch over times
Screenshot From 2026-09-04 12-39-40

@JYC11

JYC11 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

fixed the "0001-01-01" issue

Screenshot From 2026-09-04 15-57-45

@JYC11

JYC11 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

@pawelperek-da there seems to be a sentinel value/row produced by daml. I got around it by just filtering it out so that it doesn't render but is this the best approach? It doesn't show up when rendering but shows up in json and json-diff view.

Screenshot From 2026-09-04 16-10-26

@pmunot-da

Copy link
Copy Markdown

hey @JYC11 can you explain what switch-over time is and what it does so we can provide better feedback on the best design approach for this feature

@JYC11

JYC11 commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

@pmunot-da I updated the main PR description with what you requested. Pls check

@JYC11
JYC11 force-pushed the jaeyoon/sv-ui-switch-over-times branch from b2fd92c to 6833951 Compare September 7, 2026 02:09
@JYC11
JYC11 force-pushed the jaeyoon/sv-ui-switch-over-times branch from 6833951 to 3d3de7c Compare September 8, 2026 01:57
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
…ling in different timezone

Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
@JYC11
JYC11 force-pushed the jaeyoon/sv-ui-switch-over-times branch from 3d3de7c to 52ddf5e Compare September 10, 2026 01:18
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.

On-ledger synchronization for SVs on switch-over record times

8 participants