-
Notifications
You must be signed in to change notification settings - Fork 405
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
Update Application Connector documentation #1286
Update Application Connector documentation #1286
Conversation
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.
Two minor changes
@@ -18,4 +18,4 @@ The AC gives you the following: | |||
|
|||
All of the AC components scale independently, which allows to adjust it to fit the needs of the implementation built using Kyma. | |||
|
|||
>**NOTE:** To learn more about the Environments in Kyma, read the **Environments** document in the **Kyma** documentation topic. | |||
>**NOTE:** To learn more about the concept Environments in Kyma, read the **Environments** document in the **Kyma** documentation topic. |
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.
concept > concept of?
@@ -2,11 +2,11 @@ | |||
title: Overview | |||
--- | |||
|
|||
The Application Connector (AC) is a proprietary Kyma implementation that allows you to connect with external solutions. No matter if you want to integrate an on-premise or a cloud system, the integration process doesn't change, which allows to avoid any configuration and network-related problems. | |||
The Application Connector (AC) is a proprietary Kyma implementation that allows you to connect with external solutions. No matter if you want to integrate an on-premise or a cloud system, the integration process doesn't change, which allows to avoid any configuration or network-related problems. |
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.
doesn't > does not
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.
No need to be so strict about contractions. We allow them where they sound natural, that's the consensus ;)
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.
Sure, got it :)
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.
Approved
Changes proposed in this pull request:
Related issue(s)
See #1283