Feature #17990

Use foreman's toast notification instead of inline bst-alert

Added by Walden Raines over 1 year ago. Updated 7 days ago.

Status:Closed
Priority:Normal
Assignee:Walden Raines
Category:-
Target version:Katello - Katello Backlog
Difficulty:medium Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link:1412596 Found in Releases:
Pull request:https://github.com/Katello/bastion/pull/172

Description

We should use foreman's notify() function instead of our inline bst-alerts for success and error notifications.


Related issues

Blocked by Foreman - Bug #17991: Javascript notify() function should not require a <p> ele... New 01/09/2017
Blocks Katello - Refactor #18288: Updates to katello to use foreman toast notifications ins... Closed 01/27/2017
Blocked by Foreman - Refactor #14178: Replace jnotify with patternfly toast notifications Closed 03/14/2016

Associated revisions

Revision e8d37c58
Added by Walden Raines 10 months ago

Fixes #17990: use toast notifications like foreman.

Instead of using inline notifications we should be using toast
notifications like foreman does in order to match their style and UX.

http://projects.theforeman.org/issues/17990

History

#1 Updated by Walden Raines over 1 year ago

  • Priority changed from Normal to High

#2 Updated by Walden Raines over 1 year ago

  • Tracker changed from Bug to Feature

#3 Updated by Walden Raines over 1 year ago

  • Blocked by Bug #17991: Javascript notify() function should not require a <p> element added

#4 Updated by Justin Sherrill over 1 year ago

  • Priority changed from High to Normal
  • Legacy Backlogs Release (now unused) set to 114

#5 Updated by Walden Raines over 1 year ago

  • Bugzilla link set to 1412596

#6 Updated by Walden Raines over 1 year ago

  • Target version changed from 155 to 161

#7 Updated by Walden Raines over 1 year ago

  • Project changed from Katello to Bastion
  • Category deleted (Web UI)

#8 Updated by Walden Raines over 1 year ago

  • Blocks Refactor #18288: Updates to katello to use foreman toast notifications instead of bst-alert added

#9 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/bastion/pull/172 added

#10 Updated by Walden Raines over 1 year ago

Need to move to iteration 11 as we wait on https://github.com/theforeman/foreman/pull/4007

#11 Updated by Walden Raines over 1 year ago

  • Target version changed from 161 to 166

#12 Updated by Walden Raines over 1 year ago

  • Blocked by Refactor #14178: Replace jnotify with patternfly toast notifications added

#13 Updated by Walden Raines over 1 year ago

  • Target version changed from 166 to 185

#14 Updated by Thomas McKay about 1 year ago

  • Target version changed from 185 to 194

#15 Updated by Thomas McKay about 1 year ago

  • Target version changed from 194 to 195

#16 Updated by Walden Raines 10 months ago

  • Target version changed from 195 to 213

#17 Updated by Walden Raines 10 months ago

  • Target version changed from 213 to 219

#18 Updated by Walden Raines 10 months ago

  • Assignee deleted (Walden Raines)

#19 Updated by Walden Raines 10 months ago

  • Assignee set to Walden Raines

#20 Updated by Walden Raines 10 months ago

  • Target version changed from 219 to 218

#21 Updated by Walden Raines 10 months ago

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

Also available in: Atom PDF