Project

General

Profile

Bug #18166

Use ReactJsHelper on Sync controller

Added by Daniel Lobato Garcia over 1 year ago. Updated 2 months ago.

Status:
Closed
Priority:
Normal
Category:
Web UI
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Notifications will not be able to run without that helper on this page, as mount_react_component will not be available. Also the notifications_recipients_path will not be available if the proper url helpers are not included in the controller. The reason this is needed now is because the user dropdown in the top-right corner is calling those helpers on #18010


Related issues

Related to Katello - Bug #18188: undefined local variable error when navigating to red hat repositories pageClosed2017-01-20

Associated revisions

Revision 4dae79f3 (diff)
Added by Daniel Lobato Garcia over 1 year ago

Fixes #18166 - Add required helpers to Sync controller

Notifications will not be able to run without that helper on this page,
as mount_react_component will not be available. Also the
notifications_recipients_path will not be available if the
proper url helpers are not included in the controller. The
reason this is needed now is because the user dropdown in
the top-right corner is calling those helpers on #18010

History

#1 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Assignee set to Daniel Lobato Garcia
  • Pull request https://github.com/Katello/katello/pull/6560 added

#2 Updated by Anonymous over 1 year ago

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

#3 Updated by Justin Sherrill over 1 year ago

  • Related to Bug #18188: undefined local variable error when navigating to red hat repositories page added

#4 Updated by Daniel Lobato Garcia over 1 year ago

  • Target version set to 169

#5 Updated by Justin Sherrill over 1 year ago

  • Category set to Web UI
  • Target version deleted (169)
  • Legacy Backlogs Release (now unused) set to 211

#6 Updated by Daniel Lobato Garcia over 1 year ago

  • Target version set to 1.11.0

Also available in: Atom PDF