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.

Sign up for GitHub

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

Jump to bottom

ap7.ospool.osg-htc.org guide #181

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

ap7.ospool.osg-htc.org guide #181

wants to merge 37 commits into from

Conversation

rachellombardi
Copy link
Contributor

Guide requested by Brian B.

@rachellombardi
Create ap7-access.md
a750548
@rachellombardi
Create README.md
b72fe54
@rachellombardi
Rename start/account/ap7-access.md to start/account/AP7/ap7-access.md
579543a
@rachellombardi
Merge branch 'OSGConnect:master' into rachellombardi-AP7
681512f
@rachellombardi
Add image cilogon.png
1660916
@rachellombardi
Add link to cilogon.png image
e1a362e
@rachellombardi
Update ap7-access.md
9a3726b
@rachellombardi
update registry link
2c69c0c
@rachellombardi
Merge branch 'OSGConnect:master' into rachellombardi-AP7
22b86b2
@rachellombardi
Created images directory structure
9c61a70
@rachellombardi
add cilogon.png
e3201e8
@rachellombardi
Delete cilogon.png
49eb9b3
@rachellombardi
upload comanage-enrollment-form.png
dc85705
@rachellombardi
Add comanage-email-verification-form.png
641bf04
@rachellombardi
Add ssh-homepage-dropdown.png
e19beb6
@rachellombardi
Delete ssh-homepage-dropdown.png
d9a9ff3
@rachellombardi
Add new ssh-homepage-dropdown.png
1780675
@rachellombardi
Add ssh-edit-profile.png
efee588
@rachellombardi
Add ssh-authenticator-select.png and ssh-key-list.png
2057716
@rachellombardi
Update ap7-access.md
c275e40
@rachellombardi
Fixed disclamer at top
e3ffcfc
@ChristinaLK ChristinaLK self-requested a review July 8, 2022 14:49
ChristinaLK
ChristinaLK approved these changes Jul 8, 2022
View reviewed changes
Copy link
Contributor

@ChristinaLK ChristinaLK left a comment

Choose a reason for hiding this comment

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

Looks good to me! I might number the main headings (as they are three steps in a process) but it's not necessary.

Very minor - I changed "login" to "log in", because I think that's the correct usage?

start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Show resolved Hide resolved
rachellombardi and others added 5 commits July 8, 2022 13:40
@rachellombardi @ChristinaLK
Fixed "login" grammer
9c63efb 
Co-authored-by: Christina K. <[email protected]>
@rachellombardi @ChristinaLK
Fixed "Log in" grammer
10bbaa9 
Co-authored-by: Christina K. <[email protected]>
@rachellombardi @ChristinaLK
Fixed "Log in" grammer
f4d7fec 
Co-authored-by: Christina K. <[email protected]>
@rachellombardi @ChristinaLK
Fixed "Log in" grammer
ceaf9b3 
Co-authored-by: Christina K. <[email protected]>
@rachellombardi @ChristinaLK
Fixed "Log in" grammer
a73b86f 
Co-authored-by: Christina K. <[email protected]>
rachellombardi
rachellombardi commented Jul 8, 2022
View reviewed changes
Copy link
Contributor Author

@rachellombardi rachellombardi left a comment

Choose a reason for hiding this comment

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

Minor formatting edits

start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
start/account/AP7/ap7-access.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChristinaLK ChristinaLK ChristinaLK approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rachellombardi @ChristinaLK