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

User Guide: Add section about remote authentication (experimental) #2975

Closed
eaquigley opened this issue Feb 26, 2016 · 12 comments
Closed

User Guide: Add section about remote authentication (experimental) #2975

eaquigley opened this issue Feb 26, 2016 · 12 comments
Assignees
Milestone

Comments

@eaquigley
Copy link
Contributor

Need to add a section about remote authentication into the user guide

@eaquigley eaquigley self-assigned this Feb 26, 2016
@pdurbin pdurbin changed the title User Guide: Add section about remote authentication User Guide: Add section about remote authentication (experimental) Feb 26, 2016
@pdurbin
Copy link
Member

pdurbin commented Mar 8, 2016

@eaquigley In 21bd0e8 I improved the Shibboleth section of the Installation Guide and the change can be previewed at http://guides.dataverse.org/en/2939-shib/installation/shibboleth.html . Please read through the top part as I tried to keep it high level (local vs. remote, etc.).

Please make your commit(s) to the 2939-shib branch. Thanks!

@pdurbin
Copy link
Member

pdurbin commented Mar 10, 2016

@eaquigley please note that there is some new feedback from @bencomp in #2953 that relates to the User Guide.

eaquigley added a commit that referenced this issue Mar 10, 2016
This is for #2975. still need to add pictures in.
pdurbin pushed a commit that referenced this issue Mar 10, 2016
This is for #2975. still need to add pictures in.
@pdurbin
Copy link
Member

pdurbin commented Mar 10, 2016

@eaquigley 8e7d93a looks good but there are some typos. Please advise if you'd like me to assign myself to this issue and fix them.

Also, are you blocked on #3011? It looks like you'd like to add some screenshots.

@pdurbin
Copy link
Member

pdurbin commented Mar 15, 2016

@eaquigley I just deployed b6c638d (a fix for #3011) to https://shibtest.dataverse.org so if you want to grab screenshots you can. It looks like this:

log_in_-root_dataverse-_2016-03-15_16 02 23

Or do you need the Harvard Identity Provider (IdP) rather than the TestShib IdP? Are you blocked on getting the screenshots you need?

@pdurbin
Copy link
Member

pdurbin commented Mar 16, 2016

@eaquigley I just made a pull request at #3025 and in the RFI checklist I indicated that me checking "All code checkins completed" is blocked on this issue. Should we simply remove the references to the screenshots? As of 15e15d1 http://guides.dataverse.org/en/2939-shib/user/account.html looks like this with broken links to images:

account_creation_ management dataverse org-_2016-03-16_16 27 15

@eaquigley
Copy link
Contributor Author

@pdurbin I went onto beta today to take the screenshots and it doesn't seem to be the most recent shibb code (text wasn't correct). what machine is running it so i can complete this section of the guide?

@pdurbin
Copy link
Member

pdurbin commented Mar 16, 2016

The shibtest server per #2975 (comment)

@eaquigley
Copy link
Contributor Author

@pdurbin excellent! thanks. i'll need your help with getting the screenshot for converting a local account to a shibb account.

also, i'm noticing issues on shibtest.dataverse.org where i keep getting thrown to this:

screen shot 2016-03-16 at 5 30 55 pm

@pdurbin
Copy link
Member

pdurbin commented Mar 16, 2016

Oh. Sorry. If you publish the root dataverse that error should go away.

posixeleni added a commit that referenced this issue Mar 17, 2016
posixeleni added a commit that referenced this issue Mar 17, 2016
eaquigley added a commit that referenced this issue Mar 17, 2016
posixeleni added a commit that referenced this issue Mar 17, 2016
@pdurbin
Copy link
Member

pdurbin commented Mar 17, 2016

@eaquigley as we discussed I'm quite happy with the screenshots you added (thanks!) and removed the note about this being a blocker for pull request #3025 but I understand if you want to do some more reformatting. Please just pass this issue to me when you're done and I'll make sure it makes its way to QA. Thanks!!

pdurbin added a commit that referenced this issue Mar 21, 2016
Grammar and punctuation fixed. #2975

Yeah, "hearder" is a typo and I like how a few longish sentences have been broken up (though I'm often guilty of Faulknerian sentences myself). Merging.
@pdurbin
Copy link
Member

pdurbin commented Mar 21, 2016

@eaquigley heads up the the login page just changed slightly. I added the "Learn More" link to the User Guide you asked for. You can see a screenshot at #2951 (comment)

@kcondon kcondon self-assigned this Mar 25, 2016
@kcondon
Copy link
Contributor

kcondon commented Mar 25, 2016

OK, user guide section in place, reviewed. Closing. @eaquigley I've reviewed this and looks good so closing. Phil mentioned a slight change to log in text in case you wanted to update your screen shot. Seems ok to me.

@pdurbin pdurbin added this to the 4.4 milestone Jun 30, 2016
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

No branches or pull requests

3 participants