Project

General

Profile

Actions

Bug #21330

closed

http proxy test connection should clear toasts when clicking on it multiple times

Added by Ohad Levy over 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Fixed in Releases:
Found in Releases:

Description

once a user an invalid url or a proxy that is actually is not a proxy, the toast notifications stack up - instead, they should be cleared (as its really not useful to see a "history" of failures in the form).

I would also question the usage of toast notifications to show form errors, I would expect it to show the field error within the form, this will avoid the problem of seeing multiple errors on the same field and not "abuse" the toast notifications.


Related issues 1 (0 open1 closed)

Related to Foreman - Feature #12216: allow compute resource connections to use a proxyClosedJustin Sherrill10/20/2015Actions
Actions #1

Updated by Ohad Levy over 6 years ago

  • Related to Feature #12216: allow compute resource connections to use a proxy added
Actions #2

Updated by Ohad Levy over 6 years ago

also note that multiline errors are not displayed correct, for example:
server response:

"status":"failure","message":"Url can't be blank\nUrl is invalid"}

yet the actual message is on the screen is:

Url can't be blank Url is invalid

Actions #3

Updated by The Foreman Bot over 6 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Timo Goebel
  • Pull request https://github.com/theforeman/foreman/pull/4921 added
Actions #4

Updated by Timo Goebel over 6 years ago

  • Status changed from Ready For Testing to New
  • Assignee deleted (Timo Goebel)
  • Pull request deleted (https://github.com/theforeman/foreman/pull/4921)
Actions #5

Updated by The Foreman Bot over 6 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Timo Goebel
  • Pull request https://github.com/theforeman/foreman/pull/4921 added
Actions #6

Updated by The Foreman Bot over 6 years ago

  • Pull request https://github.com/theforeman/foreman/pull/4922 added
Actions #7

Updated by Ohad Levy over 6 years ago

  • translation missing: en.field_release set to 296
  • Pull request deleted (https://github.com/theforeman/foreman/pull/4922)
Actions #8

Updated by The Foreman Bot over 6 years ago

  • Pull request https://github.com/theforeman/foreman/pull/4922 added
Actions #9

Updated by Timo Goebel over 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF