Bug 1769251
Summary: | Updating the Context-Sensitive Help Links for 6.6 | ||
---|---|---|---|
Product: | Red Hat Satellite | Reporter: | Sergei Petrosian <spetrosi> |
Component: | Branding | Assignee: | Sergei Petrosian <spetrosi> |
Status: | CLOSED ERRATA | QA Contact: | Radovan Drazny <rdrazny> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | 6.6.0 | CC: | egolov, inecas, lzap, mkalyat, sshtein, vgrosu, vparekh |
Target Milestone: | 6.7.0 | Keywords: | Triaged |
Target Release: | Unused | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | tfm-rubygem-foreman_theme_satellite-5.0.1.3-1 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2020-04-14 13:26:24 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Sergei Petrosian
2019-11-06 09:54:11 UTC
*** Bug 1759021 has been marked as a duplicate of this bug. *** *** Bug 1644176 has been marked as a duplicate of this bug. *** Hi lzap, I am trying to improve UX with documentation buttons in the Satelltei web UI. We bumped upon an issue that must be fixed upstream first. Can you please take a look? As can be seen from the grep command below, the "Provisioning" documentation button is used on two pages, on /hosts/help and /domains/help. Now the link for "Provisioning" documentation button refers to "/provisioning_guide". It is fine for both cases, however, it is definitely better to create unique links for each page. Is this possible to define different links for each of these buttons? ~~~~ $ grep -R --include *.*rb "Provisioning\")%" foreman/ foreman/app/views/domains/welcome.html.erb: <p><%= link_to _('Learn more about this in the documentation.'), documentation_url("4.4Provisioning")%></p> foreman/app/views/hosts/welcome.html.erb: <p><%= link_to _('Learn more about this in the documentation.'), documentation_url("4.4Provisioning")%></p> ~~~~ Thank you Hello, apologies for the delay. I was on PTO. What is the status here? (In reply to Lukas Zapletal from comment #11) > Hello, apologies for the delay. I was on PTO. > > What is the status here? What I asked you in comment#10 still stands. For now, the button in both places leads to the same place, which is the Administering guide. This works fine, but this is not ideal. Simon suggested changing this upstream, then we will be able to set this properly in the downstream. Do you know how to set different documentation buttons in those places? Thank you I don't know exactly how the branding plugin works yet, however do you want from me to change those lines to link to some different unique archor? Then we could do something like: foreman/app/views/domains/welcome.html.erb: <p><%= link_to _('Learn more about this in the documentation.'), documentation_url("4.4Provisioning")%></p> foreman/app/views/hosts/welcome.html.erb: <p><%= link_to _('Learn more about this in the documentation.'), documentation_url("4.4.7Networking")%></p> Feel free to file a patch against Foreman core with such change if that makes life easier for you. I do not recommend some big refactoring of upstream docs tho, we are hopefully migrating to upstreamized satellite documentation next year. Hi Shimon, I like the idea lzap had suggested to use the 4.4.7Networking button for domains help. This button is also used on the Subnets page, and we describe creating both domains and subnets in the COnfiguring Networking section in the Provisioning guide. I have created a Redmine issue [1] and pushed a PR [2], and lzap has already merged it to the Foreman development branch. lzap told me that he will try to make this change cherry-picked downstream for 6.7. [1] https://projects.theforeman.org/issues/28334 [2] https://github.com/theforeman/foreman/pull/7196 Thank you I'm going through all updated links (I hope, because this BZ doesn't explicitly say which links has changed, so I'm using linked PRs to see what links should get checked). I have found a context sensitive link that points to unbranded Foreman documentation. Go to Monitor->Audits and click the Documentation button. You will get https://theforeman.org/manuals/1.24/index.html#4.1.4Auditing instead of official doc on access.redhat.com. Which part of docs should this link point to I don't know. Only mention of Audits I have found in the official docs is here: https://access.redhat.com/documentation/en-us/red_hat_satellite/6.7-beta/html/administering_red_hat_satellite/chap-red_hat_satellite-administering_red_hat_satellite-maintaining_a_red_hat_satellite_server#sect-Red_Hat_Satellite-Administering_Red_Hat_Satellite-Deleting_Audit_Records Sergei, can we fix this in this BZ, or should I submit a new separate BZ? Hi Radovan, The link that you shared contains the only procedures related to Satellite. Also, there is some context information about what Audits do in the Planning guide, direct link: https://access.redhat.com/documentation/en-us/red_hat_satellite/6.6/html/planning_for_red_hat_satellite_6/appe-red_hat_satellite-architecture_guide-glossary_of_terms#varl-Glossary_of_Terms-Audits By default, all not-defined Satellite documentation buttons must point to the Admin guide. The fact that this link points to Foreman probably means that this link will require some more work. > Sergei, can we fix this in this BZ, or should I submit a new separate BZ? I will definitely file another bug to review links for 6.7 GA. I know that the new Azure CR was added so we will need to add a link from this Azure CR page to docs. Thank you I have checked all context sensitive links I could find in Satellite 6.7. No real problems found. The non-branded link mentioned in comment #18 is tracked in BZ1771484. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2020:1454 *** Bug 1808547 has been marked as a duplicate of this bug. *** |