-
-
Notifications
You must be signed in to change notification settings - Fork 900
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
Update deploying docs for clarity #310
Conversation
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.
Approved, but left some small comments!
|
||
## Deploying your App | ||
### Steps for Deploying Properly |
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.
Hah, why Properly?
|
||
## Deploying your App | ||
### Steps for Deploying Properly | ||
|
||
These are the steps necessary for you to deploy your app properly. We recommend you follow these steps in order |
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.
Ha also properly -> how would you deploy it otherwise :D?
These are the steps necessary for you to deploy your app properly. We recommend you follow these steps in order | ||
|
||
- [ ] Get your [production API keys and environment variables](#prerequisites) | ||
- [ ] [Deploy your app](#deploying-to-flyio) |
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.
Maybe also provide link for manual deployment if they want it? (Fly or Manual)?
Description
Fixes #275
Contributor Checklist