-
Notifications
You must be signed in to change notification settings - Fork 199
Updates the Cloud Connect page FAQ #4988
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
Conversation
Vale Linting ResultsSummary: 1 warning, 4 suggestions found
|
| File | Line | Rule | Message |
|---|---|---|---|
| deploy-manage/_snippets/eis-cc-payment-faq.md | 1 | Elastic.DontUse | Don't use 'Note that'. |
💡 Suggestions (4)
| File | Line | Rule | Message |
|---|---|---|---|
| deploy-manage/cloud-connect.md | 46 | Elastic.FirstPerson | Avoid first-person pronouns such as ' I '. |
| deploy-manage/cloud-connect.md | 46 | Elastic.FirstPerson | Avoid first-person pronouns such as 'my'. |
| deploy-manage/cloud-connect.md | 61 | Elastic.FirstPerson | Avoid first-person pronouns such as ' I '. |
| deploy-manage/cloud-connect.md | 61 | Elastic.FirstPerson | Avoid first-person pronouns such as 'my'. |
The Vale linter checks documentation changes against the Elastic Docs style guide.
To use Vale locally or report issues, refer to Elastic style guide for Vale.
🔍 Preview links for changed docs |
wajihaparvez
left a comment
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.
LGTM! Small wording change suggested
Co-authored-by: wajihaparvez <wajiha.parvez@elastic.co>
maxjakob
left a comment
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.
Content looks good, approving 👍 Made some suggestions on wording and to potentially repeat some infos
Co-authored-by: Max Jakob <max.jakob@elastic.co>
leemthompo
left a comment
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.
LGTM with minor suggestion
alaudazzi
left a comment
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.
LGTM
This PR updates the Cloud Connect FAQ to reflect the current availability of EIS and clarifies that additional services (such as Synthetic) may be supported in the future.