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

Deploy docs - add GH linking info #78

Merged
merged 6 commits into from
Dec 24, 2024
Merged

Deploy docs - add GH linking info #78

merged 6 commits into from
Dec 24, 2024

Conversation

jacklynlee31
Copy link
Contributor

@jacklynlee31 jacklynlee31 commented Dec 17, 2024

Added info explaining how to link your Modus app to GH using the console & the CLI. Note that this info is very similar to what we already have in the Quickstart, but I think that's okay.

Tried to make this somewhat concise, but are there things we should add? (Or remove?) 🤔

(For reference, message in Slack)

@jacklynlee31 jacklynlee31 marked this pull request as ready for review December 17, 2024 21:03
@jacklynlee31 jacklynlee31 requested a review from a team as a code owner December 17, 2024 21:03
@ryanfoxtyler
Copy link
Member

@jacklynlee31 this is documented in Create Project today, which I think makes more sense given it's a one-time operation when initializing a project (versus with each deployment)–what do you think?

@jacklynlee31
Copy link
Contributor Author

@jacklynlee31 this is documented in Create Project today, which I think makes more sense given it's a one-time operation when initializing a project (versus with each deployment)–what do you think?

@ryanfoxtyler - That makes sense. Maybe the better way to reference it on the Deploys page, then, is to not document the entire process but just link to that 'Create Project' page instead.

@ryanfoxtyler ryanfoxtyler merged commit b0ce8df into main Dec 24, 2024
5 checks passed
@ryanfoxtyler ryanfoxtyler deleted the jacklyn/deploy-docs branch December 24, 2024 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants