-
Notifications
You must be signed in to change notification settings - Fork 757
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
Add DHCPv4 Leases widget #7865
base: master
Are you sure you want to change the base?
Add DHCPv4 Leases widget #7865
Conversation
Adds a DHCPv4 Leases widget
A lot of networks have to deal with a lot of leases, which can grow the widget extremely fast without any mechanism to organize/group/search. The Services -> ISC DHCPv4 -> Leases page already offers all of this functionality so it doesn't make much sense to duplicate this to a widget. In my humble opinion it would make more sense to display information which immediately tells you something about the state of the system; e.g. the number of online/offline clients (inactive included/excluded) etc. If there are other things we would like to know that aren't included by default in the controller, it may be an idea to add a separate endpoint for this. FWIW, 9f56534 also introduced breakout link buttons which positions all of the aforementioned functionality a single click away. |
The beauty of the widget system is that it is optional and configurable - in this case this widget can be any size you want, and the data is scrollable inside it. Having the widget included allows people who want it to use it - you are not forced to use it. There's very little downside to including something like this. For this reason I think it makes sense to have as many widgets as possible available to add, so that the end user is not faced with the frustrating situation of wanting something that is completely unavailable or difficult to access or requires manually adding files. The widget only shows non-expired leases. It shows if they are online or offline. It is valuable information at-a-glance. By comparison, the full page is much more to sort through. The information is very helpful if you are adding devices to the network which you then need to know the IP/mac of but the devices themselves do not report this information. To be blunt, the lack of this dashboard functionality nearly caused me to abandon opnsense after initial configuration. |
Just by your argument having it searchable via text input would make more sense than showing a blob of info verbatim. |
I will see if I can add that functionality, although that would make it awfully similar to the full page |
Which is the original point. 😉 I’m all for these types of additions but the scope needs to be clear and complete, because otherwise people just ask for more and when you’re not around someone else will have to deal with it because requests will keep coming and eventually it needs maintenance. |
That does not mean it is free in terms of processing or rendering, nor does it help you find anything specific any faster than referring to the original page will (which, again, is a single click away). Instead, showing the newest N leases makes much more sense in that case.
This can and will be in the order of thousands for any decent sized network and can easily be conveyed in a limited set instead.
But it is sortable, searchable and groupable by interface. I also am certainly not against this addition, but I'm against rendering dense amounts of information which is easily managed in the system elsewhere. |
6586a65
to
607e32a
Compare
This adds a DHCPv4 Leases widget.
The widget has 4 items per row: an icon to indicate online/offline, an IP address, a hostname, and MAC. It is resizable and scrollable.
It is based on the code from the dyndns widget, which I have modified significantly. It works great for me, but please review since I haven't done javascript in 10 years.