Bug #5249
closedGateways dashboard widget display selection option AWOL
100%
Description
The Gateways widget recently had and option added so you can choose to display [Gateway IP|Monitor IP|Both]. It was committed to master and RELENG_2_2. It is working in 2.2.5 as per the attached screen shot.
Some of the code is still in the version in master (e.g. to actually store the setting if the client-end has sent a $POST to save a new setting). But there is no tool icon and no code to make the radio buttons (or equivalent) if there was a tool icon to click.
Files
Updated by Anonymous about 9 years ago
- Status changed from New to Feedback
- Assignee set to Phillip Davis
Code has been added to the widget to allow the selection of monitor, gateway or both.
The settings are saved to the config file and are correctly observed on displaying the widget.
Would you please let me know if this is satisfactory? I am not sure how the display should change with each of the settings.
Thanks
Updated by Anonymous about 9 years ago
- % Done changed from 0 to 100
Applied in changeset pfsense:3511647f2d478508d92e216a9a104af46a9980c7.
Updated by Phillip Davis about 9 years ago
- File Gateways-widget-23-Gateway-IP.png Gateways-widget-23-Gateway-IP.png added
- File Gateways-widget-23-Monitor-IP.png Gateways-widget-23-Monitor-IP.png added
- File Gateways-widget-23-Both.png Gateways-widget-23-Both.png added
2.3-ALPHA (amd64) built on Mon Oct 05 18:44:18 CDT 2015
works, thanks.
For the record I have attached 3 screen shots of the Gateways widget in each settings state. The gateway of this test system is 10.49.208.250 and the monitor IP is 8.8.8.8
With display gateway IP selected, the widget shows just 10.49.208.250
With display monitor IP selected, the widget shows just 8.8.8.8
With display both selected, the widget shows the gateway IP and monitor IP.
I also tested the case when there is no alternative monitor IP defined for the gateway. In that case the gateway IP is the monitor IP. For every display setting the widget should show just the gateway IP (which is also the monitor IP), and it does that correctly.
All good here - thanks.
Updated by Chris Buechler about 9 years ago
- Target version set to 2.3
- Affected Version set to 2.3