Skip to content

Commit

Permalink
luci-app-usteer: Fix no wireless defined setup
Browse files Browse the repository at this point in the history
Some users use the usteer app in a "non wireless" device. With the current code it crashes because there is no SSID defined. Making the ui element a DynamicList fixes the issue because it allows free text input.

Signed-off-by: Miguel Angel Mulero Martinez <[email protected]>
  • Loading branch information
McGiverGim committed Feb 13, 2024
1 parent 81f1f9d commit f5d1321
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -609,7 +609,7 @@ return view.extend({
o.optional = true;
o.datatype = 'list(string)';

o = s.taboption('settings', form.MultiValue, 'ssid_list', _('SSID list'), _('List of SSIDs to enable steering on'));
o = s.taboption('settings', form.DynamicList, 'ssid_list', _('SSID list'), _('List of SSIDs to enable steering on'));
WifiNetworks.forEach(function (wifiNetwork) {
if (wifiNetwork.getSSID()) {
o.value(wifiNetwork.getSSID())
Expand Down

0 comments on commit f5d1321

Please sign in to comment.