pivotal-cf / docs-a9s-elasticsearch

1 stars 5 forks source link

Integrate anynines first version of the docs #2

Closed lucaspinto closed 7 years ago

lucaspinto commented 7 years ago

This PR consists of 3 mains changes:

Cheers, Lucas

cf-gitbot commented 7 years ago

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

jncd commented 7 years ago

Thanks @lucaspinto ! We'll take a look.

jncd commented 7 years ago

Hello @lucaspinto The doc looks good. I edited the using page of the doc to make to it procedural — made steps to make it clear what the developers have to do. I have more questions about the installing page. The first part is a very clear procedure, but then it turns into reference material... the numbered steps go away. I just want to make sure that means there are no configuration steps that the operator has to do to get a9s Elasticsearch working in a vanilla system.
Are you planning a troubleshooting page that will link back to this reference material? For example, the doc says "You can configure the dimensions of VMs..." but it doesn't say why or when you would do this. TIA, Jane

gogolok commented 7 years ago

Hey @jncd Maybe we should create an issue for your feedback/suggestions? Someone from our team can pick the issue, work on it and create incremental PRs for example.

jncd commented 7 years ago

Very good idea. Done!

On Wed, Jan 4, 2017 at 2:29 PM, Robert Gogolok notifications@github.com wrote:

Hey @jncd https://github.com/jncd Maybe we should create an issue for your feedback/suggestions? Someone from our team can pick the issue, work on it and create incremental PRs for example.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pivotal-cf/docs-a9s-elasticsearch/pull/2#issuecomment-270505962, or mute the thread https://github.com/notifications/unsubscribe-auth/AP1RdvdgXl9qMrKmRYAzHNoebdfDlAyTks5rPB1cgaJpZM4LXDuy .