Skip to content

SV UI: More descriptive names for Reward Config - #6578

Merged
dfordivam merged 10 commits into
canton-network:mainfrom
obsidiansystems:jaeyoon/adetokunbo/cip-104-use-descriptive-names-for-reward-config
Aug 12, 2026
Merged

SV UI: More descriptive names for Reward Config#6578
dfordivam merged 10 commits into
canton-network:mainfrom
obsidiansystems:jaeyoon/adetokunbo/cip-104-use-descriptive-names-for-reward-config

Conversation

@JYC11

@JYC11 JYC11 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Fixes #5986

took over from #6180

@JYC11 JYC11 changed the title SV UI: more descriptive names for amulet config SV UI: More descriptive names for Reward Config Jul 28, 2026
@JYC11

JYC11 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author
Screenshot From 2026-07-28 10-54-38 Screenshot From 2026-07-28 10-54-46

@JYC11
JYC11 force-pushed the jaeyoon/adetokunbo/cip-104-use-descriptive-names-for-reward-config branch from b521c51 to 9a0635b Compare July 28, 2026 01:58
Comment thread apps/sv/frontend/src/utils/buildAmuletConfigChanges.ts Outdated
Comment thread docs/src/release_notes_upcoming.rst Outdated

@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 @JYC11 . This is a nice improvement. I've added suggestions for some improved wording.

I leave the final approval to @pawelperek-da , as I'd like his OK wrt the ConfigField.tsx changes.

Comment thread apps/sv/frontend/src/utils/buildAmuletConfigChanges.ts Outdated
Comment thread apps/sv/frontend/src/utils/buildAmuletConfigChanges.ts Outdated
Comment thread apps/sv/frontend/src/utils/buildAmuletConfigChanges.ts Outdated
Comment thread apps/sv/frontend/src/utils/buildAmuletConfigChanges.ts Outdated
Comment thread apps/sv/frontend/src/utils/buildAmuletConfigChanges.ts Outdated
Comment thread apps/sv/frontend/src/components/form-components/ConfigField.tsx

@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 for the quick fixes to the descriptions @JYC11 . The only thing missing then is @pawelperek-da's review.

@JYC11
JYC11 force-pushed the jaeyoon/adetokunbo/cip-104-use-descriptive-names-for-reward-config branch from e06895b to f1be273 Compare August 3, 2026 02:35
Comment thread apps/sv/frontend/src/utils/buildAmuletConfigChanges.ts Outdated
Comment thread apps/sv/frontend/src/utils/types.ts
Comment thread apps/sv/frontend/src/utils/buildAmuletConfigChanges.ts Outdated
@JYC11
JYC11 force-pushed the jaeyoon/adetokunbo/cip-104-use-descriptive-names-for-reward-config branch from 218150f to 0211945 Compare August 10, 2026 03:59

@pawelperek-da pawelperek-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!

@JYC11
JYC11 force-pushed the jaeyoon/adetokunbo/cip-104-use-descriptive-names-for-reward-config branch from 0211945 to fc5041f Compare August 11, 2026 00:32
@JYC11
JYC11 deployed to ci-forks August 11, 2026 00:32 — with GitHub Actions Active
adetokunbo and others added 10 commits August 12, 2026 00:49
Signed-off-by: Tim Emiola <adetokunbo@emio.la>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
Co-authored-by: Divam <681060+dfordivam@users.noreply.github.com>
Signed-off-by: Jaeyoon Cho <jaeyoon.c@gmail.com>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
Co-authored-by: Simon Meier <simon@digitalasset.com>
Signed-off-by: Jaeyoon Cho <jaeyoon.c@gmail.com>
Co-authored-by: Simon Meier <simon@digitalasset.com>
Signed-off-by: Jaeyoon Cho <jaeyoon.c@gmail.com>
Co-authored-by: Simon Meier <simon@digitalasset.com>
Signed-off-by: Jaeyoon Cho <jaeyoon.c@gmail.com>
Co-authored-by: Simon Meier <simon@digitalasset.com>
Signed-off-by: Jaeyoon Cho <jaeyoon.c@gmail.com>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
Signed-off-by: JYC11 <jaeyoon.c@gmail.com>
@JYC11
JYC11 force-pushed the jaeyoon/adetokunbo/cip-104-use-descriptive-names-for-reward-config branch from fc5041f to 6031286 Compare August 12, 2026 00:50
@JYC11
JYC11 deployed to ci-forks August 12, 2026 00:50 — with GitHub Actions Active
@dfordivam
dfordivam merged commit 2984f04 into canton-network:main Aug 12, 2026
116 of 118 checks passed
@dfordivam
dfordivam deleted the jaeyoon/adetokunbo/cip-104-use-descriptive-names-for-reward-config branch August 12, 2026 03:08
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.

Use more descriptive names for reward config

9 participants