SV UI: Add SV frontend UI for switch-over times - #7062
Open
JYC11 wants to merge 8 commits into
Open
Conversation
pawelperek-da
self-requested a review
September 2, 2026 15:33
JYC11
force-pushed
the
jaeyoon/sv-ui-switch-over-times
branch
from
September 3, 2026 01:23
f2d79e0 to
7d4d640
Compare
| )} | ||
| </form.AppField> | ||
| ))} | ||
| {dsoConfigChanges |
Contributor
Contributor
Author
There was a problem hiding this comment.
I experienced the same bug when working on other forms for another PR but I had no idea how to replicate it
Contributor
Author
There was a problem hiding this comment.
Perhaps this could be a separate github issue?
Contributor
Author
Contributor
Author
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.
|
|
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 |
Contributor
Author
|
@pmunot-da I updated the main PR description with what you requested. Pls check |
JYC11
force-pushed
the
jaeyoon/sv-ui-switch-over-times
branch
from
September 7, 2026 02:09
b2fd92c to
6833951
Compare
JYC11
force-pushed
the
jaeyoon/sv-ui-switch-over-times
branch
from
September 8, 2026 01:57
6833951 to
3d3de7c
Compare
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>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
JYC11
force-pushed
the
jaeyoon/sv-ui-switch-over-times
branch
from
September 10, 2026 01:18
3d3de7c to
52ddf5e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.






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