Project

General

Profile

Bug #40161

Validate UPS fields in new UI

Added by Michael Reynolds over 1 year ago. Updated over 1 year ago.

Status:
Done
Priority:
No priority
Assignee:
Erin Clark
Category:
GUI (new)
Target version:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

Services -> UPS -> Configure

The Identifier field can only contain alphanumeric, period, comma, hyphen, and underscore characters.

Need a validator to refuse special characters


Related issues

Related to FreeNAS - Bug #40166: Document UPS Hostname Field in new UI version of GuideDone
Has duplicate FreeNAS - Bug #40171: Add validator for the UPS Monitor PasswordClosed

Associated revisions

Revision c90a6f3f (diff)
Added by Erin Clark over 1 year ago

Fix hiding/disabling of hostname/driver fields and add validation to UPS Ticket: #40166 #40171 #40161

History

#1 Updated by Dru Lavigne over 1 year ago

  • Target version changed from Backlog to 11.2-BETA3

#2 Updated by Erin Clark over 1 year ago

  • Related to Bug #40166: Document UPS Hostname Field in new UI version of Guide added

#3 Updated by Erin Clark over 1 year ago

  • Status changed from Unscreened to In Progress

The validator requested has been added in this PR (along with other fixes to this section)

https://github.com/freenas/webui/pull/1091

#4 Updated by Dru Lavigne over 1 year ago

  • Subject changed from UPS Identifier needs a validator to Validate UPS fields in new UI

#5 Updated by Dru Lavigne over 1 year ago

  • Has duplicate Bug #40171: Add validator for the UPS Monitor Password added

#6 Updated by Erin Clark over 1 year ago

  • Status changed from In Progress to Ready for Testing

#7 Updated by Dru Lavigne over 1 year ago

  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#8 Updated by Jeff Ervin over 1 year ago

  • Status changed from Ready for Testing to Failed Testing

Test Fails FreeNAS-11.2-MASTER-201808220903

When a non-supported character was entered, the line under the field showed red, and I was unable to save configuration, but no validation/alert showed at any time.

#9 Updated by Jeff Ervin over 1 year ago

  • Status changed from Failed Testing to Passed Testing
  • Needs QA changed from Yes to No

After consulting with Erin, this passed testing

#10 Updated by Dru Lavigne over 1 year ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF