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

Improve Repo Sharing Docs #184

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

michaeljguarino
Copy link
Member

Summary

This was missing some key information on who should run what commands and what you need to run to decrypt a cloned repo (we should probably also write a plural clone command that compresses that into one command)

Test Plan

n/a

Checklist

  • If required, I have updated the Plural documentation accordingly.
  • I have added tests to cover my changes.
  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • I have added relevant labels to this PR to help with categorization for release notes.

This was missing some key information on who should run what commands and what you need to run to decrypt a cloned repo (we should probably also write a `plural clone` command that compresses that into one command)
@michaeljguarino michaeljguarino added tutorial For educational resources. new-content For new docs content. labels Apr 11, 2023
@michaeljguarino michaeljguarino requested a review from a team as a code owner April 11, 2023 23:22
@netlify
Copy link

netlify bot commented Apr 11, 2023

Deploy Preview for pluraldocs ready!

Name Link
🔨 Latest commit f06528a
🔍 Latest deploy log https://app.netlify.com/sites/pluraldocs/deploys/648b5018a72fb70008cd48a1
😎 Deploy Preview https://deploy-preview-184--pluraldocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

### Register a public key

To register a key for your current machine, run:
First, Alice and Bob both need to register age keypairs with plural. To do that register, they should both run this command on their current machine:
Copy link
Collaborator

@dogmar dogmar Jun 15, 2023

Choose a reason for hiding this comment

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

Suggested change
First, Alice and Bob both need to register age keypairs with plural. To do that register, they should both run this command on their current machine:
First, Alice and Bob both need to register age keypairs with Plural. To do that registration, they should both run this command on their current machine:

plural deploy --commit "set up encryption"
```

Regardless, whether the console is installed or not, you should be sure the files written locally were pushed upstream, which can also be done with:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Regardless, whether the console is installed or not, you should be sure the files written locally were pushed upstream, which can also be done with:
Regardless of whether the console is installed or not, you should ensure the locally-written files were pushed upstream, which can be done with:


### Bob Decrypts the Repository

Now that the keypairs are registered, and repo encryption has moved to using age, Bob should be able to freely decrypt the repo, which is done simply by:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Now that the keypairs are registered, and repo encryption has moved to using age, Bob should be able to freely decrypt the repo, which is done simply by:
Now that the keypairs are registered and repo encryption has moved to using age, Bob should be able to freely decrypt the repo, which is done simply by:


### Helpful Hints

- Encryption keys are stored locally or in plural's keybackups, so if new teammate gets a new machine, you'll likely need to do this process again for that teammate.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Encryption keys are stored locally or in plural's keybackups, so if new teammate gets a new machine, you'll likely need to do this process again for that teammate.
- Encryption keys are stored locally or in Plural's keybackups, so if a new teammate gets a new machine, you'll likely need to do this process again for that teammate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-content For new docs content. tutorial For educational resources.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants