Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add connecting section, pull from systems guides (and point back to main connecting section) #55

Merged
merged 7 commits into from
Sep 11, 2019

Conversation

isubsmith
Copy link
Contributor

@isubsmith isubsmith commented Sep 11, 2019

Pulled all the "connecting" text out of system user guides (minus "summit is at ssh username@summit.olcf.ornl.gov") and created a new top level connecting section.

closes #9

Copy link
Contributor

@jack-morrison jack-morrison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screen Shot 2019-09-11 at 6 51 59 AM

It looks like some of the headers may be at the wrong level, and the autogenerated left hand navigation is pulling in subsections. I would assume several of these should exist under a single 'Connecting' section that expands when clicked?

@grahamlopez
Copy link
Contributor

@jack-morrison @isubsmith I think I've got it. See what you think.

@@ -209,6 +209,8 @@ Home can be accessed through ssh utilities:

ssh userid@home.ccs.ornl.gov

For more information on connecting to OLCF resources, see the <.connecting>`__ section.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just noticed that we have two duplicated sections "Systems Available to All Projects" and "Connecting for the first time" in both the "Getting Started" and "Connecting" sections. It would be good to reconcile these somehow so we don't have duplicated content that can drift out of sync. Probably one should refer back to the other?

@grahamlopez
Copy link
Contributor

@jack-morrison @isubsmith I factored out the duplicated sections from "Getting Started." See what you think. I'm good for this one to go when you are.

@jack-morrison jack-morrison merged commit 3e623f6 into olcf:master Sep 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Connecting" - initial conversion or rewrite
3 participants