Skip to content

Move Service Controls - #2066

Merged
billz merged 16 commits into
masterfrom
feat/move-service-controls
Mar 9, 2026
Merged

billz merged 16 commits into
masterfrom
feat/move-service-controls

Conversation

@Jixabon

@Jixabon Jixabon commented Feb 22, 2026 •

Copy link
Copy Markdown
Contributor

The purpose of this PR is to move service control buttons up near the service status indicator.

Why?

In a few instances there were issues with the service control buttons not being able to function due to form validation that the button was a part of. Moving the service control buttons outside of the general page forms solves this problem entirely. In addition, consolidating this part of the UI make it more clear as to what can be done with the page form (eg save setting) and where to go to manage the service.

Screenshot 2026-03-06 at 9 35 08 AM

Controls moved

  • DHCP
  • AdBlock
  • HostAPD
  • OpenVPN
  • WireGuard
  • RestAPI

ToDos

  • Verify new OpenVPN controls
  • Verify new WireGuard controls
  • Check for plugins that need similar changes
    • Tailscale
    • Wireshark
    • CaptivePortal

@Jixabon

Jixabon commented Feb 22, 2026

Copy link
Copy Markdown
Contributor Author

@billz do you have a setup handy that could test the openvpn and wireguard new controls?

@billz

billz commented Feb 22, 2026

Copy link
Copy Markdown
Member

@Jixabon sure, I can figure out a way to test these services

@Jixabon Jixabon changed the title Feat/move service controls Move Service Controls Feb 22, 2026
@billz
billz marked this pull request as ready for review March 1, 2026 23:08
dependabot Bot and others added 5 commits March 5, 2026 15:19
@billz
billz merged commit 8132833 into master Mar 9, 2026
3 checks passed
@billz
billz deleted the feat/move-service-controls branch March 9, 2026 01:43
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