Skip to content

Ftps connector edit#45

Open
jordan-modusbox wants to merge 5 commits intolatestfrom
ftps-connector-edit
Open

Ftps connector edit#45
jordan-modusbox wants to merge 5 commits intolatestfrom
ftps-connector-edit

Conversation

@jordan-modusbox
Copy link
Copy Markdown
Contributor

FTPS update and edit. Open for review

@jordan-modusbox jordan-modusbox requested a review from IanWhyte July 22, 2020 18:54
Comment thread ftps-connector/modules/ROOT/nav.adoc Outdated
@@ -1,2 +1,2 @@
* xref:ftps-connector.adoc[FTPS Connector User Guide]

** xref:use-case-example.adoc[Use Case Example - Hello World]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think I'd go with a structure here similar to the one we're talking about for JDE, something like this:

  • FTPS Connector User Guide
    ** Intro
    ** Prerequisites
    ** Installation
    ** Configuration
    ** Use Cases
    ** Use Case Example

I think the User Guide page is short enough that you can add the Use Case Example page to it (as an H2, bumping all the headings it contains down a level), but we an talk about that.

A familiar understanding of Mule,
https://docs.mulesoft.com/anypoint-studio/v/6/download-and-launch-anypoint-studio[Anypoint™ Studio] interface and FTPS is assumed.

=== Namespace
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Need some explanation of what users must do with the code.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will consult with Fer about the use of code. Found docs in MuleSoft that may be useful

Comment thread ftps-connector/modules/ROOT/pages/ftps-connector.adoc Outdated
== Configuration

To configure the FTPS connector, determine your endpoint configuration needs. See the Hello World application below for a configuration example.
To configure the FTPS Connector, determine your endpoint configuration needs. See the xref:use-case-example.adoc[Hello World] use case example for a configuration example.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Isn't this a prerquisite?

Comment thread ftps-connector/modules/ROOT/pages/ftps-connector.adoc Outdated
See the xref:use-case-example.adoc[Hello World] use case example for information on how to use and configure the FTPS connector.

=== Use Cases

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All bullets in this section are complete sentences, so end them all with periods.

Comment thread ftps-connector/modules/ROOT/pages/ftps-connector.adoc Outdated
Comment thread ftps-connector/modules/ROOT/pages/ftps-connector.adoc Outdated
Comment thread ftps-connector/modules/ROOT/pages/use-case-example.adoc Outdated
Comment thread ftps-connector/modules/ROOT/pages/use-case-example.adoc Outdated
In this example, the connector reads a file over FTPS and saves its contents locally.


. Create a new Mule Project using Mule Server 3.6.0 EE as runtime.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In order to be consistent with the PortX library, in which we:

  1. Provide an instruction
  2. Show the screen that appears as a result of the instruction
  3. Provide the next instruction, most likely referring to the screen above the current instruction.

...The appropriate way to sequence these instructions and screens starts with:

  1. Copy the "Create a Mule..." step and paste it below the screenshot.
  2. Then, change this step to something like "In Anypoint Studio, navigate to the Project Settings Page." (I'm assuming this is Anypoint Studio, but we should confirm.)
  3. Move all subsequent steps one spot down, as you did in Step 1 here.

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.

2 participants