-
Notifications
You must be signed in to change notification settings - Fork 40
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
docs: Add ORCID login documentation and email setup steps #3180
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3180 +/- ##
==========================================
+ Coverage 42.84% 44.16% +1.31%
==========================================
Files 549 593 +44
Lines 36187 37838 +1651
Branches 1132 1132
==========================================
+ Hits 15506 16710 +1204
- Misses 20479 20926 +447
Partials 202 202 ☔ View full report in Codecov by Sentry. |
docs/assets/orcid/step-3.png
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docs/oricd.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo in filename. s/oricd/orcid/
Co-authored-by: Chris Markiewicz <markiewicz@stanford.edu>
Should add this to the TOC: Lines 45 to 50 in 0bf65b0
|
Co-authored-by: Chris Markiewicz <markiewicz@stanford.edu>
e540935
to
6e71c04
Compare
Some basic documentation for ORCID accounts on OpenNeuro explaining how accounts work and what is done with the information gathered. Steps included for making an email visible to OpenNeuro.
@effigies This is what I'd like to link to from the warning that no email was readable when a user is logged in with ORCID, what do you think of the wording?