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

docs: Add Cloud Docs #7428

Merged
merged 7 commits into from
Mar 24, 2023
Merged

docs: Add Cloud Docs #7428

merged 7 commits into from
Mar 24, 2023

Conversation

tayloramurphy
Copy link
Collaborator

@tayloramurphy tayloramurphy commented Mar 22, 2023

This takes the content from https://github.com/meltano/cloud-docs and pulls it into the main docs.

@tayloramurphy tayloramurphy requested review from sbalnojan and a team as code owners March 22, 2023 21:41
@netlify
Copy link

netlify bot commented Mar 22, 2023

Deploy Preview for meltano ready!

Name Link
🔨 Latest commit 0d49239
🔍 Latest deploy log https://app.netlify.com/sites/meltano/deploys/641dbab1fadaee000882a8d8
😎 Deploy Preview https://deploy-preview-7428--meltano.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.

@tayloramurphy tayloramurphy changed the title Add Cloud Docs docs: Add Cloud Docs Mar 22, 2023
@WillDaSilva
Copy link
Member

@tayloramurphy Will we be archiving https://github.com/meltano/cloud-docs after this has been merged?

@tayloramurphy
Copy link
Collaborator Author

@WillDaSilva yes. No need for the duplicate content. I can handle moving the PRs and any relevant issues (there's not many!)

@aaronsteers
Copy link
Contributor

aaronsteers commented Mar 22, 2023

@tayloramurphy, can we take an iteration to remove/replace stale content before merging?

Probably save to assume we can close this out by EOD Thursday and merge whatever is ready by Friday AM.

In the meanwhile, we can use the URL here to preview: https://deploy-preview-7428--meltano.netlify.app/cloud/concepts

@tayloramurphy
Copy link
Collaborator Author

can we take an iteration to remove/replace stale content before merging?

@aaronsteers absolutely!

@tayloramurphy
Copy link
Collaborator Author

@aaronsteers I added an info warning to the top of every page.

Copy link
Member

@WillDaSilva WillDaSilva left a comment

Choose a reason for hiding this comment

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

I'm approving this before I consider it stable so that this can be published prior to a meeting in which it will be shared.

Follow-up PRs should promptly address:

docs/src/_cloud/security.md Outdated Show resolved Hide resolved
Co-authored-by: Will Da Silva <will@willdasilva.xyz>
@tayloramurphy
Copy link
Collaborator Author

Merging!

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.

None yet

4 participants