Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move OPNsense services functions to REST API #206

Merged
merged 6 commits into from
Sep 23, 2024

Conversation

Snuffy2
Copy link
Collaborator

@Snuffy2 Snuffy2 commented Sep 22, 2024

  • Will create more detailed service switches than before. For example, I use udpbroadcastrelay. Previously, there would be a single switch to turn on/off the entire service. Now there will be a separate switch for each entry.
    • The Service Name is more broad and the Service ID is specific to a single entry in a service (when applicable). Both will work in the HA Action (fka. Service) Calls
    • Shows the Service Name and Service ID as attributes of the Service Switches.
  • Does not create switches for services that are unable to be stopped. These are listed as locked. ex. webgui, sysctl
  • Renames the field property to prop_name to not conflict with the property decorator
  • Fixes typo of unqiue to unique

Should be released with #204

@Snuffy2 Snuffy2 changed the title Move services to REST API Move OPNsense services functions to REST API Sep 22, 2024
@Snuffy2 Snuffy2 force-pushed the Move-services-to-REST-API branch 6 times, most recently from 92814e4 to 57e21de Compare September 22, 2024 18:10
@Snuffy2 Snuffy2 marked this pull request as ready for review September 23, 2024 01:11
@alexdelprete alexdelprete merged commit dfc9593 into travisghansen:main Sep 23, 2024
2 checks passed
@Snuffy2 Snuffy2 deleted the Move-services-to-REST-API branch September 24, 2024 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants