Project

General

Profile

Bug #21330

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

Added by Ohad Levy about 3 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Difficulty:
Triaged:
Bugzilla link:

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

Related to Foreman - Feature #12216: allow compute resource connections to use a proxyClosed2015-10-20

Associated revisions

Revision 60cf8cf2 (diff)
Added by Timo Goebel about 3 years ago

fixes #21330 - http proxies: test connection clears toasts

History

#1 Updated by Ohad Levy about 3 years ago

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

#2 Updated by Ohad Levy about 3 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

#3 Updated by The Foreman Bot about 3 years ago

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

#4 Updated by Timo Goebel about 3 years ago

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

#5 Updated by The Foreman Bot about 3 years ago

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

#6 Updated by The Foreman Bot about 3 years ago

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

#7 Updated by Ohad Levy about 3 years ago

  • Legacy Backlogs Release (now unused) set to 296
  • Pull request deleted (https://github.com/theforeman/foreman/pull/4922)

#8 Updated by The Foreman Bot about 3 years ago

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

#9 Updated by Timo Goebel about 3 years ago

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

Also available in: Atom PDF