Project

General

Profile

Bug #6872

Captive Portal per user bandwidth field no longer accepts 0.

Added by Steve Wheeler over 2 years ago. Updated over 2 years ago.

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

100%

Estimated time:
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 (diff)
Added by Phillip Davis over 2 years 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 (diff)
Added by Phillip Davis over 2 years 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 (diff)
Added by Phillip Davis over 2 years 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 over 2 years 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 over 2 years ago

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

#3 Updated by Renato Botelho over 2 years ago

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

PR has been merged

#4 Updated by Jim Pingle over 2 years ago

  • Status changed from Feedback to Resolved

Text is correct now.

#5 Updated by Jim Pingle over 2 years ago

  • Target version changed from 2.4.0 to 2.3.3

Also available in: Atom PDF