Skip to content

Rudimentary permissions docs #303

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

Merged
merged 3 commits into from
Jul 13, 2021
Merged

Rudimentary permissions docs #303

merged 3 commits into from
Jul 13, 2021

Conversation

sebbacon
Copy link
Contributor

@sebbacon sebbacon commented Jul 7, 2021

No description provided.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 7, 2021

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5d010eb
Status: ✅  Deploy successful!
Preview URL: https://01675c03.opensafely-docs.pages.dev

View logs

Copy link
Contributor

@ghickman ghickman left a comment

Choose a reason for hiding this comment

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

Content looks good.

Small nit around document layout.


After the data controller has [approved your project](https://www.opensafely.org/onboarding-new-users/), we will add any Github usernames listed in your approval to our `opensafely` Github organisation. We also will transfer your existing OpenSAFELY study repository (if you have one) into the same organisation. This allows OpenSAFELY to enforce certain security standards, such as [multi-factor authentication](https://docs.github.com/en/github/authenticating-to-github/securing-your-account-with-two-factor-authentication-2fa).

Additionally, the OpenSAFELY framework requires a user to be in the `opensafely` repository in order to [run jobs in the secure backends](job-server.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

s/repository/GitHub organisation/

for OpenSAFELY's data. We are currently developing an online application
process, but for now just drop us an email at
[team@opensafely.org](mailto:team@opensafely.org).
for OpenSAFELY's data. [Read about our pilot onboarding programme here](https://www.opensafely.org/onboarding-new-users/). Once approved, your Github user account will be added to our `opensafely` Github organisation, along with your study repository, which gives you the ability to run your study against real data. [Read more about permissions here](permissions.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you wrap this to match the rest of the document?

@sebbacon sebbacon force-pushed the rudimentary-permissions-docs branch from 77eaefc to 5d010eb Compare July 13, 2021 09:00
@sebbacon sebbacon merged commit 032cb3c into main Jul 13, 2021
@sebbacon sebbacon deleted the rudimentary-permissions-docs branch July 13, 2021 09:09
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.

3 participants