Skip to content

Configuration setters getters - #52

Open
EddyTheCo wants to merge 5 commits into
mainfrom
configuration_setters_getters
Open

EddyTheCo wants to merge 5 commits into
mainfrom
configuration_setters_getters

Conversation

@EddyTheCo

Copy link
Copy Markdown
Contributor

Add missing setters and getters for configuration properties that can be set by the user.
Fix #50.

Expose method for getting Nameservers.Configuration property in the
service.

Implement part of #50.

Signed-off-by: Eduardo Gonzalez <eduardo.gonzalez@amarulasolutions.com>
Add the setters and getters for the Timeservers.Configuration
properties.
Implement part of #50.

Signed-off-by: Eduardo Gonzalez <eduardo.gonzalez@amarulasolutions.com>
Add the setters and getters for the Domains.Configuration properties.
Implement part of #50.

Signed-off-by: Eduardo Gonzalez <eduardo.gonzalez@amarulasolutions.com>
Add the setters and getters for the Proxy.Configuration.
Implement part of #50.

Signed-off-by: Eduardo Gonzalez <eduardo.gonzalez@amarulasolutions.com>
Add the setters and getters for the mDNS.Configuration.
Implement part of #50.

Signed-off-by: Eduardo Gonzalez <eduardo.gonzalez@amarulasolutions.com>
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

🧩 Build Artifacts

✅ The following build artifacts were produced:

@AndreaRicchi
AndreaRicchi self-requested a review November 3, 2026 15:00

@AndreaRicchi AndreaRicchi 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.

LGTM

This branch has not been deployed

No deployments
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.

Add missing setters and getters for the services configuration properties.

2 participants