Skip to content

Conversation

AntoineAugusti
Copy link
Contributor

@AntoineAugusti AntoineAugusti commented Apr 18, 2016

This mostly coming from https://gist.github.com/arikfr/9d061bedb55ccd137f30 and this is hopefully just a start to a great and complete documentation!

If you want me to add something before merging this, don't hesitate. I'm not familiar with everything in your Gist so a quick explanation would be awesome ;)

docs/setup.rst Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Only on AWS it's ubuntu. On GCE they add the users you setup in their management console.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@arikfr Leaving it as it is now because the GCE section is just after and I don't think users will be confused.

Copy link
Member

Choose a reason for hiding this comment

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

👍 Sorry, I forgot this section is in the context of AWS only.

@arikfr
Copy link
Member

arikfr commented Apr 18, 2016

Thanks! I really appreciate the effort to improve documentation.

@AntoineAugusti
Copy link
Contributor Author

@arikfr Updated the documentation thanks to your comments. I've also squashed commits

@arikfr arikfr merged commit 2655eec into getredash:master Apr 18, 2016
@arikfr
Copy link
Member

arikfr commented Apr 18, 2016

🎉

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