Project

General

Profile

Bug #24218

Documentation link on Content Host Registration page should use documentation helper

Added by Adam Ruzicka over 1 year ago. Updated over 1 year ago.

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

Description

Description of problem:
Documentation linked from Registration page points to katello plugin docs on theforeman.org, not RH docs

How reproducible:

Steps to Reproduce:
1. Navigate to Content Host Registration page in UI
2. Note the text at the bottom, "Learn more about these packages in the Documentation"
3. Click the 'Documentation' link

Actual results:
User is taken to foreman katello plugin docs

Expected results:
RH Docs

Additional info:

History

#1 Updated by Adam Ruzicka over 1 year ago

  • Category set to Web UI

The content host registration page needs to use a documentation_url helper to link to the docs so the link can be diverted in downstream theme.

#2 Updated by Justin Sherrill over 1 year ago

  • Description updated (diff)
  • Subject changed from Documentation link on Content Host Registration page leads to upstream docs to Documentation link on Content Host Registration page should use documentation helper

#3 Updated by Justin Sherrill over 1 year ago

  • Target version set to Katello 3.7.1

#4 Updated by Jonathon Turel over 1 year ago

  • Target version changed from Katello 3.7.1 to 931

#5 Updated by Jonathon Turel over 1 year ago

Adam, do you have an example of the aforementioned 'documentation helper' to help whoever works on this (including me)?

#6 Updated by Adam Ruzicka over 1 year ago

In Foreman there's the documentation_url1 helper. In downstream it gets overriden to return links to the product documentation.

[1] - https://github.com/theforeman/foreman/blob/develop/app/helpers/application_helper.rb#L453

#7 Updated by Christine Fouant over 1 year ago

  • Status changed from New to Closed

#8 Updated by Jonathon Turel over 1 year ago

  • Target version changed from 931 to Katello 3.9.0

Also available in: Atom PDF