Project

General

Profile

Bug #6872

Captive Portal per user bandwidth field no longer accepts 0.

Added by Steve Wheeler 7 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
Normal
Category:
Web Interface
Target version:
Start date:
10/23/2016
Due date:
% Done:

100%

Affected version:
2.3.2
Affected Architecture:
All

Description

The text says "Leave empty or set to 0 for no limit." However input error checking in the browser now no-longer allows 0. The error shown is unhelpful in determining that. It prevents any changes being saved on that zone including disabling the captive portal.

Associated revisions

Revision b7f2ebb5
Added by Phillip Davis 7 months ago

Fix #6872 CP bandwidth 0 is no valid

The front-end validation prevents zero from being entered. "Leave empty" is the way to specify no limit.

Revision 4f131b02
Added by Phillip Davis 7 months ago

Fix #6872 CP bandwidth 0 is no valid

The front-end validation prevents zero from being entered. "Leave empty" is the way to specify no limit.
(cherry picked from commit b7f2ebb5448f7992cceec899504bdd0a29058fb9)

Revision 0b1715e9
Added by Phillip Davis 7 months ago

Fix #6872 CP bandwidth 0 is no valid

The front-end validation prevents zero from being entered. "Leave empty" is the way to specify no limit.
(cherry picked from commit b7f2ebb5448f7992cceec899504bdd0a29058fb9)

History

#1 Updated by Phillip Davis 7 months ago

The front-end validation is stopping a zero from being entered, so "Leave empty" is the (only) way to specify "no limit".
This fixes up the text:
https://github.com/pfsense/pfsense/pull/3205

#2 Updated by Renato Botelho 7 months ago

  • Assignee set to Renato Botelho
  • Target version set to 2.4.0

#3 Updated by Renato Botelho 7 months ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

PR has been merged

#4 Updated by Jim Pingle 6 months ago

  • Status changed from Feedback to Resolved

Text is correct now.

#5 Updated by Jim Pingle 3 months ago

  • Target version changed from 2.4.0 to 2.3.3

Also available in: Atom PDF