Skip to content
This repository has been archived by the owner on Aug 29, 2018. It is now read-only.

help_helper.rb: update SSH help page link #6386

Closed
wants to merge 1 commit into from

Conversation

thrasher-redhat
Copy link
Contributor

Bug 1331745
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1331745

The previous link for the SSH help page was
https://www.openshift.com/developers/remote-access. This page used to
redirect to the newer page
https://developers.openshift.com/managing-your-applications/remote-connection.html,
but no longer redirects and now will 404. This commit updates the links to
point to the correct page.

Bug 1331745
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1331745

The previous link for the SSH help page was
<https://www.openshift.com/developers/remote-access>.  This page used to
redirect to the newer page
<https://developers.openshift.com/managing-your-applications/remote-connection.html>,
but no longer redirects and now will 404.  This commit updates the links to
point to the correct page.
@thrasher-redhat
Copy link
Contributor Author

[test], please! Want to make sure theres no tests with a hardcoded link...

@thrasher-redhat
Copy link
Contributor Author

@Miciah @tiwillia

Easy fix to review at your leisure.

@Miciah
Copy link
Contributor

Miciah commented May 6, 2016

Please [test] again!

@thrasher-redhat
Copy link
Contributor Author

Deleted my extended request, can we do a regular [test]?

@openshift-bot
Copy link

Evaluated for online test up to db234eb

@openshift-bot
Copy link

@thrasher-redhat
Copy link
Contributor Author

@luciddreamz
Just want to double-check that these links are being updated to the correct links.

@luciddreamz
Copy link
Contributor

Looks good. On a related note, the issue causing the 404 has been resolved on the www.openshift.com site last week.

@thrasher-redhat
Copy link
Contributor Author

Ahhhhh nice. Does the 404 being resolved make this a non-issue then? Or should we go ahead with the update to avoid the redirect anyways?

@luciddreamz
Copy link
Contributor

Correct. This is a non-issue. I'd say just leave it. If there is a failure we probably need to fix the www site.

@thrasher-redhat
Copy link
Contributor Author

Sounds good - thanks. Closing this and the related bug.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants