Skip to content
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

revisions to Deploy chapter #197

Merged
merged 2 commits into from
Nov 21, 2014
Merged

revisions to Deploy chapter #197

merged 2 commits into from
Nov 21, 2014

Conversation

rebkwok
Copy link
Contributor

@rebkwok rebkwok commented Nov 21, 2014

Based on @judy2k's feedback from Django Girls Amsterdam - his group had some problems with the end of the Deploy chapter, where they saw an error message when they first open the app after deploying - the error was mentioned but not very clearly, and caused some confusion. I also made some English edits again to the beginning section :)

@@ -1,6 +1,6 @@
# Deploy!

__Note__ Following chapter can be sometimes a bit harder to get through. Insist on finishing it, deployment is an important part of website development process. Chapter is placed in the middle of tutorial so that mentor can help you get your website online while you can still finish the tutorial alone if you run out of time.
> __Note__ The following chapter can be sometimes a bit hard to get through. Persist and finish it; deployment is an important part of thewebsite development process. This chapter is placed in the middle of the tutorial so that your mentor can help with the slightly tricker process of getting your website online. This means you can still finish the tutorial on your own if you run out of time.
Copy link
Contributor

Choose a reason for hiding this comment

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

"thewebsite" -> "the website"

@bmispelon
Copy link
Contributor

Thanks for the feedback and suggestions as always!

I've added a few comments here and there.

bmispelon added a commit that referenced this pull request Nov 21, 2014
revisions to Deploy chapter
@bmispelon bmispelon merged commit 61a9b55 into DjangoGirls:master Nov 21, 2014
@bmispelon
Copy link
Contributor

Thanks!

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