-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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] Relationships: Grammar fix in the "Design Relationships and Restrictions" section #10003
Changes from 3 commits
0078f7b
6dd3a2d
86ecf9a
ba30f01
1dac8c0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,6 +6,8 @@ permalink: guides/configuration-management/importing-apps | |
category: configuration | ||
type: guides | ||
language: en | ||
redirect_from: | ||
- /installation/quick-start.md | ||
leecalcote marked this conversation as resolved.
Show resolved
Hide resolved
|
||
--- | ||
|
||
Import your existing application definitions and your existing infrastructure configruations into Meshery | ||
|
@@ -50,7 +52,7 @@ This command enable users to import their existing applications from sources as | |
|
||
**Step 1: Access the Meshery UI** | ||
|
||
To import a Docker Compose app into Meshery using the Meshery UI, you must first [install Meshery](../installation/quick-start.md) | ||
To import a Docker Compose app into Meshery using the Meshery UI, you must first [install Meshery](/installation/quick-start) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why are there changes here -- commits from another issue perhaps(?), this isn't part of the issue description/request. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Those are commits from the issue #9991 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for this review, @sandramsc 👏 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @neerajmanivarnan, don't include commits from a separate PR into another PR. |
||
|
||
**Step 2: Navigate to the Application section in the Configuration** | ||
|
||
|
@@ -67,3 +69,4 @@ When you import an app into Meshery, it will create a Meshery App based on defin | |
<a href="{{ site.baseurl }}/assets/img/applications/ImportApp.png"><img alt="Import-Application" style="width:500px;height:auto;" src="{{ site.baseurl }}/assets/img/applications/ImportApp.png" /></a> | ||
|
||
Once the Meshery App has been created, you can use Meshery to manage, operate and observe your cloud native infrastructure. You can also use Meshery to deploy your Meshery App to any of your connected kubernetes clusters. For more information, see [connections](/installation/kubernetes) | ||
|
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. This is not only inaccurate, but improper syntax.