Skip to content

Feature: DHCP Config Overview - #2079

Merged
Jixabon merged 12 commits into
masterfrom
feat/dhcp-config-overview
Apr 24, 2026
Merged

Feature: DHCP Config Overview#2079
Jixabon merged 12 commits into
masterfrom
feat/dhcp-config-overview

Conversation

@Jixabon

@Jixabon Jixabon commented Mar 22, 2026

Copy link
Copy Markdown
Contributor

This PR creates a new tab on the DHCP Server page with a table overview of configuration settings for each interface. Currently shows all interfaces on the system.

Also contains a rework of the javascript that handles the DHCP settings page fields.

@billz

billz commented Apr 5, 2026

Copy link
Copy Markdown
Member

Possibly an issue with the tab id's?

image

@Jixabon
Jixabon requested a review from billz April 20, 2026 22:55
@billz

billz commented Apr 21, 2026

Copy link
Copy Markdown
Member

Interface attributes aren't loading. Cleared the cache twice. Console output:

RaspAP app.js initialized  app.js?v=1776734148:35
Current path: /dhcpd_conf  app.js?v=1776734148:39 
RaspAP DHCP module initialized   dhcp.js:54 
RaspAP DHCP ajax module initialized   dhcp.js:4 
Uncaught ReferenceError: jsonData is not defined  dhcp.js:25 
    at Object.success (dhcp.js:25:22)
    at c (jquery.min.js?v=1768789362:2:27742)
    at Object.fireWith [as resolveWith] (jquery.min.js?v=1768789362:2:28487)
    at l (jquery.min.js?v=1768789362:2:78789)
    at XMLHttpRequest.<anonymous> (jquery.min.js?v=1768789362:2:81117)

@billz

billz commented Apr 21, 2026

Copy link
Copy Markdown
Member

I wonder if this tab should be called 'Summary' to align with the Networking tab 🤔

@Jixabon
Jixabon merged commit 496fa6a into master Apr 24, 2026
3 checks passed
@Jixabon
Jixabon deleted the feat/dhcp-config-overview branch April 24, 2026 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@billz billz Awaiting requested review from billz

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Jixabon @billz