Skip to content

Remove internal note from docs #578

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

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

justinegeffen
Copy link
Contributor

@justinegeffen justinegeffen commented May 1, 2025

Removed an internal note from legacy content. We may still want to provide the images referenced in the note but we can create a Jira issue for that.

No formatting was changed in these pages, but we should consider reformatting them.

@justinegeffen justinegeffen added the 1. Editor review Needs a language review label May 1, 2025
Copy link

netlify bot commented May 1, 2025

Deploy Preview for seqera-docs ready!

Name Link
🔨 Latest commit 4b64340
🔍 Latest deploy log https://app.netlify.com/projects/seqera-docs/deploys/6874b94dba7dd40008c90ab2
😎 Deploy Preview https://deploy-preview-578--seqera-docs.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 project configuration.

@justinegeffen justinegeffen requested a review from llewellyn-sl May 1, 2025 12:59
Copy link
Member

@christopher-hakkaart christopher-hakkaart left a comment

Choose a reason for hiding this comment

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

I think there are other changes required.

Line 14 looks like AWS should be a heading, with text below. Similarly, for line 55 (Azure) and 83 (Google cloud).

Also note that the numbering for Google cloud starts at 5.

@justinegeffen
Copy link
Contributor Author

@llewellyn-sl, these pages don't seem to be in the LHS in prod but they're still part of the doc set (e.g., https://deploy-preview-578--seqera-docs.netlify.app/platform-enterprise/24.1/enterprise/advanced-topics/seqera-container-images). Are we still using this content?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. Editor review Needs a language review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants