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

Add general notes for deployments #554

Merged
merged 9 commits into from
Mar 12, 2024

Conversation

sztupy
Copy link
Contributor

@sztupy sztupy commented Jan 27, 2024

For more info see #515

This is splitting out that PR into multiple parts. This is part 1 - the core description for deployments

See also

For more info see CodeYourFuture#515

This is splitting out that PR into multiple parts. This is part 1 - the core description for deployments
Copy link

netlify bot commented Jan 27, 2024

Deploy Preview for cyf-curriculum failed.

Name Link
🔨 Latest commit 91ba94d
🔍 Latest deploy log https://app.netlify.com/sites/cyf-curriculum/deploys/65f06be3f26398000822ad25

@sztupy
Copy link
Contributor Author

sztupy commented Jan 27, 2024

@sztupy sztupy mentioned this pull request Jan 27, 2024
5 tasks
@Dedekind561
Copy link
Contributor

@sztupy Your deploy preview isn't building here - how are you planning on linking to these pages on the /guides page?

Copy link

netlify bot commented Mar 12, 2024

👷 Deploy request for curriculum-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 91ba94d

@Dedekind561
Copy link
Contributor

@sztupy I've fixed the issue - you were using incorrect links to point to the netlify guide. Now it is building properly! 😄

Copy link
Contributor

@Dedekind561 Dedekind561 left a comment

Choose a reason for hiding this comment

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

@sztupy Thanks for setting up this guide - and splitting up the PRs!
Let's get this merged and we can start iterating from there...

@Dedekind561 Dedekind561 merged commit 302ae13 into CodeYourFuture:main Mar 12, 2024
2 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants