Skip to content

#41312 first config property (sync enabled/disabled) can be manipulat…#52

Merged
rsteppac merged 7 commits intodevelopfrom
feature/41312-add-enable-disable-shortcut-function-to-update-client-configuration
Jun 12, 2025
Merged

#41312 first config property (sync enabled/disabled) can be manipulat…#52
rsteppac merged 7 commits intodevelopfrom
feature/41312-add-enable-disable-shortcut-function-to-update-client-configuration

Conversation

@rsteppac
Copy link
Copy Markdown
Collaborator

…ed with the UI and persisted through the cdr service.

To update the single property I implemented a generic way to update any property that is flagged as updatable (by a marker interface) and applied the new mechanism to the existing client credential auto-renewal option. The test coverage has suffered. Will need to be brought up again...

…ed with the UI and persisted through the cdr service.

To update the single property I implemented a generic way to update any property that is flagged as updatable (by a marker interface) and applied the new mechanism to the existing client credential auto-renewal option.
The test coverage has suffered. Will need to be brought up again...
@rsteppac rsteppac requested a review from Lesrac June 10, 2025 19:13
Comment thread build.gradle.kts Outdated
Comment thread build.gradle.kts Outdated
rsteppac and others added 5 commits June 12, 2025 11:17
Co-authored-by: Lesrac <Lesrac@users.noreply.github.com>
…/client/handler/ConfigurationWriter.kt

Co-authored-by: Lesrac <Lesrac@users.noreply.github.com>
…r/client/ui/ComposeUtils.kt

Co-authored-by: Lesrac <Lesrac@users.noreply.github.com>
…r/client/ui/data/CdrClientApiClient.kt

Co-authored-by: Lesrac <Lesrac@users.noreply.github.com>
@rsteppac rsteppac requested a review from Lesrac June 12, 2025 10:22
@rsteppac rsteppac merged commit 13e5014 into develop Jun 12, 2025
4 checks passed
@rsteppac rsteppac deleted the feature/41312-add-enable-disable-shortcut-function-to-update-client-configuration branch June 12, 2025 11:54
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.

2 participants