Project

General

Profile

Bug #4459

UI plugin pages error finding root_path

Added by Thomas McKay almost 7 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

Encountered during UI plugin work

Fix is to change in _topbar.html.erb
<%= link_to "Foreman", root_path %>
to
<%= link_to "Foreman", main_app.root_path %>


Related issues

Related to Foreman - Bug #4194: session timeout fails to redirect to foreman login screenClosed2014-01-27

Associated revisions

Revision bce4494d (diff)
Added by Thomas McKay almost 7 years ago

fixes #4459 - allows proper rendering of pages using root_path

Revision ccc1c1e3
Added by Thomas McKay almost 7 years ago

Merge pull request #3751 from thomasmckay/4459

fixes #4459 - allows proper rendering of pages using root_path

Revision 33b384f2 (diff)
Added by Thomas McKay over 6 years ago

fixes #4194, #4459 - add main_app to root_path references for isolated engines

Revision 24de6ee3 (diff)
Added by Thomas McKay over 6 years ago

fixes #4194, #4459 - add main_app to root_path references for isolated engines

(cherry picked from commit 33b384f2b29363014608ac1b85061b5a8b4791d5)

History

#1 Updated by Dominic Cleal almost 7 years ago

  • Status changed from Assigned to Ready For Testing
  • Target version set to 1.9.1

#2 Updated by Thomas McKay almost 7 years ago

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

Applied in changeset katello:katello|commit:bce4494d8c3d1d1d017db6b7e2fffae5a8f04203.

#3 Updated by Thomas McKay almost 7 years ago

  • Status changed from Closed to Ready For Testing

#4 Updated by Dmitri Dolguikh almost 7 years ago

  • Target version changed from 1.9.1 to 1.9.0

#5 Updated by Dominic Cleal over 6 years ago

  • Legacy Backlogs Release (now unused) set to 7

#6 Updated by Dominic Cleal over 6 years ago

  • Related to Bug #4194: session timeout fails to redirect to foreman login screen added

#7 Updated by Thomas McKay over 6 years ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF